在某些情况下终止线程的正确方法是什么？

Question

一些线程在其run方法中执行一系列操作。 车站关闭时，它将通过onStationClosed通知所有乘客。 发生这种情况时，线程必须执行某些操作（例如，leaveStation），并且线程必须终止而未完成所有剩余操作。

正确的做法是：

// 1 - By checking station's state between each operationN?

public class Passenger extends Thread
{
     Station station;
     public void onStationClosed()
     {
          // Do some action before thread terminates
     }
     @Override
     public void run()
     {
         operation1();
         if(station.getState == Station.Closed) return;
         operation2();
         if(station.getState == Station.Closed) return;
         operation3();
         if(station.getState == Station.Closed) return;
         ..
         operationN();
     }
}

// 2 - Throw StationClosedException from onStationClosed and catch it in Station.

public class Passenger extends Thread
{
    Station station;
    public void onStationClosed()
    {
         // Do some action before thread terminates
         throw new StationClosedException();
    }
    @Override
    public void run()
    {
        operation1();
        operation2();
        ..
        operationN();
    }
}

Answer 1

第一个解决方案是非常好的。 但是不是很干燥 ，请考虑在一些小的操作对象中包装操作，并在执行每个操作之前检查station状态：

List<Operation> operations = Arrays.asList(new Operation1(), new Operation2(), new Operation3());

for(Operation operation: operations) {
  if(!perform(operation)) {
    return;
  }
}

其中perform()定义如下：

private boolean perform(Operation operation) {
  if(station.getState == Station.Closed)
    return false;
  operation.run();
  return true;
}

有点牵强，但是当操作数量增加时，您会很感激的。

我不太理解异常解决方案。 如果从onStationClosed()回调引发该异常，则它将被引发回事件发送者线程，而不是Passenger线程。 它不会中断您的线程。

但是，您可以使用InterruptedException来控制此流程。 此解决方案与检查站状态非常相似，但是您可以检查Thread.isInterrupted()标志。 额外的好处：I / O操作和睡眠会自动中断。 您要做的就是打电话

Thread passenger = new Passenger();
passenger.interrupt();