清除线程的方法。中断（）标志

Question

我最近继承了一个大型Java应用程序，其中几乎没有线程安全性。我目前正在做的是让所有线程正确处理被中断的情况，而不是使用非常糟糕的线程。停止（）。

部分问题是，我不知道清除中断标志的每个方法调用。

目前我知道以下操作将清除中断标志：

Thread.interrupted()
Thread.sleep(long)
Thread.join()
Thread.join(long)
Object.wait()
Object.wait(long)

我还缺什么？非常感谢。

Answer 1

这里有一个非常有趣的例子：

信道qos。返回日志。果心版本1.1.4之前的AsyncAppenderBase捕获并接受InterruptedException，而不重置线程上的标志。

所以，如果你使用任何路由到这个记录器的东西（比如slf4j），它会默默地吃掉你的线程中断状态。因为，我的意思是，谁不在每个可能的日志操作前后检查线程中断状态？

Answer 2

常见的约定如下：任何抛出中断异常（Thread.interrupted（））的方法都会清除中断标志。

因此，为了使线程具有可中断性，您需要找到所有捕获到中断异常的位置，而无需追溯它或恢复中断标志。因为InterruptedException是一个选中的异常，所以不难做到。

Answer 3

部分问题是，我不知道清除中断标志的每个方法调用。

必须澄清的是，以下方法只需调用它们即可清除中断标志：

Thread.interrupted()
Thread.isInterrupted(true) -- added to your list

因此，线程。currentThread（）。应始终使用isInterrupted（）。

以下方法将通过立即抛出InterruptedException来清除中断的标志，如果它们被调用然后线程被中断，或者如果线程已经被中断然后它们被调用（请参阅下面的jUnit代码）。所以清除标志的不是方法，而是抛出异常。

您的初始列表：

Thread.interrupted()
Thread.sleep(long)
Thread.join()
Thread.join(long)
Object.wait()
Object.wait(long)

添加到您的列表：

Thread.sleep(long, int)
Thread.join(int, long)
Thread.isInterrupted(true)
Object.wait(int, long)
BlockingQueue.put(...)
BlockingQueue.offer(...)
BlockingQueue.take(...)
BlockingQueue.poll(...)
Future.get(...)
Process.waitFor()
ExecutorService.invokeAll(...)
ExecutorService.invokeAny(...)
ExecutorService.awaitTermination(...)
CompletionService.poll(...)
CompletionService.take(...)
CountDownLatch.await(...)
CyclicBarrier.await(...)
Semaphore.acquire(...)
Semaphore.tryAcquire(...)
Lock.lockInteruptibly()
Lock.tryLock(...)

请注意，捕获中断异常（InterruptedException）的任何代码的正确模式是立即重新中断线程。我们这样做是为了防止其他人依赖于线程。isInterrupted（）方法：

try {
    ...
} catch (InterruptedException e) {
    // immediately re-interrupt the thread
    Thread.currentThread().interrupt();
    // log the exception or [likely] quit the thread
}

演示其中一些的JUnit代码：

assertFalse(Thread.currentThread().isInterrupted());
// you can do this from another thread by saying: someThread.interrupt();
Thread.currentThread().interrupt();
// this method does _not_ clear the interrupt flag
assertTrue(Thread.currentThread().isInterrupted());
// but this one _does_ and should probably not be used
assertTrue(Thread.interrupted());
assertFalse(Thread.currentThread().isInterrupted());
Thread.currentThread().interrupt();
assertTrue(Thread.currentThread().isInterrupted());
try {
    // this throws immediately because the thread is _already_ interrupted
    Thread.sleep(1);
    fail("will never get here");
} catch (InterruptedException e) {
    // and when the InterruptedException is throw, it clears the interrupt
    assertFalse(Thread.currentThread().isInterrupted());
    // we should re-interrupt the thread so other code can use interrupt status
    Thread.currentThread().interrupt();
}
assertTrue(Thread.currentThread().isInterrupted());