一尘不染

Android Service类中的哪里停止/销毁线程?

java

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

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);
         }
    }
}

...
}

关闭应用程序后,电话的工作速度非常慢,我猜这是由于线程终止失败引起的。

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


阅读 790

收藏
2020-09-08

共1个答案

一尘不染

附录
:Android框架为一次性工作,后台工作等提供了许多帮助程序,与在许多情况下尝试滚动自己的线程相比,它们可能更可取。如下面的文章所述,AsyncTask是一个很好的起点。我鼓励读者在开始考虑自己编写线程之前先研究一下框架规定。

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

1)Thread.stop()已被弃用一段时间,因为在某些情况下,它可能会使因变量处于不一致状态。有关更多详细信息,请参见此Sun答案页面(编辑:该链接现在已失效,对于为何不使用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
  }
}

这只是如何停止线程的一个示例,但总的来说,您有责任像退出其他任何方法一样退出线程。维护跨线程通信的方法(在这种情况下为易失性变量,也可以通过互斥锁等),并在线程逻辑内,使用该通信方法检查是否应及早退出,清理等。

2)您的度量值列表可以同时被多个线程(事件线程和用户线程)访问,而不会进行任何同步。看起来您不必进行自己的同步,可以使用BlockingQueue

3)您将在发送线程的每次迭代中创建一个新的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将确保将其自动销毁。

遵循关于线程的最佳实践应确保您的应用不会挂起或降低电话速度,尽管它们可能非常复杂:)

2020-09-08