Question

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

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]指向文件中的最后一个元素。有没有人对我的代码中有什么问题有所了解？谢谢〜

Answer 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 *指针数组（不能通过当前在问题中发布的代码来判断）。 / p>

请注意，如果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]`

Answer 2

停止使用这样的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);
}

此：

使用fgets()读取整行。
使用access()检查文件是否可以打开。请注意，这种检查总是容易出现竞争条件。
使用strcpy()复制文件名，假设filename_ii[]是正确设置的数组。

关于指针数组的C代码

2 个答案: