多线程/易失变量/条件/循环的特殊行为（Java）

Question

这是我目前正在构建的网络服务器的 sn-p...

// ...

threadPool = Executors.newCachedThreadPool();
while (true)
  if(this.isOn) {
                try { // listen for incoming connection
              this.clientSocket = serverSocket.accept();
      } catch (IOException e) {
                   System.err.println("LOG: >> Accept failed! ");
                   System.exit(1);
                  }
      // as soon as a connection is established send the socket
      // with a handler/processor to the thread pool for execution

                    threadPool.execute(new ClientRequestProcessor(clientSocket));
        }

//...

请注意，isOn 变量是一个 VOLATILE 布尔值。

如果我把 if 变成一段时间......这段代码可以工作......但实际上它没有。我可以问为什么吗？从逻辑的角度来看，即使我在 if 中测试该标志，两者都应该工作......我错过了什么吗？！

[稍后编辑：] 不工作我的意思是......浏览器（例如 firefox）无法连接，实际上它一直在尝试但最终超时。同样，如果我将 if(isOn) 更改为 while(isOn) 它就像一个魅力。

欢迎提出任何建议/想法！！！

附：我需要这个组合“while(true) if/while(test flag) {...}”，因为服务器可以从 GUI 启动/停止...所以顶层 while(true) 是需要的，所以我可以重新检查我是打开（并因此监听连接）还是关闭（并且并不真正关心传入连接）。不用说GUI的事件处理程序可以随时修改标志。

Answer 1

更好的解决方案是在您希望它停止时关闭服务器套接字，并在您希望它启动时在新线程中启动一个新套接字。这样你就拒绝新连接，并且在它不做任何事情时不会消耗 CPU。

当 isOn == true 并且您将其设置为 false 时，它​​只会在下一个新连接之后不接受连接。 （可能在以后的任何时间）此外，任何客户端新连接都将等待调用接受（或最终超时）默认情况下，您最多可以有 50 个连接等待被接受。

当 isOn == false 时，您的线程将忙于等待，消耗 CPU。我建议你放一点延迟，比如 Thread.sleep(250)。这将显着减少 CPU，但不会过多延迟重新启动。

顺便说一句：

• 如果您遇到异常，您应该将其记录/打印出来。否则当它失败时，你将不知道为什么。
• 如果接受失败，可能是进程文件不足，因此您不希望它直接死掉，杀死所有现有连接。

Answer 2

我的假设将基于您那里的紧密循环。您是否有可能在您的服务器上运行该程序的多个实例？如果您将 isOn 设置为 false，if(isOn) 版本将不会关闭，而是会简单地循环永远烧毁您的 CPU。

Answer 3

如果您有 while(true) 和 if(this.isOn)，则 while 循环无法停止。当 isOn 标志变为 false 时会发生什么。 while 循环永远不会停止，因为它本质上是无限的。插入 else 语句使其中断，它应该可以按预期工作。

如果您取出 if 语句并使其成为 while(this.isOn) 则当 isOn 标志变为 false 时循环结束。没有无限循环。

这些是我第一眼的想法......