我使用CountDownLatch来同步两个线程之间的初始化过程,我想知道它可能会正确处理InterruptedException。
我最初写的代码是:
private CountDownLatch initWaitHandle = new CountDownLatch(1);
/**
* This method will block until the thread has fully initialized, this should only be called from different threads Ensure that the thread has started before this is called.
*/
public void ensureInitialized()
{
assert this.isAlive() : "The thread should be started before calling this method.";
assert Thread.currentThread() != this, "This should be called from a different thread (potential deadlock)";
while(true)
{
try
{
//we wait until the updater thread initializes the cache
//that way we know
initWaitHandle.await();
break;//if we get here the latch is zero and we are done
}
catch (InterruptedException e)
{
LOG.warn("Thread interrupted", e);
}
}
}
这种模式有意义吗?基本上忽略InterruptedException只是一直等到成功为止是个好主意。我想我只是不明白这会被打断的情况所以我不知道我是否应该以不同方式处理它们。
为什么会在这里抛出InterruptedException,处理它的最佳做法是什么?
答案 0 :(得分:9)
这正是您不应该为InterruptedException
做的事情。 InterruptedException
基本上是该线程终止的礼貌请求。线程应该尽快清理并退出。
IBM发布了一篇很好的文章:http://www.ibm.com/developerworks/java/library/j-jtp05236.html
这就是我要做的事情:
// Run while not interrupted.
while(!(Thread.interrupted())
{
try
{
// Do whatever here.
}
catch(InterruptedException e)
{
// This will cause the current thread's interrupt flag to be set.
Thread.currentThread().interrupt();
}
}
// Perform cleanup and exit thread.
这样做的好处是:如果在阻塞方法中线程被中断,则不会设置中断位,而是抛出InterruptedException
。如果您的线程在没有阻塞方法的情况下被中断,则将设置被中断的位,并且不会抛出任何异常。因此,通过调用interrupt()
在异常上设置标志,两种情况都被标准化为第一种情况,然后由循环条件检查。
作为一个额外的好处,这也可以让你通过简单地打断它来停止你的线程,而不是发明你自己的机制或接口来设置一些布尔标志来完成同样的事情。
答案 1 :(得分:2)
如果你没有预见到Thread
可能被打断的任何正当理由,并且想不出任何合理的反应,我会说你应该这样做
catch (InterruptedException e){
throw new AssertionError("Unexpected Interruption",e);
}
这样,如果发生这种中断,应用程序将明显失败,从而使测试期间更容易发现。然后你可以考虑应用程序应该如何处理它,或者它们是否在设计中有任何问题。