這個中斷（）是否必要？

Question

這裏的片段：

public class LogService { 

    public void stop() { 
     synchronized (this) { isShutdown = true; } 
     loggerThread.interrupt(); /* Is it necesarry? */ 
    } 

    public void log(String msg) throws InterruptedException { 
     synchronized (this) { 
      if (isShutdown) 
      throw new IllegalStateException(...); 
      ++reservations; 
     } 
     queue.put(msg); 
    } 

    private class LoggerThread extends Thread { 
     public void run() { 
      try { 
       while (true) { 
        try { 
         synchronized (LogService.this) { 
          if (isShutdown && reservations == 0) 
           break; 
         } 
         String msg = queue.take(); 
         synchronized (LogService.this) { 
         --reservations; 
         } 
         writer.println(msg); 
        } catch (InterruptedException e) { } /* Do nothing */ 
       } 
      } finally { 
       writer.close(); 
      } 
     } 
    } 
} 

正如上面的代碼，即使我們把LoggerThread.interrupt（）在stop（）方法，中斷只是被抓線程什麼都不做。

那麼LoggerThread.interrupt（）是必要的嗎？

Answer 1

是的，它是必要的。如果隊列爲空，則該語句String msg = queue.take();將阻塞，直到元素放入隊列或被中斷。

如果你想保證線程沒有掛起，你需要中斷它。

不過似乎有一個小故障：如果reservations不爲0，當你調用close方法和隊列爲空，似乎你的循環將繼續下去，並在while循環迭代中斷下面就queue.take()掛起。