为什么我的文件输出被覆盖？

Question

我的程序接受用户输入并将其存储在我定义为结构的记录数组中：struct Record.
用户输入是结构的字段。一切都没有错误，但似乎我无法正确设置格式。我的程序一直要求用户输入，直到用户在询问是否还有记录时输入 'n'。
一旦没有更多的记录，程序将循环遍历创建的记录，文件将打印每条记录，它们之间用制表符隔开，最后以换行符开始下一条记录。但是，它不是从新的一行开始并以相同的方式打印另一条记录，而是覆盖打印的前一条记录并进一步标记下一条记录。
是什么原因导致这种情况发生？

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

struct Record
{
    char fname[51];
    char lname[51];
    char address[51];
    char city[51];
    char state[51];
    char zipcode[51];
    char phoneNumber[51];
};

int main()
{
    FILE *fileWriter;
    const char filename[] = "data.txt";
    char answer = 'y';
    int size = 1;
    int i = 0;
    struct Record *records;
    struct Record *records_temp = NULL;

    while(answer == 'y' || answer == 'Y')
    {
        struct Record *records_temp = calloc((size),sizeof(*records));         
        records = records_temp;
        printf("First Name: \n");
        scanf("%s", records[size-1].fname);

        printf("Last Name: \n");
        scanf("%s", records[size-1].lname);

        printf("Address: \n");
        scanf(" %[^\n]", records[size-1].address);

        printf("City: \n");
        scanf("%s", records[size-1].city);

        printf("State: \n");
        scanf("%s", records[size-1].state);

        printf("Zipcode: \n");
        scanf("%s", records[size-1].zipcode);

        printf("Phone Number: \n");
        scanf("%s", records[size-1].phoneNumber);
        //stores all record info

        printf("Are there anymore records? [y/n] ");
        scanf(" %c", &answer);
        if(answer == 'y' || answer == 'Y')
        {
            size++;
            printf("\n");
        }
    }
        //open file

    fileWriter = fopen(filename,"wb");

    if(fileWriter != NULL)
    {
        for(;i< size; i++)
        {
            fprintf(fileWriter,"%s\t",records[i].fname);
            fprintf(fileWriter,"%s\t",records[i].lname);
            fprintf(fileWriter,"%s\t",records[i].address);
            fprintf(fileWriter,"%s\t",records[i].city);
            fprintf(fileWriter,"%s\t",records[i].state);
            fprintf(fileWriter,"%s\t",records[i].zipcode);
            fprintf(fileWriter,"%s\n",records[i].phoneNumber);
        }
        free(records);
        fclose(fileWriter);
    }
    else
    {
        printf("Error opening file.");   
    }
}

Answer 1

使用 "append" 中的其他 a 模式打开文件。那有什么用我想不用多解释了吧？

但是，实际问题似乎是您覆盖了指向输入循环中已有记录的指针。那应该如何工作？只需通过该循环并尝试按照程序的说明进行操作即可。

问题是你不能这样做。为分配的块使用链表。

Answer 2

while(answer == 'y' || answer == 'Y')
{
    struct Record *records_temp = calloc((size),sizeof(*records));

    records = records_temp;
    ...
}

你是不是这个意思？

while(answer == 'y' || answer == 'Y')
{
    struct Record *records_temp = realloc(records, size * sizeof *records);
    if (records_temp == NULL)
    {
         /* Handle allocation error */
    }
    records = records_temp;
    /* ... */
}

不要混淆calloc with realloc。如果您需要进一步说明，请阅读手册。

不要忘记将 records 初始化为 NULL...

如果您关心的是优化，这里最重要的瓶颈将是您的文件 input/output。这是不可避免的，除了研究 setvbuf 之外，您无能为力。下一个瓶颈将是对内核分配函数的底层调用。您可以通过减少调用分配函数来减少瓶颈。例如，您可以通过每次将其大小加倍而不是添加 1:

来增加数组

size_t size = 0;
int answer;
do {
    size_t index = size++;
    if ((index & size) == 0) {
        void *temp = realloc(array, (2 * index + 1) * sizeof *array);
        if (temp == NULL) {
            /* Handle allocation error */
        }
        array = temp;
    }

    puts("First Name:");
    scanf("%s", array[index].fname);

    /* snip */

    answer = getchar();
} while (answer != EOF && tolower(answer) == 'y');

或者，您可以重新发明轮子并执行与 realloc 在幕后执行的相同工作（并且可能会失去一些优化的好处），方法是添加对 memcpy 和 free 像这样添加到您的代码中：

while(answer == 'y' || answer == 'Y')
{
    struct Record *records_temp = calloc((size),sizeof(*records));
    if (records_temp == NULL)
    {
        /* Handle allocation error */
    }
    if (records != NULL)
    {
        memcpy(records_temp, records, (size - 1) * sizeof *records);
        free(records);
    }
    records = records_temp;
    /* ... */
}

P.S。如果您第一次错过了它：不要忘记将 records 初始化为 NULL...

Answer 3

我看到的最大问题是内存分配逻辑。第一次通过循环时，您为 1 条记录分配内存并增加大小。第二次循环，你又为 2 条记录分配了内存，因为 size == 2。第三次循环，你又为 3 条记录分配了内存，总共 1+2+3=6。

这回答了为什么会这样。 Olaf 建议的修复方法，链表，是一个很好的方法。

Answer 4

我稍微修改了你的代码，但我认为你应该在这里使用链表作为数据结构，它更简单并且消耗更少的内存。我做了一些尝试，一切顺利。 :) 希望对你有帮助！！

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

typedef struct Record Record;

struct Record
    {
        char fname[51];
        char lname[51];
        char address[51];
        char city[51];
        char state[51];
        char zipcode[51];
        char phoneNumber[51];

        Record *next;
    };


int main()
{
    FILE *fileWriter;
    const char filename[] = "data.txt";
    char answer = '[=10=]';
    // int size = 1;
    // int i = 0;
    Record *records = NULL;
    Record *records_first = NULL;
    Record *records_previous = NULL;

    fileWriter = fopen(filename,"wb");

    if(fileWriter != NULL) {

        for( ; ; ) {
            records = (Record*) malloc(sizeof(Record));  

            if(records_first == NULL)
                records_first = records;

            if(records_previous != NULL)
                records_previous->next = records;

            records = records_first;
            printf("First Name: \n");
            scanf("%s", records->fname);
            fprintf(fileWriter,"%s\t",records->fname);

            printf("Last Name: \n");
            scanf("%s", records->lname);
            fprintf(fileWriter,"%s\t",records->lname);

            printf("Address: \n");
            scanf(" %[^\n]", records->address);
            fprintf(fileWriter,"%s\t",records->address);

            printf("City: \n");
            scanf("%s", records->city);
            fprintf(fileWriter,"%s\t",records->city);

            printf("State: \n");
            scanf("%s", records->state);
            fprintf(fileWriter,"%s\t",records->state);

            printf("Zipcode: \n");
            scanf("%s", records->zipcode);
            fprintf(fileWriter,"%s\t",records->zipcode);

            printf("Phone Number: \n");
            scanf("%s", records->phoneNumber);
            fprintf(fileWriter,"%s\t\n\n",records->phoneNumber);

            records->next = NULL;
            records_previous = records;

            printf("Are there anymore records? [y/n] ");
            scanf(" %c", &answer);

            if(tolower(answer) != 'y') {
                free(records);
                fclose(fileWriter);
                break;
            }
        }

    } else
        printf("Error opening file.");

    return 0;
}

为什么我的文件输出被覆盖？

Why is my file output overwritten?

c

formatting

struct

file