为什么这个break语句不起作用？-Whydoesthisbreakstatementbreaknotwork?

3  

You are starting a new thread every time you call the post method. I thing the method is OK but caller program isn't.

每次调用post方法时都会启动一个新线程。我的方法是可以的但是来电程序不是。

4  

Could it be that this line succeeds:

可能是这条线成功了：

output.println(mess);

but this line is throwing a null pointer exception:

但是这一行抛出了一个空指针异常：

Game.log.fine(...

In this case you'll see the output on the console, but the break statement is never reached. Is Game.log perhaps null?

在这种情况下，您将在控制台上看到输出，但永远不会达到break语句。 Game.log可能是null吗？

4  

What exactly does Game.log.fine do? Could it be that it throws a NullPtrException after output, or could it be that you call the post-method several times?

Game.log.fine究竟做了什么？可能是它在输出后抛出NullPtrException，还是可能是你多次调用post方法？

Remove the catch of the NullPointerException, this is bad style (occurrence of a NullPointerException is always a programming error) and add some more log-messages in the method (or use a debugger).

删除NullPointerException的catch，这是错误的样式（出现NullPointerException总是一个编程错误）并在方法中添加一些更多的日志消息（或使用调试器）。

2  

Are you sure you want to keep going on a NullPointerException? If you get one inside the loop, you're likely to wait forever.

你确定要继续使用NullPointerException吗？如果你在循环中得到一个，你可能会永远等待。

If you're sure that status will always eventually be "serviceResolved", then put a try...finally inside the if statement so that if something fails, the loop still exits:

如果你确定状态最终总是“serviceResolved”，那么在if语句中最后放一个try ...，这样如果出现故障，循环仍会退出：

if (status.equals("serviceResolved")) {
    // No matter what happens next, we have to bail
    try {
        output.println(mess);
        Game.log.fine("The following message was successfully sent: " + mess);
    } finally {
        break;
    }
} else {
    try {Thread.sleep(1000);} catch (InterruptedException ie) {}
}

0  

You're assuming that the break statement isn't working, but it could be that your post method is being called repeatedly. Try putting another log statement at the beginning of the method to see how often it is being called. Also, put a log statement after the while loop but before the end of the run method to verify that the break actually broke out of the loop.

您假设break语句不起作用，但可能是您的post方法被重复调用。尝试在方法的开头放置另一个日志语句，以查看它被调用的频率。此外，在while循环之后但在run方法结束之前放置一个日志语句，以验证中断实际上是否已从循环中断开。

I also agree with the other posters that catching NullPointerException is a code smell. You should check your variables for null first.

我也同意其他海报，捕捉NullPointerException是一种代码气味。您应首先检查变量是否为null。