Question

我试图将文件名存储在数组中。该数组位于结构中，我想存储在数组中的目录中找到的文件的名称。但是，我用来存储名称的过程似乎在此过程中损坏了2个或3个名称。我认为问题出在strdup关键字上。每当我运行我的程序时，它都会读入程序可执行文件（位于目录上方的目录中，我正在读取文件），或者存储在前几个数组位置的奇怪符号。以下是我的程序的一部分，我试图捕获和存储文件名，以及输出结果的图片：

typedef struct{
  char *filename;
}filename;

typedef struct Configs{
  char file_data_path[50];
  char event_log_path[50];
  filename *fn_data;
}Configs;

typedef struct TestConfigs{
  bool done;
  int selection;
  int attempts_counter;
  Configs tConfig;
}TestConfigs;

void read_files(struct TestConfigs *setup);

int main(void) {
  printf("Hello Test\n");
  TestConfigs setup;
  read_files(&setup);
  system("pause");
  return EXIT_SUCCESS;
}

void read_files(struct TestConfigs *setup)
{
  setup->done = false;
  setup->attempts_counter = 3;

  char cwd[1024];
  DIR *dir = NULL;
  struct dirent *pent = NULL;
  struct stat info;
  int total_num_of_files = 0;

  strcpy(setup->tConfig.file_data_path, "data/");

  chdir(setup->tConfig.frame_data_path);
  if((getcwd(cwd, sizeof(cwd))) != NULL)
  {
      printf("Current Directory: %s\n", cwd);
  }

  dir = opendir(cwd);

  if(dir != NULL)
  {
     while((pent = readdir(dir)) != NULL)
     {
         if(stat(pent->d_name, &info))
         {
             printf("ERROR: stat%s: %s\n", pent->d_name, strerror(errno));
         }
         else
         {
             if(S_ISREG(info.st_mode))
             {
                 if((strcmp(pent->d_name, ".cproject") == 0) || (strcmp(pent->d_name, ".project") == 0))
                 {
                     continue;
                 }
                 else
                 {
                     total_num_of_files++;
                 }
             }
         }
     }
     printf("# of files found: %d\n", total_num_of_files);
     rewinddir(dir);

     // SETUP ARRAY HERE!
     setup->tConfig.fn_data = malloc(total_num_of_files);
     total_num_of_files= 0;

     printf("During Storage Process:\n");
     while((pent = readdir(dir)) != NULL)
     {
         if(stat(pent->d_name, &info))
         {
             printf("ERROR: stat%s: %s\n", pent->d_name, strerror(errno));
         }
         else
         {
             if(S_ISREG(info.st_mode))
             {
                 if((strcmp(pent->d_name, ".cproject") == 0) || (strcmp(pent->d_name, ".project") == 0))
                 {
                     continue;
                 }
                 else
                 {
                     setup->tConfig.fn_data[total_num_of_files].filename = (char*)malloc(sizeof(pent->d_name));
                     setup->tConfig.fn_data[total_num_of_files].filename = strdup(pent->d_name);  // <- Possible source of the storage problem
                     printf("Filename stored in fn_data[%d] = %s\n", total_num_of_files, setup->Config.fn_data[total_num_of_files].filename);
                     total_num_of_files++;
                 }
             }
         }
     }

     printf("\n");
     printf("After Storage Process:\n");
     for(int i = 0; i < total_num_of_files; i++)
     {
         printf("Filename stored in fn_data[%d] = %s\n", i, setup->tConfig.fn_data[i].filename);
     }
  }
  closedir(dir);
}

here

如何解决前几个阵列位置中文件名的损坏存储问题？为什么只有前几个位置没有正确存储文件名，但其他位置是否正常？是strdup的问题，如果是，那么在数组中捕获和存储文件名的好方法是什么？提前谢谢！

Answer 1

此：

setup->tConfig.fn_data[total_num_of_files].filename = (char*)malloc(sizeof(pent->d_name));
setup->tConfig.fn_data[total_num_of_files].filename = strdup(pent->d_name);  // <- Possible source of the storage problem

毫无意义; strdup()将覆盖malloc()返回的指针，并且该内存将永远丢失（“泄露”），这是不好的。

您不必为strdup()分配内存，它会为您执行此操作。基本上是这样的：

char * strdup(const char *s)
{
  const size_t sz = strlen(s) + 1;
  char * const p = malloc(sz);
  if (p != NULL)
    memcpy(p, s, sz);
  return p;
}

Answer 2

setup->tConfig.fn_data = malloc(total_num_of_files);

每个文件一个字节？

需要，例如setup->tConfig.fn_data = malloc(total_num_of_files * sizeof(filename));

C编程 - Strdup没有正确捕获和存储数组中的文件名

2 个答案: