get () не работает - PullRequest
       31

get () не работает

2 голосов
/ 01 октября 2009

У меня есть программа, написанная на C, и она вызывает get () от коммутатора, когда пользователь выбирает вариант 3. Вот мой код. Кажется, он не ждет, пока пользователь что-то введет. Скорее программа продолжается в коммутаторе.

void getField();

#include <stdio.h>
#include <string.h>
/*#include "stubs.c"
#include "record.h" */

int debugMode;

void getField(){
    char name[25];
    char address[80];
    int yearofbirth;
    char telno[15];
    int counter = 0;

    if(debugMode == 1){
        printf("***DEBUG*** Entering getField function \n");
    }

    printf("Enter your name:");
    gets(name);

    printf("Name: %s \n", name);
    printf("\n");
}

void main(int argc, char * argv[])
{
    struct record* start = NULL;
    int userChoice;
    debugMode = 0;

    if(argv[1] != NULL){
        if( (strcmp(argv[1], "debug") == 0) && (argv[2] == NULL) ){
            debugMode = 1;
            printf("Welcome, to the personal address book application \n");
        }
        else{
            int i = 0;
            while(argv[i] != NULL){
                printf(argv[i]);
                printf(" ");
                i++;
            }
            printf(": Command not found \n");
            userChoice = 6;
        }
    }

    if(argv[1] == NULL){
        printf("Welcome, to the personal address book application \n");
        userChoice = 0;
    }


    while(userChoice != 6)
    {
        if(debugMode == 1){
            printf("***DEBUG*** Entering do-while loop \n");
        }

        printf("Enter number corresponding number to option below \n\n");   

        printf("1) Add a new record in the database \n");
        printf("2) Modify a record in the database \n");
        printf("3) Print information about a record in the database \n");
        printf("4) Print all information in the database \n");
        printf("5) Delete an existing record from the database \n");
        printf("6) Quit program \n\n >");


        scanf("%d", &userChoice);

        switch(userChoice){

            case 1:
                /*addRecord(start, arrayHolder, arrayHolder, 0, arrayHolder);
                */userChoice = 0;
                break;
            case 2:
                /*modifyRecord(start, arrayHolder, arrayHolder, arrayHolder);
                */userChoice = 0;
                break;
            case 3:
                /*printRecord(start, arrayHolder);
                */userChoice = 0;
                getField();
                break;
            case 4:
                /*printAllRecords(start);
                */userChoice = 0;
                break;
            case 5:
                /*deleteRecord(start, arrayHolder);
                */userChoice = 0;
                break;
            case 6:
                printf("case 6 \n");
                break;
            default:
                printf("default \n");
                userChoice = 0;
                break;
        }

    }
    printf("\n");
}

Ответы [ 3 ]

9 голосов
/ 01 октября 2009

Когда вы вводите опцию с помощью вызова scanf(), вы набираете 2 клавиши на клавиатуре, например, 3 и ENTER. scanf() потребляет '3', но оставляет ENTER висящим во входном буфере.
Когда позже вы сделаете gets(), что ENTER все еще находится в буфере ввода, и это то, что gets() получает.

У вас есть два варианта:

  • очищать входной буфер после каждого scanf()
  • очищать входной буфер перед каждым gets()

Чтобы очистить входной буфер, используйте этот код:

int clear_input_buffer(void) {
    int ch;
    while (((ch = getchar()) != EOF) && (ch != '\n')) /* void */;
    return ch;
}

Oh! И прекратить использование gets(). gets() невозможно безопасно использовать. Использование fgets() вместо этого.

2 голосов
/ 01 октября 2009

Когда вы читаете число, используя scanf ("% d", ....) , новая строка, набранная после номера, все еще там, ожидает в буфере ввода, когда ваша программа позже добирается до получает . Строка, которая получает чтение, будет очень короткой и состоит только из этой новой строки.

Не используйте fflush (stdin) , так как это не гарантируется стандартом для работы. Вместо этого вы можете просто читать символы в цикле, пока не пропустите символ новой строки:

while (getchar() != '\n')
    ;

Есть и некоторые другие проблемы с вашим кодом, среди которых вы вообще не должны использовать get, поскольку он не проверяет, что строка, которую вы читаете, действительно помещается в переменную. Вместо этого используйте fgets .

0 голосов
/ 01 октября 2009

Добавьте "\ n" в строку scanf! получает читает пустую строку и CR после вашего выбора.

    scanf("%d\n", &userChoice);

А в GetField () после printf произнесите «fflush»:

void getField(){
    char name[25];
    char address[80];
    int yearofbirth;
    char telno[15];
    int counter = 0;

    if(debugMode == 1){
        printf("***DEBUG*** Entering getField function \n");
    }

    printf("Enter your name:");
    fflush(stdout);
    gets(name);

    printf("Name: %s \n", name);
    fflush(stdout);
    printf("\n");
}
...