程序无法从文本文件中正确读取

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

在您的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 与您的输入文件一致。

Answer 2

你的错误在这里：

        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。