C无限做while循环

Question

这是一个简单的课程程序，可提示用户以分钟为单位的淋浴时间（正整数，根据需要重新提示），然后打印等量的水（整数）。

假定淋浴器每分钟使用1.5加仑水（192盎司），塑料瓶尺寸为16盎司

我的do-while循环成功拒绝了负数和0，但是，如果在几分钟后提示输入淋浴时间，如果我输入“ foo”之类的文本，程序将进入无限循环，永远运行循环并打印“ How你的淋浴多久（以分钟为单位）？：”

任何想法如何完善while条件以避免这种情况？

#include <stdio.h>

int min_to_bottles(int *n);

int main(void)
{
    int minutes;
    int bottles;
    do
    {
        printf("How long is your shower(in minutes)?:");
        scanf("%i", &minutes);
    }
    while (minutes < 1);

    bottles = min_to_bottles(&minutes);

    printf("Equivalent of bottles used per shower is: %i\n", bottles);

}

int min_to_bottles(int *n)
{
    int bottles;
    int oz = 192 * *n;
    bottles = oz/16;
    return bottles;
}

Answer 1

始终检查scanf()的返回值：

int result;
do {
    printf("How long is your shower(in minutes)?:");
    result = scanf("%d", &minutes);
    if(result != 1) 
        break;
} while (minutes < 1);

较短的版本（如果仅需要一次扫描）：

printf("How long is your shower(in minutes)?:");

while ((scanf("%d", &minutes) == 1) && (minutes < 1))
    ;

---

int min_to_bottles(int *n);不需要使用指针作为参数int min_to_bottles(int *n); ：

#include <stdio.h>

int min_to_bottles(int n)
{
    return (192 * n) / 16;
}

int main(void)
{
    int minutes = 0;
    int bottles = 0;

    printf("How long is your shower(in minutes)?: ");

    while ((scanf("%d", &minutes) == 1) && (minutes < 1 || minutes > 100))
        printf("Enter a number between 1 and 100 : ");

    // if(minutes == 0) here means invalid data was entered.
    //    so a check could be done before continuing.

    bottles = min_to_bottles(minutes);

    printf("Equivalent of bottles used per shower is: %d\n", bottles);

    return 0;
}

将minutes初始化为0将避免在scanf()失败的情况下scanf()例如通过输入文本scanf()使用不确定的值来计算bottles 。

Answer 2

当您输入文本时，它与%i格式说明符不匹配，因此文本会卡在输入缓冲区中，并且会继续尝试读取同一内容。

如果没有很好的匹配，则需要刷新缓冲区。 通过检查scanf的返回值（返回成功匹配的模式数量），您将知道情况是否如此。

int minutes = 0;
while (minutes < 1)
{
    printf("How long is your shower(in minutes)?:");
    int count = scanf("%i", &minutes);
    if (count < 1) {
        scanf("%*s");   // The * tells scanf to read the input but not assign it to anything
    }
}

Answer 3

不要使用scanf("%i",...) 。

主要问题是直到另一个函数读取它之前，没有转换为数字的错误输入仍保留在stdin 。 由于代码没有检查scanf()的返回值，因此minutes的值不佳，并且do ... while (minutes < 1); 可以轻松地重复循环。

解决方案：读取一行输入，转换为数字，有效数字：

---

要处理意外的用户输入，请先读取一行输入

char buf[80];
if (fgets(buf, sizeof buf, stdin) == NULL) Handle_EOF();

然后解析该缓冲区的数字。

errno = 0;
char *endptr;
long num == strtol(buf, &endptr, 0);

// detect overflow and no conversiosn
if (errno || buf == endptr) Handle_BadInput();

// Ignore trailing white-space
while (isspace((unsigned char) *endptr) endptr++;

if (*endptr) Handle_BadInput();

验证号码。

#define SHOWER_TIME_MIN (1 /* minute */)
#define SHOWER_TIME_MAX (60 /* minute */)
if (num < SHOWER_TIME_MIN || num > SHOWER_TIME_MAX) Handle_BadInput();

全部放到一个辅助函数中

示例如何测试输入是否正常