使用 fgets() 和 strtok() 读取带有逗号分隔符的文件

Question

我有一个文本文件，其中包含三个以逗号分隔的字段。 我的文本文件内容示例：12345，真正的编程新手，BS ME 要将文件加载到程序中，我使用了下面的代码......我的问题是，有时代码可以工作，有时则不能（没有错误消息出现，程序只是自行关闭并且不继续）。我还观察到文本文件是空白的（没有写入任何内容）它会自动关闭并且不会继续。我们将非常感谢您的帮助。谢谢！

int read(){
     FILE *stream = NULL;
     int ctr;
     char linebuffer[45];
     char delims[]=", ";
     char *number[3];
     char *token = NULL;
     stream = fopen("student.txt", "rt");
     if (stream == NULL) stream = fopen("student.txt", "wt");
     else {
          printf("\nReading the student list directory. Wait a moment please...");
          while(!feof(stream)){            
                ctr=0;
                fgets(linebuffer, 46, stream);
                token = strtok(linebuffer, delims);
                while(token != NULL){
                  number[ctr] = linebuffer;
                  token = strtok(NULL, delims); 
                  ctr++;
                }
          recordCtr++;                      
          }                     
     recordCtr--;
     }
     fclose(stream);
}    

Answer 1

一旦找到令牌，您就永远不会复制它。您无法复制linebuffer，因为加载下一行时，其中的数据将被覆盖。

此行：

number[ctr] = linebuffer;

应引用 token 来保存最近找到的令牌，但事实并非如此。它可能应该是这样的：¹：

strcpy(number[ctr], token);

但是你必须更改声明以确保有空间：

char number[3][32];

显然，这会带来缓冲区溢出风险，如果有一个很长的令牌，则不会合身。如何最好地处理这个问题留作练习。 :)

¹ 为什么临时向量用于存储两个数字和一个字符串（名称）时被称为“数字”，这超出了我的理解。

Answer 2

您的 fgets() 调用需要指定 45 作为大小，否则当 fgets 写入 NULL 终止符时您会溢出缓冲区。这会将“delims”字符串设置为空字符串。

此外，即使函数声明声明它返回 int，您也不会返回任何值。

我不知道你的“struct Student”的定义是什么，但是在使用strcpy()时你可能会溢出缓冲区。您还可以减少“recordCtr”。为什么？如果无法打开文件进行写入，为什么还要打开文件进行写入？为什么？如果也失败，则对 NULL 指针调用 fclose。我怀疑这有多大帮助。

我刚刚注意到你没有初始化“number”。如果第一行没有得到三个数字，则可以从未初始化的指针中获取 strcpy()。它可能有一个 NULL 值，因此程序会出现段错误。

另外，您还有一个大小为 3 的数组，但如果您读取的行包含超过 3 个逗号分隔的字段，则会导致数组溢出。

可能还有许多其他错误。

许多程序员只是懒得去做所有好的编码实践，比如检查返回值、初始化变量等等。他们通常会得到这样的代码。如果你想成为一名真正优秀的程序员，请尝试并养成做所有这些事情的习惯，或者至少始终考虑是否需要这样做。

这段代码中有很多潜在的错误。如果一行长度超过 45 个字符会发生什么？你不删除换行符。您不会将字符串转换为数字（尽管 number[1] 似乎是字符串数据，所以为什么将其存储在名为“numbers”的数组中？）或检查 fgets 是否实际返回了任何数据或检查您有多少条数据得到。

Answer 3

让买家小心。

strtok 可能需要担心一些边缘情况。

“一、二、三” 将产生 3 个代币。

“一、三” 将产生 2 个代币。