4

当我试图杀死我的 Robber 线程时，有些死了，但有些卡在 wait() 块中，杀死所有线程的更好方法是什么，或者我如何让被阻塞的线程被杀死？

private int robberId;
private static int robberGlobalId=0;
private TreasureChest chest;
private boolean alive = true;

public Robber(TreasureChest chest) {
    robberId = robberGlobalId;
    robberGlobalId++;

    this.chest = chest;
}

public void run() {
    while (alive) {
        try {
            synchronized(chest){
                robCoin();
            }
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }
    System.out.println("Robber " +robberId +" just died");
}

public void robCoin() throws InterruptedException {
    if (chest.getTreasureAmount() <= 0 ) {
        chest.wait();
    } else { 
        chest.removeCoin();
    }
    Thread.sleep(50);
}

public void killRobber() {
    alive = false;
}
4

2 回答 2

7

当我试图杀死我的 Robber 线程时，有些死了，但有些卡在 wait() 块中，杀死所有线程的更好方法是什么，

“杀死”线程的正确方法是用thread.interrupt(). 如果线程在wait(...)调用中被阻塞，这将立即抛出InterruptedException. 捕获 InterruptedException时最好立即重新中断线程以保留中断标志，因为当抛出异常时，中断位被清除。

try {
    ...wait();
} catch (InterruptedException ie) {
    Thread.currentThread().interrupt();
    // handle the interrupt
    return;
}

由于并非所有方法都 throw InterruptedException，您还可以检查以确保线程已被以下内容中断：

if (Thread.currentThread().isInterrupted()) {
    // stop processing
    return;
}

或者在你的情况下是这样的：

while (alive && !Thread.currentThread().isInterrupted()) {

顺便说一句，alive应该是volatile因为它看起来可以被多个线程访问。

于 2013-10-17T17:34:05.463 回答
1

正如@Gray 的回答所证明的那样，中断线程是一种方法，但是当你“杀死”强盗而不是中断它们时，唤醒等待的线程可能更干净。

在下面的这个例子中，“Robber 任务”（由run()方法实现）将等待，只要强盗还活着并且箱子是空的（小于或等于 0）。如果killRobber()被称为等待线程被唤醒并run()优雅退出（活着会false）。

public void run() {
    try{
        synchronized(chest){
            while (chest.getTreasureAmount() <= 0 && alive) {
                chest.wait();
            } 
            if(alive){
                chest.removeCoin();
            }
        }
    }catch (InterruptedException ie){
        /* Thread interrupted do something appropriate,
           which may be to do nothing */
    }
}

public void killRobber() {
    synchronized(chest){
        alive = false;
        chest.notifyAll();
    }
}
于 2013-10-17T18:03:31.370 回答