我通过以下方式创建了一个线程服务:

public class TCPClientService extends Service{  
...

@Override
public void onCreate() {
    ...
    Measurements = new LinkedList<String>();
    enableDataSending();    
}

@Override
public IBinder onBind(Intent intent) {
    //TODO: Replace with service binding implementation
    return null;
}

@Override
public void onLowMemory() {
    Measurements.clear();
    super.onLowMemory();
}

@Override
public void onDestroy() {
    Measurements.clear();
    super.onDestroy();
    try {
        SendDataThread.stop();
    } catch(Exception e){
        ...     
    }

}

private Runnable backgrounSendData = new Runnable() {

    public void run() {
        doSendData();
    }
};

private void enableDataSending() {
    SendDataThread = new Thread(null, backgrounSendData, "send_data");
    SendDataThread.start();
}

 private void addMeasurementToQueue() {
     if(Measurements.size() <= 100) {
         String measurement = packData();
         Measurements.add(measurement);
     }
 }

 private void doSendData() {
     while(true) {
         try {      
             if(Measurements.isEmpty()) {
                 Thread.sleep(1000);
                 continue;
             }
             //Log.d("TCP", "C: Connecting...");
             Socket socket = new Socket();
             socket.setTcpNoDelay(true);
             socket.connect(new InetSocketAddress(serverAddress, portNumber), 3000);
             //socket.connect(new InetSocketAddress(serverAddress, portNumber));
             if(!socket.isConnected()) {
                 throw new Exception("Server Unavailable!");
             }
             try {
                 //Log.d("TCP", "C: Sending: '" + message + "'");
                 PrintWriter out = new PrintWriter( new BufferedWriter( new OutputStreamWriter(socket.getOutputStream())),true);
                 String message = Measurements.remove();
                 out.println(message);
                 Thread.sleep(200);
                 Log.d("TCP", "C: Sent.");
                 Log.d("TCP", "C: Done.");
                 connectionAvailable = true;              
             } catch(Exception e) {
                 Log.e("TCP", "S: Error", e);
                 connectionAvailable = false;
             } finally {
                 socket.close();
                 announceNetworkAvailability(connectionAvailable);
             }
         } catch (Exception e) {
             Log.e("TCP", "C: Error", e);
             connectionAvailable = false;
             announceNetworkAvailability(connectionAvailable);
         }
    }
}

...
}

在我关闭应用程序后，手机运行速度真的很慢，我猜这是由于线程终止失败。

有谁知道在终止应用程序之前终止所有线程的最佳方法是什么？

最佳答案

附录:Android 框架为一次性工作、后台工作等提供了许多帮助程序，在许多情况下这可能比尝试滚动您自己的线程更可取。如下文所述，AsyncTask 是一个很好的起点。我鼓励读者在开始考虑自己的线程之前先查看框架规定。

您发布的代码示例中有几个问题我将按顺序解决:

1) Thread.stop() 已经被弃用了一段时间，因为在某些情况下它会使因变量处于不一致的状态。见 this Sun answer page有关更多详细信息(编辑:该链接现已失效，请参阅 this page for why not to use Thread.stop() )。停止和启动线程的首选方法如下(假设您的线程将无限期地运行):

private volatile Thread runner;

public synchronized void startThread(){
  if(runner == null){
    runner = new Thread(this);
    runner.start();
  }
}

public synchronized void stopThread(){
  if(runner != null){
    Thread moribund = runner;
    runner = null;
    moribund.interrupt();
  }
}

public void run(){
  while(Thread.currentThread() == runner){
    //do stuff which can be interrupted if necessary
  }
}

这只是如何停止线程的一个示例，但要点是您负责退出线程，就像您使用任何其他方法一样。维护一个跨线程通信的方法(在这种情况下是一个 volatile 变量，也可以通过互斥锁等)并在您的线程逻辑中，使用该通信方法来检查您是否应该提前退出、清理等。

2) 您的测量列表由多个线程(事件线程和您的用户线程)同时访问，无需任何同步。看起来您不必滚动自己的同步，您可以使用 BlockingQueue .

3) 您在发送线程的每次迭代中都创建一个新的 Socket。这是一个相当重量级的操作，只有在您期望测量非常罕见(比如一小时或更少)时才真正有意义。要么你想要一个不会在线程的每个循环中重新创建的持久套接字，要么你想要一个一次性可运行的，你可以“触发并忘记”它创建一个套接字、发送所有相关数据并完成。 (关于使用持久 Socket 的快速说明，阻塞的套接字方法，例如读取，不能被 Thread.interrupt() 中断，所以当你想停止线程时，你必须关闭套接字以及调用中断)

4) 从线程中抛出自己的异常没有什么意义，除非您希望在其他地方捕获它。更好的解决方案是记录错误，如果无法恢复，则停止线程。一个线程可以用类似的代码(在与上面相同的上下文中)停止自己:

public void run(){
    while(Thread.currentThread() == runner){
      //do stuff which can be interrupted if necessary

      if(/*fatal error*/){
        stopThread();
        return; //optional in this case since the loop will exit anyways
      }
    }
  }

最后，如果您想确保线程与应用程序的其余部分一起退出，无论如何，一个好的技术是在创建之后和启动线程之前调用 Thread.setDaemon(true)。这会将线程标记为守护线程，这意味着如果没有非守护线程在运行(例如，如果您的应用退出)，VM 将确保自动销毁它。

遵循线程方面的最佳做法应确保您的应用不会挂起或拖慢手机速度，尽管它们可能非常复杂 :)

关于java - 在哪里停止/销毁 Android Service 类中的线程？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/680180/