Question

我正在尝试为练习问题编写一个简单的程序，如下所示：

（销售佣金计算器）一家大型化工公司向其销售人员支付佣金，任务依据。销售人员每周可获得200美元，另加该周销售总额的9％。例如，某销售人员在一周内售出价值$ 5000的化学品，将获得$ 200的收益以及9％的 5000美元，或总计650美元。制定程序，以输入每个销售人员的最终销售总额周，并将计算并显示该销售人员的收入。在一个处处理一个销售人员的数据时间。

我正在尝试处理用户是否输入数字以外的其他内容，例如字符串/字符但是，当我尝试运行它并通过输入字符串/字符对其进行测试时，它会一直循环播放直到VSC崩溃。

我的代码：

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

int main(void){
    float sales = 0;
    float salary = 0;

    while (sales != -1)
    {
        printf("Enter sales in dollars (-1 to end): ");
        scanf("%f", &sales);

        while ((sales != -1) && !(sales >= 0))
        {
            puts("Input error, please try again");
            fflush(stdin);
            printf("Enter sales in dollars (-1 to end): ");
            scanf("%f", &sales);
        }

        if (sales != -1)
        {
            salary = 200 + (0.09 * sales);

            printf("Salary is: $%.2f\n", salary);
        }
    }

    return 0;
}

我在另一个程序中使用了类似的方法，并且效果很好，所以我不知道这里出了什么问题。

另一个程序的代码：

while(!(collected >= 0))
{
    fflush(stdin);
    puts("Input error, please try again.");
    printf("Enter total amount collected (-1 to quit): $");
    scanf("%f", &collected);
}

不胜感激！

Answer 1

scanf("%f", ...)永远不会接受任何格式不正确的数字，因此，如果您输入一个字符，那么所有scanf()调用都只会看到有一个字符需要使用，但是格式说明符不允许这样做，他们将不会读取任何内容并返回0。如果输入的不是有效数字，则加上您的while将导致无限循环。

您必须阅读所有内容，直到行尾，或者在scanf返回0时跳过多余的字符（这意味着它无法正确读取任何值）。另外，检查scanf是否返回EOF，这意味着出现错误或到达文件末尾。

无论如何，fflush(stdin) is undefined behavior，永远不要这样做！

正确的方法是：

#include <stdio.h>

int main(void) {
    float sales = 0;
    float salary;
    int res;

    do {
        printf("Enter sales in dollars (-1 to end): ");
        fflush(stdout);

        res = scanf("%f", &sales);

        if (res == EOF) {
            // Something bad happened.

            if (feof(stdin))
                fputs("End of input reached.\n", stderr);
            else
                perror("Error reading input");

            return 1;
        }

        if (sales != -1 && sales < 0) {
            // Perhaps it would be a better idea to say "invalid value" here?
            puts("Input error, please try again.");
            res = 0;
        }
        
        // Either the value was read correctly (res = 1) or it wasn't (res = 0).
        // In any case, consume the rest of the line.
        scanf("%*[^\n]");

    } while (res != 1 && sales != -1.0);

    if (sales != -1.0) {
        salary = 200 + (0.09 * sales);
        printf("Salary is: $%.2f\n", salary);
    }

    return 0;
}

如何防止无限的stdin循环？

1 个答案: