Question

我只是在想什么。我有以下代码：

#include <iostream>
using namespace std;

int main()
{
    int number, largest, counter = 1;
    while (counter <= 10)
    {
          cout << "Enter a number: ";
          cin >> number;
          if (counter = 1)
          {
               largest = number;
          }
          else if (number > largest)
          {
               largest = number;
              }
          counter++;
    }
    cout << "\n\nThe largest number is: " << largest;
    system("pause");
    return 0;
}

问题是，它永远不会终止。我确实设法通过稍微修改代码来解决问题，但我想知道为什么会发生这种情况。这是固定代码：

#include <iostream>
using namespace std;

int main()
{
    int number, largest, counter = 1;
    cout << "Enter a number: ";
    cin >> number;
    largest = number;
    while (counter < 10)
    {
      cout << "Enter a number: ";
      cin >> number;
      if (number > largest)
      {
           largest = number;
      }
      counter++;
    }
    cout << "\n\nThe largest number is: " << largest << endl;
    system("pause");
    return 0;
}

似乎在删除else if语句后它起作用了。发生了什么事？

Answer 1

if (counter = 1)

这应该是

if (counter == 1)

否则，您将在每次迭代时将计数器重置为1。

Answer 2

一个常见的错误：

if( counter = 1) // assignment operator

这将在每次迭代时将counter的值设置为1，并且循环永远不会完成。

你应该使用

if( counter == 1) // equality operator
           ^^^^

这正是你的意思

Answer 3

if (counter = 1)重新分配1来计算每个循环，这始终是< 10。

您想要if (counter == 1)。

Answer 4

  if (counter = 1)

这不会与counter和1进行比较，它会将1分配给counter，然后检查counter - 我们设置为1所以它总是积极的，永远是<= 10。

Answer 5

该行

if (counter = 1)

应该是

if (counter == 1)

因为你想比较，而不是分配价值。

Answer 6

你的第一个例子是

if (counter = 1)

而不是

if (counter == 1)

因此if语句会在每次迭代期间将计数器重置为1。

Answer 7

另见通知，这是一个常见的错误。您可以通过键入

来避免它

if( 1 == counter )

而不是

if( counter == 1 )

因为

if( 1 = counter )

无法编译（如果您输入错误并忘记了'='）。

Answer 8

你的问题在这里：

if (counter = 1)

分配而不是比较。编译具有更高的警告级别。

Answer 9

if (counter = 1)

因此，counter值永远为1。

Answer 10

您将1分配给计数器而不是比较它，使用==而不是=

非终止while循环

10 个答案: