1

我的程序读取在命令行指定的目录的内容。它递归地读取目录,即如果我们正在读取目录“test”的内容,并且在其中我们有另一个目录“inside”,那么它还将读取名为“inside”的目录的内容。问题是如果我不读取隐藏目录,即以“。”开头的目录,它工作正常。. 但是如果我也读了隐藏目录,它会说分段错误。

代码如下:

主文件:

#include "helper.h"

/*
 * Display's content of String array passed to it,
 * that should conatin full path to files.
 */
void display(char **);

/*
 * Free's the memory utilized by the program
 */
void cleanup(char **);

int main(int argc, char *argv[])
{
    // ensure proper usage
    if (argc != 2)
    {
        printf("Usage: %s [dir]\n\n", argv[0]);
        return 1;
    }

    char **files = calloc(1, sizeof(char *));

    // get files from the directory specified
    getFiles(&files, argv[1]);

    // display files
    display(files);

    // free memory utilized by files array
    cleanup(files);

// that's all folks
return 0;
}


/*
 * Display's content of String array passed to it,
 * that should conatin full path to files.
 */
void display(char **files)
{
    // Color Red
//  printf("[0;31;40m");

    // display files
    for (int i = 0; files[i]; i++)
    {
        printf("%s\n", files[i]);
    }

    // turn off color
//  printf("[0;37;40m");
}


/*
 * Free's the memory utilized by the program
 */
void cleanup(char **files)
{
    // free memory utilized by files array
    for (int i = 0; files[i]; i++)
        free(files[i]);
    free(files);
}

getFiles 函数在 helpers.c 文件中定义,其中包含以下代码:

#include "helper.h"
#include <dirent.h>
#include <stdlib.h>
#include <sys/types.h>

/*
 * Stores the list of files present in direectory pointed by 'dir' 
 * in array of strings pointed by 'files'
 */
void getFiles(char ***files, const char* dir)
{
    static int i;

    // ensure directory is valid
    if (dir == NULL)
    {
        printf("Error: Invalid Directory\n\n");
        exit(1);
    }

    // declare and initialize directory handler
    DIR *dd = opendir(dir);
    if (dd == NULL)
    {
        printf("Error: Directory Not Found\n\n");
        exit(2);
    }

    // structure that store file attributes read
    struct dirent *content;

    // read directory until all files are scanned
    while ((content = readdir(dd)) != NULL)
    {
        // ignore '.' and '..' directories
        if (strcmp(content->d_name, ".") == 0 || 
            strcmp(content->d_name, "..") == 0)
            continue;
        /*if (content->d_name[0] == '.')
            continue;*/

        //store full file path from current directory
        char temp[1024] = {0};

        // make full path
        makepath(temp, dir, content->d_name);

        // recall itself if another directory found
        if (isdir(temp))
        {
            // read this new directory found
            getFiles(files, temp);
            continue;
        }

        // allocate memory to store locations of char *
        *files = realloc(*files, (i + 2)*(sizeof(char *)));

        // allocate heap memory and store location
        *(*(files + 0) + i) = (char *)strdup(temp);

        // move to next location
        i++;
    }

    // free directory handler
    closedir(dd);

    // set NULL after last file name
    *(*(files + 0) + i) = '\0';
}


/*
 * returns true if 'dir' refers to a directory, false otherwise
 */
bool isdir(const char * dir)
{
    DIR *temp;
    temp = opendir(dir);

    if (temp != NULL)
    {
        closedir(temp);
        return true;
    }

return false;
}


/*
 * appends dir and file/directory name to src, 
 * thus makes a full file/directory path, from current directory
 */
void makepath(char src[], const char *dir, const char *file)
{   
    // prepend directory name
    strcat(src, dir);
    strcat(src, "/");

    // append file/directory name
    strcat(src, file);
}

我将必要的头文件包含在 helper.h 文件中。

另外我想知道我在内存分配方面是否犯了错误。(在 getFiles 函数中的 realloc 中)。

我此时评论了忽略隐藏文件行。

/*if (content->d_name[0] == '.')
                continue;*/

如果我取消注释上面的行,那么程序可以正常工作。

如果您在想为什么我将文件名存储为 readdir 函数读取的文件名,因为这些名称以后对我来说是必需的,所以这就是为什么我不立即显示文件名的原因。

任何建议我可以如何更好地实现这个程序,以及如何解决我读取隐藏目录时出现的问题。

4

1 回答 1

1

我不知道这是否是问题,但在这里:

// set NULL after last file name
*(*(files + 0) + i) == '\0';

                     ^ you are not setting to NULL, you are comparing
于 2013-10-12T17:42:37.097 回答