Where to stop/destroy threads in Android Service class?
Addendum: The Android framework provides many helpers for one-off work, background work, etc, which may be preferable over trying to roll your own thread in many instances. As mentioned in a below post, AsyncTask is a good starting point to look into. I encourage readers to look into the framework provisions first before even beginning to think about doing their own threading.
There are several problems in the code sample you posted I will address in order:
1) Thread.stop() has been deprecated for quite some time now, as it can leave dependent variables in inconsistent states in some circumstances. See this Sun answer page for more details (Edit: that link is now dead, see this page for why not to use Thread.stop()). A preferred method of stopping and starting a thread is as follows (assuming your thread will run somewhat indefinitely):
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
}
}
This is just one example of how to stop a thread, but the takeaway is that you are responsible for exiting a thread just as you would any other method. Maintain a method of cross thread communcation (in this case a volatile variable, could also be through a mutex, etc) and within your thread logic, use that method of communication to check if you should early exit, cleanup, etc.
2) Your measurements list is accessed by multiple threads (the event thread and your user thread) at the same time without any synchronization. It looks like you don't have to roll your own synchronization, you can use a BlockingQueue.
3) You are creating a new Socket every iteration of your sending Thread. This is a rather heavyweight operation, and only really make sense if you expect measurements to be extremely infrequent (say one an hour or less). Either you want a persistent socket that is not recreated every loop of the thread, or you want a one shot runnable you can 'fire and forget' which creates a socket, sends all relevant data, and finishes. (A quick note about using a persistent Socket, socket methods which block, such as reading, cannot be interrupted by Thread.interrupt(), and so when you want to stop the thread, you must close the socket as well as calling interrupt)
4) There is little point in throwing your own exceptions from within a Thread unless you expect to catch it somewhere else. A better solution is to log the error and if it is irrecoverable, stop the thread. A thread can stop itself with code like (in the same context as above):
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
}
}
}
Finally, if you want to be sure a thread exits with the rest of your application, no matter what, a good technique is to call Thread.setDaemon(true) after creation and before you start the thread. This flags the thread as a daemon thread, meaning the VM will ensure that it is automatically destroyed if there are no non-daemon threads running (such as if your app quits).
Obeying best practices with regards to Threads should ensure that your app doesn't hang or slow down the phone, though they can be quite complex :)
How to stop thread inside Android service
Use thread.interrupt()
to interrupt the thread. Add try-catch
around Thread.sleep(1000)
call:
class TestRunnable : Runnable {
override fun run() {
var isInterrupted = false
for(i in 1..15) {
Log.d("Debug", "startThread " + i)
try {
Thread.sleep(1000)
} catch (e: InterruptedException) {
isInterrupted = true
}
if (isInterrupted) break
}
}
}
You can also use some AtomicBoolean
variable to break the loop in Runnable
implementation:
class TestRunnable : Runnable {
private val isStopped = AtomicBoolean(false)
fun stop() {
isStopped.set(true)
}
override fun run() {
for(i in 1..15) {
if (isStopped.get()) break
// ...
}
}
}
override fun onDestroy() {
if(thread.isAlive) {
runnable.stop()
thread.interrupt()
}
super.onDestroy()
}
How to stop or destroy a running Thread
you cannot destroy...only the android will stop the thread when requires.. you cannot stop or destroy it.. instead try like this..
class MyThread extends Thread
{
void run()
{
while(bool){
//My code which takes time.
}
}
}
//-------------------------- To run the thread
MyThread mThread = new MyThread();
mThread.start();
now when u want to stop the thread... change bool value to false
bool=false;
now your code doesnt run... and you can start new thread...
How do I kill an Android thread completely?
to kill the thread , i think you can do like this :
myService.getThread().interrupt();
NOTE : the method Thread.stop()
is deprecated
EDIT : : try this
public void stopThread(){
if(myService.getThread()!=null){
myService.getThread().interrupt();
myService.setThread(null);
}
}
How to close a background thread in a service when the service get destroyed
Thread life cycle is not the same as Android Context Component life cycle. Stopping the service is not enough to stop the thread which service created. Thread.interrupt()
is a option. - You should catch InterruptedException
though. If it is not enough, you can check if service is not stopped inside your connect()
method.
Stop threads created by Android Service
boolean mStatus = true;
@Override
public void onCreate() {
1Thread = new Thread() {
public void run() {
while (mStatus) {
try {
Thread.sleep(180000); // 3 minutes
} catch (InterruptedException e) {
e.printStackTrace();
continue;
}
Log.i("TEST", "Thread is still here!");
}
};
}
@Override
public void onDestroy() {
mStatus = false;
1Thread.interrupt();
}
Related Topics
Is Default No-Args Constructor Mandatory for Gson
How to Limit the Number of Characters in Jtextfield
How to Scroll More Than One Object at the Same Time
Can Not Find the Tag Library Descriptor for "Http://Java.Sun.Com/Jsp/Jstl/Core"
Printing a Jframe and Its Components
Java.Lang.Illegalstateexception: Scanner Closed
Use of Initializers VS Constructors in Java
How to Set the Style of List View Cells Based on a Condition in Javafx
How to Give Images Rounded Corners in Android
Java.Lang.Runtimeexception: Takepicture Failed
Where to Stop/Destroy Threads in Android Service Class
Inside Onclicklistener I Cannot Access a Lot of Things - How to Approach
Check for Active Internet Connection Android