执行后C程序挂起

Question

我是C编程新手。 我编写了一个非常简短的程序来合并文件夹中的所有文件。 该程序运行并产生正确的输出，但是执行后挂起，我必须手动将其杀死。 有什么想法吗？

这里的重要功能是scandir和append_to_file

/*
MERGE: Merges text files. Gives the ability to merge a list of files or all files in a 
directory with specified extension.
*/
#include <stdio.h>
#include <dirent.h>
#include <stdbool.h>
#include <string.h>
#include <stdlib.h>

/* Function prototypes */
int append_to_file(const char *filename, const char *outfilename); // Appends the contents of filename to outfilename
int scandir(char dirname[], char const *ext, char outfile[]); // Scans a directory for files of a specific extension and merges them
bool has_extension(char const *name, char const *ext);
void usage(); // Prints out usage information (help) to the console
void path_combine(char *dest, const char *path1, const char *path2); // Combines a directory name and filename to a single filepath

int main(int argc, char *argv[])
{    

    int i, // Counters
        nfiles; // Number of files merged

    if (argc == 4)
    {
        nfiles = scandir(argv[1], argv[2], argv[3]);      
        printf("Merged %s files\n", nfiles);

        return 0;  
    }
    else
    {
        printf("Wrong input, quitting");
        return 1;
    }

}

int append_to_file(const char *filename, const char *outfilename)
{
    FILE *infile, *outfile;
    char ch;
    infile = fopen(filename, "r");
    outfile = fopen(outfilename, "a");

    if (infile == NULL)
    {
        printf("Input file is empty, skipping...\n");
        return 1;
    }

    while ((ch = fgetc(infile)) != EOF)
        fputc(ch, outfile);

    fclose(infile);
    fclose(outfile);

    return 0;

}

int scandir(char dirname[], char const *ext, char outfile[])
/* Scans a directory and merges all files of given extension */
{
    DIR *d = NULL;
    struct dirent *dir = NULL;
    char filepath[strlen(dirname) + 255];
    int i = 0;   

    d = opendir(dirname);

    if (d)
    {
        while ((dir = readdir(d)) != NULL)
        {   
            if (has_extension(dir->d_name, ext))
            {   

                path_combine(filepath, dirname, dir->d_name);
                printf("%s\n", filepath);
                append_to_file(filepath, outfile);
                i++;
            }

        }
        closedir(d);
    }
    return i;
}


bool has_extension(char const *name, char const *ext)
{
    size_t len = strlen(name);
    return len > 4 && strcmp(name+len-4, ext) == 0;
}


void path_combine(char *dest, const char *path1, const char *path2)
{
    const char *last_char = path1;
    int append_sep = 0;
    char sep[] = "/";

#ifdef WIN32
    sep[0] = '\\';
#endif

    /* Find the last character in the first path*/
    while(*last_char != '\0')
        last_char++;

    /* If the last character is not a seperator, we must add it*/
    if (strcmp(last_char, sep) !=0)
    {
        append_sep = 1;
    }

    strcpy(dest, path1);
    if (append_sep)
        strcat(dest, sep);
    strcat(dest, path2);       

}


void usage()
{
    printf("\t=================\n");
    printf("\t      MERGE\n");
    printf("\t=================\n");
    printf("Merge two or more text files\n");
    printf("Usage:\n");
    printf("\tCall merge with a directory name and desired extension:\n");
    printf("\tmerge DIRNAME .csv OUTPUTFILE\n\n");

};

Answer 1

如编译器警告您（如果您使用gcc -Wall -g编译），则以下行：

    printf("Merged %s files\n", nfiles);

是错误的，因为nfiles是一个int 。 你可能想要

    printf("Merged %d files\n", nfiles);

阅读有关未定义行为的信息 。 你有一个。 请仔细阅读您正在使用的每个函数的文档，从printf（3） ＆ fopen（3） ＆ perror（3） ＆ exit（3）开始 。 不要忘记处理失败，例如：

FILE *infile, *outfile;
char ch;
infile = fopen(filename, "r");
outfile = fopen(outfilename, "a");

if (infile == NULL) {
    printf("failed to open %s (%s), skipping...\n", 
           filename, strerror(errno));
    return 1;
}
if (outfile == NULL) {
    perror(outfilename);
    exit(EXIT_FAILURE);
}

了解如何使用调试器（ gdb ）。 如果在Linux上，请同时使用strace（1）和valgrind 。

Answer 2

啊。 我以为我在使用调试器，通过在编译时指定-g ：

gcc main.c -g -o main.exe

但是它仍然挂着。

如果我包含标志-Wall和-Werror它很快就会告诉我我有字符串格式错误。

需要将行printf("Merged %s files\\n", nfiles)更改为printf("Merged %d files\\n", nfiles) 。

使用-Wall和-Werror进行编译很快指出了该错误。