Reputation: 2367
I am working on app that check some of the status of phone every 5 seconds. I done it that:
Thread checking = new Thread() {
public void run(){
while( <<some status>> ) {
<<checkstatus and do something>>
Thread.sleep(5000);
}
}
}
This is bad for the battery? How can I do it in the other way? Service stop working after couple seconds.
Upvotes: 7
Views: 4337
Reputation: 17895
Thread.sleep() makes current thread to suspend, which means CPU will be available for other threads or will simply go off. Please also note sleep forces context switch, but you are on the safe side with big sleep interval. So, generally speaking, everything is up to you.
You can use, for example, PowerTutor to make sure your application is properly designed and implemented.
Upvotes: 3
Reputation: 5971
To answer your question, it depends what you are doing inside the while loop.
If you are not using and waking up any "expensive" components (such as internet, bluetooth, other sensors, and screen), you should be fine. Essentially this is how the OS works in the background anyways and using a Thread is one of the cheapest ways.
Also, if your task is CPU intensive, you would drain the battery as well.
Your service should work fine if you do it right.
Another way would be using Handlers, which will abstract much of the thread managements for you.
Upvotes: 7