1

这是我的函数，它在给定目录中查找常规文件，然后将它们的完整路径存储在列表中。

static my_func(const char *path, Files **list) //list - storage for file names
{
    DIR *d;
    struct dirent *dir;
    char buf[PATH_MAX + 1];

    d = opendir(path);
    if (d) {
        while ((dir = readdir(d)) != NULL) {
            if ((DT_REG == dir->d_type)) {

                realpath(dir->d_name, buf);
                List_push(list, buf);
                printf("%s\n", dir->d_name);
                // memset(buf, 0, PATH_MAX + 1);
            }
        }
    }
    closedir(d);
    return 0;
}
...
...

int main()
{
    // list creation
    // my_func call
    ...
    List_print(...)
}

预期输出：

FILE_1.txt
FILE_2.txt
FILE_3.txt
FILE_4.txt
FILE_5.txt
/home/user/c/FILE_1.txt
/home/user/c/FILE_2.txt
/home/user/c/FILE_3.txt
/home/user/c/FILE_4.txt
/home/user/c/FILE_5.txt

当前输出：

FILE_1.txt
FILE_2.txt
FILE_3.txt
FILE_4.txt
FILE_5.txt
/home/user/c/FILE_1.txt
/home/user/c/FILE_1.txt
/home/user/c/FILE_1.txt
/home/user/c/FILE_1.txt
/home/user/c/FILE_1.txt

它可以与我的链表实现有关吗？它工作正常，因为我测试了它：

List_push(list, dir->d_name)

并得到了预期的结果。这是 List_push 的实现（文件只是简单的结构char *和指向下一个元素的指针）：

void List_push(Files **head, char *x)
{
    Files *new;

    new   = malloc(sizeof(Files));

    if (NULL != new) {
        new->next = *head;
        new->text = x;
        *head = new;
    } else {
        printf("malloc error");
    }

}

此外，如您所见，我试图buf用 memset 清除，但没有成功 - 输出是：

FILE_1.txt
FILE_2.txt
FILE_3.txt
FILE_4.txt
FILE_5.txt






[console]$

是的，空格似乎被归档了一些东西（或者这些只是'\n'List_print 中的符号），所以 list 不是空的。

这里有什么问题？

4

1 回答 1

2

在List_push(list, buf);您存储指向buf列表中的指针。您对每个文件都执行此操作，因此最终会在列表中获得多个指向相同文件的指针buf。打印列表项时，它将显示buf.

为避免这种情况，您需要创建一个副本并存储它，以便在下一个文件buf重用时不会覆盖存储的数据。buf

于 2015-02-22T14:25:35.140 回答