Java 中的 while 循环不起作用

Question

好吧，问题是这样的：

我想输入一堆数字，一组随机的数字，比如 5 个数字。在该数字的末尾，我将添加第六个数字 0，它将作为循环结束的信号。

为了实现这个目标，这就是我要做的。

我运行我的程序，这是我的输入：

1 3 4 2 5 0

所以我希望程序打印

1 3 4 2 5“序列结束”

这是我的代码，据说会做这样的事情（所有这些都在 main 方法中）：

Scanner input = new Scanner(System.in);
System.out.println("Alright, what?");
boolean notZero = true;
while (notZero)  
 {
   System.out.println(input.nextInt());
   if (input.nextInt() == 0)
       System.out.println("End of sequence");
       notZero = false;    
 }

但是，这些是我得到的结果：

https://i.sstatic.net/DUFhs.jpg

有趣的是，如果我没有 while 循环停止，我再次输入相同的数字列表 (1 3 4 2 5 0）它会输出 3 和 2，这是我的数字列表中的偶数 - 所有这些都让我更加烦恼。

有什么想法吗？

Answer 1

将所有块（包括 if 块）包裹在大括号中！

这：

if (foo) 
  doSomething();
  doSomethingElse();

并不像你想象的那样工作，因为它的行为是相反，

if (foo) {
  doSomething();
}

doSomethingElse();

将 all 块包裹在大括号中，这样就毫无疑问：

if (foo) {    
  doSomething();
  doSomethingElse();
}

Answer 2

这看起来像是一个学习练习，所以我会给你一些提示，而不是直接告诉你。

1. 在您选择的 IDE 中，告诉它自动缩进您的代码。仔细查看 if 语句及其结果所在的位置。

2. 仔细考虑 nextInt() 的作用以及调用它的位置和频率。

Answer 3

您在循环中调用 input.nextInt() 两次，仅打印其中之一。实际上，这就像跳过输入中的所有其他数字。更改代码以将下一个输入存储到变量中，然后检查该变量。我怀疑这不是您的实际代码，因为检查条件时没有将 notZero 设置为 false，它只会执行一次循环。 FWIW，notZero 是一个糟糕的变量名。它应该是语义的，而不是表明其期望值。如果输入值不是输入指示符结束，您还需要仅打印输入值。

Scanner input = new Scanner(System.in);

System.out.println("Alright, what?");

boolean endOfInput = false;

while (!endOfInput) {
   int next = input.nextInt();

   if (next == 0) {
      System.out.println("End of sequence");
      endOfInput = true;
   }  
   else {
      System.out.println(next);
   } 
}

Answer 4

你的问题在这里：

 if (input.nextInt() == 0)

当你调用 nextInt() 两次时，它会删除两个数字。每次迭代只需调用一次。

 while (notZero)  
 {
    int num = input.nextInt();
    System.out.println(num);

    if (num == 0) {
        System.out.println("End of sequence");
        notZero = false;    
    }

  }

解决这个问题以及充满鳗鱼的气垫船指出的问题，你应该会很好。

Answer 5

每次循环中都会调用 nextInt() 两次。这意味着每次，您都会使用两个数字，对每个数字执行不同的操作 - 即，您打印的数字和与 0 比较的数字是两个不同的数字。您只需调用 nextInt() 一次并将结果存储在变量中，然后对其进行操作。

Answer 6

除了循环内的两次读取之外，我假设您希望在 if 语句之后有一个块

if (input.nextInt() == 0) {
    System.out.println("End of sequence");
    notZero = false;    
}

Answer 7

据我所知，notZero 的唯一目的是终止循环。如果您使用在循环条件下终止的循环，则不需要它。您也不需要在循环内打印“序列结束”，因为根据定义，它会在序列结束后打印。

如果我们直接测试条件，

int n;
while ((n = input.nextInt()) != 0)
{
    System.out.println(n);
}
System.out.println("End of sequence");

您也可以说 for (int n = input.nextInt(); n != 0; n = input.nextInt()) 而不是 而。它确保 n 仅在循环内可见，但这也意味着重复自己 - 并留下了以后有人可能使两个“初始化”和“增量”部分不同的可能性。