C、char*（字符串）的循环数组不起作用。为什么？

Question

我的 char* 数组有问题-

char *original_file_name_list[500];

while(dp=readdir(dir)) != NULL) {
   original_file_name = dp->d_name;
   original_file_name_list[counter] = original_file_name;
   printf("%s\n",original_file_name_list[0]);
   printf("%d\n",counter); 
   counter++;
}

问题是，它可以很好地打印所有文件。它应该只打印第一个文件，对吗？ 如果我尝试printf("%s\n",original_file_name_list[1]); 它不起作用，这意味着它只在第一个字符串中写入。知道为什么吗？

edit：编译器没有语法错误。

Answer 1

您根本没有复制字符串-您的file_name_list 数组也没有足够的空间容纳文件名列表-仅用于指针列表。但是dp->d_name 只是一个指向char* 的指针——你不知道指针后面的内存有效的时间有多长。因此，您必须自己制作副本。

#include <string.h>
#include <dirent.h>

int main(int argc, char** argv){
  char original_file_name_list[50][50];
  size_t counter = 0;
  while(dp=readdir(dir)) != NULL) // does work fine (ordinary reading files from dir)
  {
    size_t len = strlen(dp->d_name);
    if(len >= 50) len = 49;
    strncpy(original_file_name_list[counter], dp->d_name, len);
    original_file_name_list[counter][len] = '\0';
    printf("%d\n",counter); 
    counter++;
  }
  printf("%s\n",original_file_name_list[1]); // <- will work if you have at least 2 files in your directory
  return 0;
}

Answer 2

我不确定counter2 的用途（我已将其替换为counter），但我可以建议使用strdup() 调用以下代码来存储文件名：

char *original_file_name_list[500] = {0}; // it is better to init it here

while(dp=readdir(dir)) != NULL) {
   original_file_name_list[counter] = strdup(dp->d_name); // strdup() is ok to use
                                                          // here, see the comments
   printf("%s\n%d\n",original_file_name_list[counter], counter);
   counter++;
}

/* some useful code */

/* don't forget to free the items of list (allocated by strdup(..) )*/
for (int i = 0; i < 500; ++i) {
   free(original_file_name_list[i]);
}