程序将无法正确读取文本文件

Question

好的，我知道代码可能很长，但我尽可能地尽量减少它。我想让代码工作，以便重新创建我的问题。我的问题是，当我尝试读取文本文件时，它不会读取所有项目。它似乎只读了最后几个。我可能无意中改变了一些东西，因为它以前工作得很好。当你在程序和注册项目，它计数项目就好了。但是，当您打开文件时，只是将项目保存在其中。它计数的项目数量少于实际文件中的项目数量，并且阵列全部错误。如果任何人都可以在我的代码中看到问题，那将非常感激。

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

#define MAX 20 

struct items 
{ 
    int itemnumber; 
    char name[30]; 
    int balance; 
}; 

void open_file(FILE *enter_filename, char filename[], struct items aItems[], int *num_items) 
{ 
    int i=0; 

     printf("Choose filename (.txt).\n"); 
     scanf("%19s", filename); 
     enter_filename=fopen(filename, "r+"); 
     if(enter_filename) 
     { 
      while(!feof(enter_filename)) 
      { 
       for(i = 0; i < *num_items; i++) 
        { 
        fscanf(enter_filename, "Itemnumber: %d\n", &aItems[i].itemnumber); 
        fscanf(enter_filename, "Name: %s\n", aItems[i].name); 
        fscanf(enter_filename, "Balance: %d\n", &aItems[i].balance); 
        } 
       if(!feof(enter_filename)) 
       { 
        *num_items=*num_items + 1;  
       } 
      } 
      printf("\nNumber of items: %d \n",*num_items); 
      fclose(enter_filename); 
     } 
     else 
     { 
      printf("That file doesn't exist! Create a new one.\n"); 
      printf("What name do you want for your new file?\n"); 
      scanf("%19s", filename); 
      enter_filename=fopen(filename, "w+"); 
      printf("File is created!\n"); 
      *num_items = 0;     
      fclose(enter_filename); 
     } 
} 
void register_item(struct items *aItems, int *num_items) 
{ 
    int success=1; 
    if(*num_items < MAX) 
     { 
     while(1) 
     { 
      printf("Item number:\n");        
      scanf("%d", &aItems[*num_items].itemnumber); 
      for(int i=0; i < *num_items; i++) 
      { 
      if(aItems[*num_items].itemnumber == aItems[i].itemnumber) 
       { 
       printf("Item number already exists, choose a unique item number.\n"); 
       success=0; 
       break; 
       } 
      else 
       { 
       success=1; 
       } 
      } 
     if(success)break; 
     } 
     printf("Name:\n"); 
     scanf("%29s", aItems[*num_items].name); 
     strlwr(aItems[*num_items].name); 
     printf("Balance:\n"); 
     scanf("%d", &aItems[*num_items].balance); 
     *num_items+=1; 
     } 
} 
void print_item(struct items aItems[], int num_items) 
{ 
    int i; 
    for (i=0; i < num_items; i++) 
    { 
    printf("%d. Item number: %d Name: %s Balance: %d\n", i+1, aItems[i].itemnumber, aItems[i].name, aItems[i].balance);   
    } 
} 
void quit_program(char filename[], struct items aItems[], int *num_items) 
{ 
    FILE *fil; 
    fil=fopen(filename, "w+");            
    int i; 
    for(i = 0; i < *num_items; i++) 
     { 
     fprintf(fil, "Itemnumber: %d\n", aItems[i].itemnumber); 
     fprintf(fil, "Name: %s\n", aItems[i].name); 
     fprintf(fil, "Balance: %d\n\n", aItems[i].balance); 
     } 
    fclose(fil); 
} 
int main(void) 
{ 
    FILE *enter_filename; 
    struct items aItems[MAX]; 

    int menu, num_items=0; 
    char filename[20]; 

    open_file(enter_filename,filename, aItems, &num_items); 

    while(menu!=3) 
    { 
     printf("\n"); 
     printf("1. Register new items to inventory.\n"); 
     printf("2. Print all items from inventory.\n"); 
     printf("3. Quit\n"); 
     scanf("%d", &menu); 

     if(menu==1) 
     { 
      register_item(aItems, &num_items); 
     } 

     if(menu==2) 
     { 
      print_item(aItems, num_items); 
     } 

     if(menu==3) 
     { 
     quit_program(filename, aItems, &num_items); 
     } 
    } 
return 0; 
} 

Answer 1

你的错误是在这里：

 while(!feof(enter_filename)) 
     { 
      for(i = 0; i < *num_items; i++) 
       { 
       fscanf(enter_filename, "Itemnumber: %d\n", &aItems[i].itemnumber); 
       fscanf(enter_filename, "Name: %s\n", aItems[i].name); 
       fscanf(enter_filename, "Balance: %d\n", &aItems[i].balance); 
       } 
      if(!feof(enter_filename)) 
      { 
       *num_items=*num_items + 1;  
      } 
     } 

在第一次通过你的循环

，*num_items是零，所以核能研究所循环，它的实际读数将不会进入和aItems[i]将未初始化。然后你增加物品计数器。

在下一回合中，只会读取一个项目，即aItems[1]，但会收到第一个项目的数据。您增加计数器。在第三遍中，您读取的是aItems[2]，但立即覆盖它，因为内部循环会创建与数组中当前元素一样多的遍历。显然，这是错误的。

当您阅读文件时，您不知道有多少物品。因此，您必须阅读一个项目，测试它是否可以读取，然后相应地增加计数器。测试文件是否结束或者输入是否正确是通过fscanf的返回值完成的，该值返回已成功转换的项目数。

你的循环可以这样工作的：

 while (*num_items < MAX) { 
      struct items *p = &aItems[*num_items]; 

      if (fscanf(f, "Itemnumber: %d\n", &p->itemnumber) < 1 
      || fscanf(f, "Name: %s\n", p->name) < 1 
      || fscanf(f, "Balance: %d\n", &p->balance) < 1) { 
       break; 
      } 

      (*num_items)++; 
     } 

（我称为文件处理f：enter_filename既是太长太误导）

其他注意事项：

• 文件句柄和文件名是本地函数;它们不在外面使用，手柄可以正确打开和关闭。因此，你不应该将它们作为参数传递，而是将它们作为局部变量。
• 变量menu未初始化，启动程序时可能为3。

Answer 2

在你open_file功能使用while(!feof(enter_filename))。

这不是读取文件的一种可靠的方法，as it is stated in this question. 对于你的情况，你最终是什么，既然你在一个错误的while(!feof(enter_filename))回路设置num_items，你num_items持有一个错误的值，并将它传播通过你们的节目因为您几乎在任何地方都在使用它，特别是在再次写回文件时，这解释了缺失的行。一旦执行here或here中提到的方法之一，请使用您的调试器确保num_items与您的输入文件一致。