5

我正在尝试读取从 C 转换为 C++ 的电话簿应用程序的文件末尾。当我从文件中打印结果时，我得到了这个：

johnny smith
(Home)3
(Cell)4
x☺> x☺>
(Home)4
(Cell)4

它应该打印：

johnny smith
(Home)3
(Cell)4

现在我正在使用while(!infile.eof())我读过的一个不好的做法，但是当我使用时，infile.getline()我会重复名字和姓氏，并且格式都被抬高了。无论如何（或另一种方式）来摆脱输入末尾的垃圾或另一种在 C++ 中读取到文件末尾的方式来解决这个问题。我一直在阅读不同的解决方案，但很多网站似乎都同意的是fgets，这是我在原始 C 版本中所拥有的，但显然fgets不适ifstream用于我正在使用的。这是代码：

void contacts:: readfile(contacts*friends ,int* counter, int i,char buffer[],char    user_entry3[])
{
   ifstream read;
   read.open(user_entry3,ios::in);
   int len;
   contacts temp;
   *counter=0;
   i=0; 

     while (!read.eof()) { 
       temp.First_Name=(char*)malloc(36); 
       temp.Last_Name=(char*)malloc(36); 

       read>>temp.First_Name>>temp.Last_Name;

       read>>buffer;
       len=strlen(buffer);
       if(buffer[len-1]=='\n')
          buffer[len-1]='\0';

       temp.home=(char*)malloc(20); 
       strcpy(temp.home, buffer);

       read>>buffer;
       len=strlen(buffer);
       if(buffer[len-1]=='\n')
       buffer[len-1]='\0';


       temp.cell=(char*)malloc(20); 
       strcpy(temp.cell, buffer); 

      friends[i].First_Name=(char*)malloc(MAXNAME);
      friends[i].Last_Name=(char*)malloc(MAXNAME);
      friends[i].home=(char*)malloc(MAXPHONE);
      friends[i].cell=(char*)malloc(MAXPHONE);


  //adds file content to the structure
      strcpy(friends[*counter].First_Name,temp.First_Name);
      strcpy(friends[*counter].Last_Name,temp.Last_Name);
      strcpy(friends[*counter].home,temp.home);
      strcpy(friends[*counter].cell,temp.cell);


     (*counter)++;
     i++; 

   }
   //closes file and frees memory
    read.close();
    free(temp.Last_Name);
    free(temp.First_Name);
    free(temp.home);
    free(temp.cell);
}
4

2 回答 2

7

不要用于确定您是否到达文件末尾。eof()相反，阅读您想要阅读的内容，然后检查您是否成功读取数据。Obce reading failed 您可以eof()在生成有关格式错误的错误报告之前确定错误是否已到达文件末尾。

既然你提到你读到使用!infile.eof()是一种很好的做法：你能指出这个错误信息的来源吗？此信息需要更正。

于 2012-11-24T18:36:37.053 回答
7

  1. 不要使用!eof(). 它检查最后一次读取失败是否是由于到达文件末尾。它不能预测未来。

  2. 不要malloc在 C++ 中使用。如果这样做，请检查返回值是否有错误！

  3. 不要operator>>用于char *. 没有大小检查，所以只是要求缓冲区溢出。

  4. '\n'对缓冲区的检查是无用的。operator>>对于字符串在空格处停止。

  5. 你盲目地strcpy将一个长度未知的字符串插入temp.home大小为 20 的字符串。这是另一个缓冲区溢出。

  6. ...我有点停止阅读那里。如果您想从文件中读取内容但在 eof/error 上停止，您可以执行以下操作：

.

string a, b, c;
while (true) {
    if (!(in >> a)) break;
    if (!(in >> b)) break;
    if (!(in >> c)) break;
    do_stuff_with(a, b, c);
}
于 2012-11-24T18:40:33.747 回答