0

我的目标是读取一个文件，并将该文件中的每个元素保存到一个新数组中。

rewind(fp); ii = 0; while (!feof(fp)) {
    ii ++;
    fscanf(fp, "%s\n", filename_i);
    fp_i = fopen(filename_i, "r");
    if (fp_i == NULL) {
        fprintf(stderr, "can't open input file %s \n", filename_i);
        exit(1);
    }
    filename_ii[ii] = filename_i;
    printf("%s, %d\n", filename_ii[ii],ii);
    fclose(fp_i);

}


printf("a %s %d\n",filename_ii[9],DataSize[2]);
printf("a %s %d\n",filename_ii[1],DataSize[2]);

在 while() 函数中，我的输出是每个元素，但我不知道为什么最后两个 printf() 返回相同的结果，即似乎 filename_ii[1] 和 filename_ii[9] 都指向最后一个元素文件。有人知道我的代码出了什么问题吗？谢谢~

4

2 回答 2

1

您需要使用strcpy来复制字符串。改变：

filename_ii[ii] = filename_i;             // this just assigns a pointer -
                                          // it doesn't actually copy a string

至：

strcpy(filename_ii[ii], filename_i);      // copy the *contents* of `filename_i`
                                          // to `filename_ii[ii]`

这当然假设该filename_ii数组已正确初始化，而不仅仅是一个悬空char *指针数组（无法从问题中当前发布的代码中看出）。

请注意，如果filename_ii 只是一组未初始化的char *指针，那么您可以使用strdup在一个方便的函数调用中处理内存分配和复制所有内容。在这种情况下，您可以将上面的行更改为：

filename_ii[ii] = strdup(filename_i);     // allocate memory to `filename_ii[ii]` and
                                          // copy the *contents* of `filename_i`
                                          // to `filename_ii[ii]`
于 2013-10-04T13:27:53.313 回答
1

停止使用feof()/fscanf()那样的东西，它非常脆弱，而且很难正确处理。

反而：

char line[1024]; /* or whatever makes you feel comfortable */
while(fgets(line, sizeof line, fp) != NULL)
{
  size_t len = strlen(line);
  if(len == 1) /* Ignore blank lines. */
    continue;
  if(line[len - 1] == '\n')
    line[--len] = '\0'; /* Remove linefeed. */
  if(access(line, R_OK) == 0)
    strcpy(filename_ii[ii++], line);
}

这个：

  1. 用于fgets()阅读整行。
  2. 用于access()检查文件是否可以打开。请注意，这种检查总是容易出现竞争条件。
  3. 用于strcpy()复制文件名，假设filename_ii[]是一个正确设置的数组。
于 2013-10-04T13:37:37.450 回答