为什么这个循环是不良实践？

我本来想留个评论的，但是我还没有足够的声望。我看到的解释中，没有人解释为什么你的s值没有改变。
在一个典型的for循环中：

for(a=1; a<=10; a+=1) {body}

初始语句'a=1'仅在初始化时执行一次。

第三个语句'a+=1'在每个循环结束时执行一次，直到...

第二个语句'a>=10'的结果为假。

因此，一个for循环可以用“伪代码”表示如下：

    a=1         // first phrase
:LoopLabel
    {body}
    a+=1        // third phrase
    if (a<=10)  // second phrase (boolean evaluation)
        then goto LoopLabel

同样地，你的例子，在类似的假代码中可能看起来像这样：

    Scanner in = new Scanner(System.in);
    int i = 0;
    String s = in.next()
:LoopLabel
    {
        System.out.println("The value of i is: " + i + " and you entered " + s);
    }
    ++i
    if (!s.equals("end"))
        goto LoopLabel

所以你的程序成为无限循环的原因是's'的值只在进入循环时被设置，而在每个循环执行期间从未更改，这通常是不希望的。

这是一种不好的做法，因为它仅在下一个获得的标记是“end”时终止。它不考虑输入流结束等情况。
所以当流结束并且没有出现“end”时，您将得到`s=null`和`NullPointerException`在`s.equals("end")`处。
您可以通过将条件更改为`in.hasNext() && !"end".equals(s)`来进行更正。
此外，在初始化后，s永远不会改变。

如果问题是“为什么要重写它”，答案基本上就像其他人指出的那样，它目前是一个无限循环，并且作为它所站立的阅读性不太好。个人认为我会将其重写为while循环，正如其他几个人已经指出如何做到这一点，因为它使你的意图比计数器正在计算到无穷大的for循环更加清晰。对于不熟悉代码应该如何工作的人来说，无限递增很容易被认为是编写它的程序员的疏忽。

这种方式并不好，因为有以下两个原因：

1. for循环的本意是遍历数据集合。
2. for循环由迭代器的初始状态、循环条件和迭代函数组成，它们是相关联的。

for语句将两个不同的信息（流和计数器）混合在一起。即使它可以工作，也不是一个好的做法。

for (int i = 0; in.hasNext(); i++) {
    String s = in.next();
    if (s.equals("end")) {
        break;
    }
    ...

无限循环，或者没有循环（当s最初为"end"时）。

上面的一些回答是正确的，说你写的是一个无限循环。但我想澄清一下为什么这是一个无限循环。你使用的for循环与你可能想到的另一种形式不同：

String[] stringArray = { "1", "2", "3" };
for (String s : stringArray) {
   System.out.println(s);
}

在这种情况下，变量s在每次迭代中都会用集合或数组的下一个值进行初始化。但是那种形式的for循环只适用于集合和数组，无法与像Scanner类这样的迭代器一起使用。
你正在使用的for循环形式不同之处在于初始化子句（你有String s = in.next()的地方）仅在第一次通过循环时被调用。 s在第一次设置后就不再改变。
你可以像这样重新编写：

int i = 0;
for (String s = in.next(); !s.equals("end"); s = in.next()) {
    System.out.println("The value of i is: " + i++ + " and you entered " + s);
}

但这里的另一个坏处是没有空值或结束检查。如果你在找到等于“end”的字符串之前用尽了所有字符串，那么 for 测试子句（中间的那个）在尝试调用 equals() 方法时会给你一个 NullPointerException。这绝对是不好的实践。我可能会像这样重新编写它：

int i = 0;
while (in.hasNext()) {
    String s = in.next();
    if (s.equals("end")) {
        break;
    }
    System.out.println("The value of i is: " + i++ + " and you entered " + s);
}

如果你真的想要一个 for 循环而不是 while，最好这样做：

int i = 0;
for (Scanner in = new Scanner(System.in); in.hasNext();) {
    String s = in.next();
    if (s.equals("end")) {
        break;
    }
    System.out.println("The value of i is: " + i++ + " and you entered " + s);
}

最后一种保留针对测试子句中字符串测试的变体如下：

int i = 0;
String s = "";
for (Scanner in = new Scanner(System.in);
     in.hasNext() && !s.equals("end"); 
     s = in.next()) {
    System.out.println("The value of i is: " + i++ + " and you entered " + s);
}

在s.equals("end")之前，您还可以添加一个空值检查以确保完全安全。

这并不是一个好的做法，可能是因为你正在比较String s与一个字符串，但你并没有比较值，而是比较了s值的内存位置。