Проблема быстрого бронирования

Question

#include <stdio.h>

#pragma pack(push)
#pragma (1)

typedef struct contact {    
    char firstname [40];    
    char lastname [40]; 
    char address [100]; 
    char phone[10];
}contact;

#pragma pack(pop)

int main ()
{   FILE *pFile;
    contact entry = {"", "", "", ""};
    char choice;

    pFile = fopen("C:\\contacts.txt", "w+");

if(!pFile){
    printf("File could not be open");
    return 1;
    }

printf("Choose a selection\n\n");
printf("1. Enter First Name\n");
printf("2. Enter Last Name\n");
printf("3. Enter Address\n");
printf("4. Enter Phone Number\n\n");
scanf( "%d", &choice);

switch (choice){

     case 1:
        printf("First name: \n");     
        fgets(entry.firstname, sizeof(entry.firstname),pFile);
        break;
     case 2:
        printf("Last name: \n");      
        fgets(entry.lastname, sizeof(entry.lastname),pFile);
        break;
     case 3:
        printf("Address: \n");    
        fgets(entry.address, sizeof(entry.address),pFile);
        break;
     case 4:
        printf("Phone Number: \n");   
        fgets(entry.phone, sizeof(entry.phone),pFile);
        break;
     default:
        printf(" No Choice selected, Ending Address Book Entry system");
        break;
    }
        fwrite(&entry, sizeof(contact), 1, pFile);

        printf("Enter a new contact?");
        scanf("%s", &choice);

     //while(choice != 'n');

fclose(pFile);

getchar();
return 0;
}

Этот код после того, как я выбираю запись и после того, как я вставляю запись и нажимаю ввод, происходит сбой, сообщая, что стек вокруг переменной 'entry' был поврежден. Я довольно уверен, что это моя функция fwrite, которую я использую. Я знаю, что первый параметр, который ищет fwrite, - это указатель на массив элементов, которые нужно записать, но я думаю, что сейчас я просто сбит с толку. Любая помощь будет высоко ценится.

Answer 1

Вы должны изменить все свои

fgets(entry.firstname, sizeof(entry.firstname),pFile);

на

fgets(entry.firstname, sizeof(entry.firstname),stdin);

Поскольку вы читаете с консоли, а не из файла.

Также с помощью

scanf("%s", &choice);

и

scanf( "%d", &choice);

вы пытаетесь прочитать строку и цифру и сохранить ее в char.Они оба должны быть

scanf("%c", &choice);

Тем не менее, вы должны подумать о переписывании этого, используя ifstream, cin, getline и std::string, чтобы сделать вашу жизнь проще, если вы не смотритедля максимальной производительности.