C中的fscanf，带有没有空格的文本文件

Question

我有一个名称如下的文本文件：

"玛丽","帕特里夏","琳达","芭芭拉","伊丽莎白","詹妮弗","玛丽亚","苏珊","玛格丽特",

我已使用以下代码尝试将名称放入数组中：

char * names[9];
int i = 0;
FILE * fp = fopen("names.txt", "r");

for (i=0; i < 9; i++) {
  fscanf(fp, "\"%s\",", names[i]);
}

当我尝试运行该程序时出现分段错误。我已经仔细调试过，当我尝试读取第二个名称时，我注意到错误出现了。

有人知道我的代码为什么不能工作，以及为什么会发生分段错误吗？

Answer 1

您的代码中有undefined behavior，因为您没有为在fscanf 调用中写入的指针分配内存。

您有一个由九个未初始化指针组成的数组，并且由于它们是局部变量的一部分，它们具有不确定的值，即它们将指向看似随机的位置。写入内存中的随机位置（当您调用 fscanf 时会发生这种情况）会做坏事。

解决问题的最简单方法是使用数组数组，例如

char names[9][20];

这将为您提供一个由九个数组组成的数组，每个子数组为 20 个字符（允许您的名称最长为 19 个字符）。

为了不超出范围，你还应该修改你的调用，这样你就不会读到很多字符：

fscanf(fp, "\"%19s\",", names[i]);

---

但是，您使用fscanf 函数还有另一个问题，那就是读取字符串的格式"%s" 会一直读取，直到在输入中找到空格（或直到达到限制，如果提供了字段宽度）。

简而言之：您不能使用fscanf 来读取您的输入。

相反，我建议您使用fgets 一次将整行读入内存，然后使用逗号分割字符串，例如strtok.

---

一种处理任意长行作为文件输入的方法（伪代码）：

#define SIZE 256

size_t current_size = SIZE;
char *buffer = malloc(current_size);
buffer[0] = '\0';  // Terminator at first character, makes the string empty

for (;;)
{
    // Read into temporary buffer
    char temp[SIZE];
    fgets(temp, sizeof(temp), file_pointer);

    // Append to actual buffer
    strcat(buffer, temp);

    // If last character is a newline (which `fgets` always append
    // if it reaches the end of the line) then the whole line have
    // been read and we are done
    if (last_character_is_newline(buffer))
        break;

    // Still more data to read from the line
    // Allocate a larger buffer
    current_size += SIZE;
    buffer = realloc(buffer, current_size);

    // Continues the loop to try and read the next part of the line
}

// After the loop the pointer `buffer` points to memory containing the whole line

[注意：以上代码sn-p不包含任何错误处理。]

Answer 2

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main(void) {
    char *names[9], buff[32];
    int i = 0;
    FILE *fp = fopen("names.txt", "r");

    for(i = 0; i < 9; i++) {
        if(1==fscanf(fp, "\"%31[^\"]\",", buff)){//"\"%s\"," does not work like that what you want
            size_t len = strlen(buff) + 1;
            names[i] = malloc(len);//Space is required to load the strings of each
            memcpy(names[i], buff, len);
        }
    }
    fclose(fp);
    //check print & deallocate
    for(i = 0; i< 9; ++i){
        puts(names[i]);
        free(names[i]);
    }
    return 0;
}

Answer 3

试试这个...

for (i=0; i < 9; i++) 
{
   names[i]=malloc(15);// you should take care about size
   fscanf(fp, "\"%s\",", names[i]);
}