需要帮助理解循环

Question

好吧，我想做的事情似乎相当简单，但它并没有按照我想要的方式工作。 我知道我只是没有得到什么。 本质上我试图读取控制台输入，将其分配给变量。 然后我想检查该变量以查看它是否是有效数字。 如果不是，我想告诉用户它无效并重新开始循环，直到我得到一个有效的数字，然后退出。 这是我的代码，请你帮我理解我做错了什么？

    const int AVERAGE_IQ = 100;
    int userIQ;
    bool done = false;

    do
    {
        Console.Write("Please enter an IQ Score between 1 and 200: ");
        userIQ = Convert.ToInt32(Console.ReadLine());

        if (userIQ == 0 || userIQ >= 200)
        {
            Console.WriteLine("You have entered an invalid IQ Score, please try again.");
            done = false;
        }
        else if (userIQ >= AVERAGE_IQ)
        {
            Console.WriteLine("{0} is an above average IQ.", userIQ);
            done = true;
            break;
        }
        else if (userIQ <= AVERAGE_IQ)
        {
            Console.WriteLine("{0} is an below average IQ.", userIQ);
            done = true;
            break;
        }
        else if (userIQ == AVERAGE_IQ)
        {
            Console.WriteLine("{0} is anaverage IQ.", userIQ);
            done = true;
            break;
        }

    } while (done =! true);

Answer 1

=! 应该!=

你正在进行任务，设置为false。 以来：

while (done =! true); 与while (done = !true); 这与while (done = false);

在每次迭代中，您将done指定为false，其自身的计算结果为false，这意味着您的循环将永远不会再次迭代。

将循环表达式更改为while (done != true); （或者更好while (!done);

Answer 2

错误在最后一行：

while (done =! true);

你正在做的是分配done !true ，这是false所以不管在执行这个语句之前done的值，它将变为false并且循环将退出。

你真正想要的是：

while (!done);

此外，当您使用break来突破循环时，您可以删除使用done变量的每一行，只需使用此行而不会出现任何问题：

while (true);

Answer 3

done =! true done =! true应该在done != true更改为done != true

Answer 4

正如xanatos所说。 但是，你的休息与完成标志有一个风格问题。 break的替代方法是保持done标志 - 并删除break语句。 你的时间会变成一会儿（！完成）;

Answer 5

1. 检查IQ < 0

2. 将>=更改为> ， <=更改为<

3. 删除所有已done并将while更改为while (true)

Answer 6

几点说明：

1. 将else if (userIQ >= AVERAGE_IQ)更改为else if (userIQ > AVERAGE_IQ)因为你有最后一个“如果它们相等”
2. 因为你使用break; 要退出循环，您不需要设置done=true

Answer 7

“你不按我想要的方式工作”是什么意思？ 是不是正确地提示用户输入？ 是否显示错误信息？ 这是一个例外吗？

乍一看，我已经看到了一个错误。 检查> = AVERAGE_IQ然后<= AVERAGE_IQ然后== AVERAGE_IQ。 如果用户输入AVERAGE_IQ，则第一个条件会命中，并且它会说它高于平均IQ。 > =和<=应该是>和<而不是。

此外，完成标志是多余的，因为您正在破坏。 如果你打算使用休息时间，你也可以使用while（true）循环。

另一件事，你没有正确处理负输入。 它还应显示超出范围的错误消息。

编辑：正如许多其他人所指出的那样，“=！” 和“！=”不是一回事。

Answer 8

循环用于处理让用户输入正确的输入。 评估输入超出适当范围应该移动到其他地方。

如果您将代码重构为类似的内容，则更清楚您要执行的操作。

int minIQ = 1;
int maxIQ = 200;
do
{
    Console.Write("Please enter an IQ Score between " + minIQ + " and " + maxIQ + ": ");
    userIQ = Convert.ToInt32(Console.ReadLine());
} while (userIQ >= minIQ && userIQ <= maxIQ);

//Test userIQ and output here.