函数中fgets()无法工作

Question

函数中fgets()无法工作

3

我正在编写一个程序，使用fgets()扫描一个3个字符长度的字符串作为起飞和到达机场的代码。当我在方法外部编写语句（在主函数中）时，它能正常工作，但在方法内部则无法等待输入并留下变量为空。

void newFlight ()
    {
        printf("\n");
        printf("============= CREATE A NEW FLIGHT ============= \n");
        printf("Type 0 at any point to exit to main menu. \n");
        printf("\n");

        printf("EnterFlight ID (0 to cancel) : ");
        scanf("%d", &flyList[curFly].flightID);

        char codeA [4];
        printf ("Enter Destination (Airport Code): ");
        fgets(codeA, 3, stdin);
        strcpy(codeA, flyList[curFly].arrive);
        printf("%s   %s \n", codeA, flyList[curFly].arrive);  //TEST

        printf("Enter Place Of Departure: ");
        char codeD[4];
        scanf("%s", codeD);
        strcpy(codeD, flyList[curFly].depart);
        printf("%s   %s \n", codeD, flyList[curFly].depart);  //TEST

        printf("Enter Date Of Departure (DD MM YYYY): ");
        scanf("%hd %hd %hd", &flyList[curFly].timeOfDep.day, &flyList[curFly].timeOfDep.month, &flyList[curFly].timeOfDep.year);

        printf("Enter Time Of Departure (HH MM)in 24Hr Format: ");
        scanf("%hd %hd", &flyList[curFly].timeOfDep.hour, &flyList[curFly].timeOfDep.minute);       

        curFly++;
    }

- Matthew Cassar

scanf保留换行符。并通过下一个strcpy(codeA, flyList[curFly].depart);进行重写。 - BLUEPIXY

@Barmar 我已经检查过了，我认为这与他遇到的迭代问题不同，我的问题在于实际扫描字符串。 - Matthew Cassar

1

%d 会将换行符留在输入缓冲区中，供下一次输入操作使用；而这个输入操作是 fgets()，它会查找直到下一个换行符的内容。输入的第一个字符是换行符；这也是 fgets() 只读取该字符的原因。这是一个标准问题，@Barmar 确认了其中一个可能的重复问题。 - Jonathan Leffler

1

最后，就我而言（目前为止），您没有检查任何输入操作，因此不知道 fgets() 是否正常工作。您必须检查每个输入操作以确保它正确工作。如果您不这样做，那么您将面临灾难。 - Jonathan Leffler

再次检查 strcpy - BLUEPIXY

显示剩余5条评论

2个回答

1

问题不是来自于 fgets，而是来自于 scanf，因为当执行以下代码行时： scanf("%d", &flyList[curFly].flightID); 并在按下回车键后，换行符仍然留在缓冲区中，所以当调用 fgets 时，它立即接收到该字符，因此表现得好像你按下了 Enter 并继续执行。防止这种情况发生的一种简单方法是在每个 scanf 后面放置一个 getchar，以便它接收换行符。

- Farouq Jouti

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- chux - Reinstate Monica · Accepted Answer

混合使用fgets()和scanf()是有问题的。 fgets()会消耗Enter (\n)。
scanf("%d", ...看到了\n，这会停止%d转换，并将\n放回stdin以供下一个IO操作使用 - 这恰好是OP的fgets()，它会迅速返回一个短字符串。

还需要检查scanf()的结果。指定"%s的宽度是好的，比如"%3s。

快速解决方案：只使用scanf()。

 //  scanf("%d", &flyList[curFly].flightID);
if (1 != scanf("%d", &flyList[curFly].flightID)) handle_error();
 ...
char codeA [4];
// fgets(codeA, 3, stdin);
if (1 != scanf("%3s", codeA) handle_error();
...
// Likely backwards
// strcpy(codeA, flyList[curFly].arrive);
strcpy(flyList[curFly].arrive, codeA);
...
char codeD[4];
// scanf("%3s", codeD);
if (1 != scanf("%3s", codeD)) handle_error();
...
// scanf("%hd %hd %hd", &flyList[curFly].timeOfDep.day, &flyList ...
if (3 != scanf("%hd %hd %hd", &flyList[curFly].timeOfDep.day, &flyList[curFly].timeOfDep.month, &flyList[curFly].timeOfDep.year)) handle_error();
...
// scanf("%hd %hd", &flyList[curFly].timeOfDep.hour, &flyList ...
if (2 != scanf("%hd %hd", &flyList[curFly].timeOfDep.hour, &flyList[curFly].timeOfDep.minute)) handle_error();

更好的解决方案：使用 fgets()/sscanf()。

 //  scanf("%d", &flyList[curFly].flightID);
char buf[100];
if (fgets(buf, sizeof buf, stdin) == NULL) Handle_EOForIOError();
if (1 != sscanf(buf, "%d", &flyList[curFly].flightID)) handle_parse_error();
... 
if (fgets(buf, sizeof buf, stdin) == NULL) Handle_EOForIOError();
if (1 != sscanf(buf, "%3s", flyList[curFly].arrive) handle_parse_error();
...
etc.

顺便提一下：scanf()格式"%hd %hd"和"%hd%hd"的作用是相同的。