c - 程序在ID号调用时循环，并且没有在数组中输入信息

Question

因此，我的程序试图从用户那里获取一些信息，并将其添加到结构体数组中。如果可以的话，我将不胜感激。程序不断循环显示“请输入有效的ID号：”，但我不知道为什么。同样，当以后使用打印功能时，用户的信息也不会被添加到阵列中。

void add_employee(Employee *db, int db_size){
    int tst1, tst2, tst3, tst4 = 0;
    char temp_first[MAXNAME];
    char temp_last[MAXNAME];
    int temp_six_digit_id;
    int temp_salary;
    db_size++;
    printf("Please enter a first name:\n");
    if(scanf("%s", temp_first) == 1){tst1 == 1;}
    else{
            printf("Please enter a valid name.\n");
            while(tst1 == 0){
                    if(scanf("%s", temp_first) == 1){tst1 == 1;}
                    else{printf("Please enter a valid name.\n");}
                    }
    }
    printf("Please enter a last name:\n");
    if(scanf("%s", temp_last) == 1){tst2 == 1;}
    else{
            printf("Please enter a valid name.\n");
            while(tst2 == 0){
                    if(scanf("%s", temp_last) == 1){tst2 == 1;}
                    else{printf("Please enter a valid name.\n");}
                    }
    }
    printf("Please enter the six digit ID number:\n");
    if(scanf("%d", &temp_six_digit_id) == 1){tst3 ==1;}
    else{
            printf("Please enter a valid ID number:\n");
            while(tst3 == 0){
                    if(scanf("%d", &temp_six_digit_id) == 1){tst3 == 1;}
                    else{printf("Please enter a valid ID number:\n");}
                    }
    }
    printf("Please enter the salary:\n");
    if(scanf("%d", &temp_salary) ==1){tst4 == 1;}
    else{
            printf("Please enter a valid Salary:\n");
            while(tst3 == 0){
                    if(scanf("%d", &temp_salary) == 1){tst3 == 1;}
                    else{printf("Please enter a valid Salary:\n");}
            }
    }
    strcpy(db[db_size].first_name, temp_first);
    strcpy(db[db_size].last_name, temp_last);
    db[db_size].six_digit_id = temp_six_digit_id;
    db[db_size].salary = temp_salary;

}

Answer 1

继续进行注释，使用scanf读取用户输入时涉及许多注意事项。这就是为什么，作为一个普遍的主张，优先选择面向行（例如fgets或getline），它将消除由于'\n'留在输入缓冲区（此处为stdin）而引起的问题。 fgets和getline最多读取并包括换行符（当用户按键盘上的Enter键时生成。使用任意一种读取后，只需将读取的字符串传递给sscanf即可从读取的字符串中解析所需的信息。

也就是说，您可以使用scanf处理输入，但是您有责任处理'\n'。

在查看scanf之前，让我们谈一谈当前以void类型输入的功能。这完全是对回报的浪费，可以用来指示输入的成功或失败。您将如何处理用户取消输入的操作（例如，使用Ctrl + d（在Linux上）或Ctrl + z（关闭））在函数中进行输入时，请始终提供可用于指示成功/失败的返回值。在这里，您可以简单地返回指向新节点的指针以指示成功，或者返回NULL指示失败。这样，您的函数将是Employee *类型而不是void。

就是说，具有scanf格式说明符的%s读取的内容最多，但不包括第一个空格或'\n'。因此，当用户输入name并按Enter时，会在变量中捕获"name"，但'\n'仍保留在stdin中。如果下一个输入是数字类型，或键入char，则scanf会很乐意使用'\n'作为用户的值（鉴于scanf跳过了提示）。为了防止scanf将'\n'用作下一个值，而不管类型如何，可以在格式说明符之前加上space（例如" %s"），这将导致scanf忽略第一个值之前的所有空格。非空白字符（'\n'被视为空白）。

您还可以将分配抑制运算符用于scanf。 （例如"%s%*c"），其中'*'之前的c告诉scanf读取并丢弃下一个字符（不影响scanf返回的匹配计数）。您可以在man scanf中阅读。 （在读取最多可包含嵌入式空格的字符时也更有意义，例如，" %99[^\n]%*c"表示最大的100 char字符串-包括nul-byte）

要使用scanf读取不包含空格的字符串，可以确保收到输入，同时还允许用户使用手动EOF取消输入而没有所有tst1, tst2, ...变量。以temp_first值为例。您可以执行以下操作：

    int return_val;  /* capture the return of scanf */
    ...
    printf ("\nPlease enter a first name: ");
    while ((return_val = scanf (" %s", temp_first)) != 1) {
        if (return_val == EOF)      /* trap manual EOF  */
            return NULL;            /* return NULL for error */
        fprintf (stderr, "error: invalid first name.\n");
        printf ("Please enter a first name: ");
    }

这将以更加直接的方式完成您尝试使用所有其他 tst值的操作。

当您阅读 temp_six_digit_id时，将输入作为字符串更有意义。这样就可以简单地调用 6来确定您确实具有 strlen位。在这里，您可以执行以下操作：

    printf ("Please enter the six digit ID number: ");
    while ((return_val = scanf (" %s", temp_id)) != 1 ||
            strlen (temp_id) != 6) {
        if (return_val == EOF)
            return NULL;
        fprintf (stderr, "error: invalid ID.\n");
        printf ("Please enter the six digit ID number: ");
    }
    temp_six_digit_id = (int)strtol (temp_id, NULL, 10);
    /* strtol validations omitted (see man strtod example) */

（您也可以将 salary阅读为字符串，并保留 strtol提供的错误检查/处理）。

下面是一个简短的示例，将所有部分组合在一起，可以在完整代码之外进行试验。您将不得不使其适应其余的代码，但这应该是相对琐碎的。

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

/* constants, max(employees, ID, salary, name) */
enum { MAXE = 3, MAXID = 32, MAXSAL = 64, MAXNAME = 256 };

typedef struct {
    char first_name[MAXNAME],
        last_name[MAXNAME];
    int six_digit_id,
        salary;
} Employee;

int main (void) {

    Employee db[MAXE] = {{ .first_name = "" }};
    char temp_first[MAXNAME] = "",
         temp_last[MAXNAME] = "",
         temp_id[MAXID] = "",
         temp_sal[MAXSAL] = "";
    int temp_six_digit_id = 0,
        temp_salary = 0,
        return_val = 0,
        db_size = 0;

    while (db_size < MAXE) {

        printf ("\nPlease enter a first name: ");
        while ((return_val = scanf (" %s", temp_first)) != 1) {
            if (return_val == EOF)      /* trap manual EOF  */
                return 1;           /* return NULL for error */
            fprintf (stderr, "error: invalid first name.\n");
            printf ("Please enter a first name: ");
        }

        printf ("Please enter a last name: ");
        while ((return_val = scanf (" %s", temp_last)) != 1) {
            if (return_val == EOF)
                return 1;
            fprintf (stderr, "error: invalid last name.\n");
            printf ("Please enter a last name: ");
        }

        printf ("Please enter the six digit ID number: ");
        while ((return_val = scanf (" %s", temp_id)) != 1 ||
                strlen (temp_id) != 6) {
            if (return_val == EOF)
                return 1;
            fprintf (stderr, "error: invalid ID.\n");
            printf ("Please enter the six digit ID number: ");
        }
        temp_six_digit_id = (int)strtol (temp_id, NULL, 10);
        /* strtol validations omitted (see man strtod example) */

        printf ("Please enter the salary: ");
        while ((return_val = scanf (" %s", temp_sal)) != 1) {
            if (return_val == EOF)
                return 1;
            fprintf (stderr, "error: invalid salary.\n");
            printf ("Please enter a the salary: ");
        }
        temp_salary = (int)strtol (temp_sal, NULL, 10);

        strcpy(db[db_size].first_name, temp_first);
        strcpy(db[db_size].last_name, temp_last);
        db[db_size].six_digit_id = temp_six_digit_id;
        db[db_size].salary = temp_salary;

        db_size++;  /* 'db_size' is a copy here, changes are not visible in the
                     caller, pass a pointer to db_size to make change visible */
    }

    for (int i = 0; i < db_size; i++)
        printf ("\n name   : %s %s\n ID     : %d\n salary : %d\n",
                db[i].first_name, db[i].last_name, db[i].six_digit_id,
                db[i].salary);

    return 0;
}

（请特别注意有关 db_size的注释。如果希望在调用函数中看到对 db_size的更改，则需要将指针传递给 db_size而不是 db_size本身。）

示例使用/输入

$ ./bin/emp_input

Please enter a first name: Mary
Please enter a last name: Jane
Please enter the six digit ID number: 123
error: invalid ID.
Please enter the six digit ID number: 123456
Please enter the salary: 937

Please enter a first name: John
Please enter a last name: Doe
Please enter the six digit ID number: 234567
Please enter the salary: 1024

Please enter a first name: Bill
Please enter a last name: Jones
Please enter the six digit ID number: 234568
Please enter the salary: 1020

 name   : Mary Jane
 ID     : 123456
 salary : 937

 name   : John Doe
 ID     : 234567
 salary : 1024

 name   : Bill Jones
 ID     : 234568
 salary : 1020

注意：有很多方法可以处理用户输入。选择该方法是为了尽可能接近您最初的尝试。如果您有疑问或在进一步实施方面遇到问题，请告诉我，我们将很乐意为您提供进一步的帮助。