虽然 scanf EOF 循环行为不端

Question

我是 C 的初学者，我遇到了我无法解决的问题，并且无法在此处的其他线程上找到解决方案。

我正在尝试使用以下代码从键盘输入/txt 文件中读取整数：

int grades[MAX_GRADES_LENGTH]={0}, histogram[HISTOGRAM_SIZE]={0};
int maxGradesHistogramBucket=0, median=0, gradesLength=0;
double avg=0.0;
int grade=0;

printf("Enter grades:\n");
while (scanf("%d",&grade) != EOF)
{
    grades[gradesLength]=grade;
   gradesLength=gradesLength+1;
}

我应该在 Grades[] 数组中设置这些“等级”，并沿循环计算数组的长度。

不知何故循环行为不端，似乎某些输入与循环无关，但对于某些输入（实际上大多数输入），无论是文件的实际结尾，还是终端中的 ^D 命令。

我听说 scanf() 不是最可靠的读取数据的方法，但不幸的是我不能使用其他任何方法，因为我们还没有在课堂上学习任何其他方法，所以我们只能使用 scanf()我们的作业。

我已尝试将!= EOF 更改为== 1，但都一样。

对于输入

100 0 90 10 80 20 70 30 60 40 50

例如，它工作正常。 但对于输入：

0 50 100

循环是无限的。

顺便说一句，我正在使用 macbook pro（如果重要的话）。

有什么想法吗？

Answer 1

如果您键入字母而不是数字，scanf() 将返回 0（如“零成功转换的数字”）而不是 EOF（如“没有数据可供读取”）。正确的测试是确保您获得预期数量的值——在本例中为 1：

while (scanf("%d", &grade) == 1)

如果您需要知道是否到达 EOF 或没有结果（但阅读该行的其余部分可能会解决问题），然后从 scanf() 捕获返回值：

int rc;
while ((rc = scanf("%d", &grade)) == 1)
{
}
if (rc != EOF)
    …read the rest of the line, or at least the next character, before resuming the loop…

而且，如果你真的想，你可以写：

int rc;
while ((rc = scanf("%d", &grade)) != EOF)
{
    if (rc == 1)
        grades[gradesLength++] = grade;
    else
    {
        printf("Discarding junk: ");
        int c;
        while ((c = getchar()) != EOF && c != '\n')
            putchar(c);
        putchar('\n');
        if (c == EOF)
            break;
    }
}

else 子句中的代码可以合理地放入函数中。它还可能将消息报告给标准错误而不是标准输出。让用户知道您反对的是什么是礼貌的。您可以在换行符之前停止使用不同的测试（&& !isdigit(c) && c != '+' && c != '-'，使用来自<ctypes.h> 的isdigit()）。但是，用户没有机会重新编辑他们放在字母后面的内容，因此您可能会误解他们的输入。最好把剩下的输入行扔掉，让它们重新开始。

---

作为chuxnoted，在读取可能是整数一部分的字符后，需要将该字符放回输入流中。因此，如果我要分析该行的其余部分并重新开始扫描第一个实际上可能是整数一部分的数据，我会考虑使用类似的东西：

#include <ctype.h>

static inline int could_be_integer(int c)
{
    return isdigit(c) || c == '+' || c == '-');
}

然后 else 子句可能是：

else
{
    printf("Discarding junk: ");
    int c;
    while ((c = getchar()) != EOF && c != '\n' && !could_be_integer(c))
        putchar(c);
    putchar('\n');
    if (could_be_integer(c))
        ungetc(c, stdin);
    else if (c == EOF)
        break;
}

如您所见，这变得一团糟。有时（如comment 中的Weather Vane 所述），使用fgets() 读取一行然后在循环中使用sscanf() 会更容易（请参阅How to use sscanf() in a loop?）。警惕关于Using fflush(stdin)的建议；它不会在任何地方自动出错，但在正常情况下它不会在 MacBook Pro 上运行。

总体而言，简单地忽略输入行的其余部分通常是更好的界面决策。

Answer 2

它对我有用。

我因此附上了你的 sn-p：

#include <stdio.h>
#include <errno.h>

#define MAX_GRADES_LENGTH 20
#define HISTOGRAM_SIZE 20

main()
{

    int grades[MAX_GRADES_LENGTH]={0}, histogram[HISTOGRAM_SIZE]={0};
    int maxGradesHistogramBucket=0, median=0, gradesLength=0;
    double avg=0.0;
    int grade=0;
    int i;

    printf("Enter grades:\n");
    while (scanf("%d",&grade) != EOF)
    {
        grades[gradesLength]=grade;
        gradesLength=gradesLength+1;
    }

    if (errno)
        perror("grade");

    for (i = 0; i < gradesLength; ++i) {
        printf("%d\n", grades[i]);
    }
}

然后运行它：

$ a.out
Enter grades:
100 0 90 10 80 20 70 30 60 40 50
100
0
90
10
80
20
70
30
60
40
50
$ a.out
Enter grades:
0 50 100
0
50
100
$

也许你找错地方了。也许错误在您的输出例程中？

就个人而言，如果不得不这样做，考虑到scanf 何时返回的内容有些模糊，并且不重写它，那么这个小的更改可能更可靠：

int i, r;

printf("Enter grades:\n");
while ((r = scanf("%d",&grade)) > 0)