Reputation: 1973
I need to receive a status from the server every 10 seconds.
I tried to do that by sending a http request via service.
The problem is that my code executes only once.
This is the code for my service:
public class ServiceStatusUpdate extends Service {
@Override
public IBinder onBind(Intent intent) {
// TODO Auto-generated method stub
return null;
}
@Override
public int onStartCommand(Intent intent, int flags, int startId) {
while(true)
{
try {
Thread.sleep(10000);
} catch (InterruptedException e) {
new DoBackgroundTask().execute(Utilities.QUERYstatus);
e.printStackTrace();
}
return START_STICKY;
}
}
private class DoBackgroundTask extends AsyncTask<String, String, String> {
@Override
protected String doInBackground(String... params) {
String response = "";
String dataToSend = params[0];
Log.i("FROM STATS SERVICE DoBackgroundTask", dataToSend);
HttpClient httpClient = new DefaultHttpClient();
HttpPost httpPost = new HttpPost(Utilities.AGENT_URL);
try {
httpPost.setEntity(new StringEntity(dataToSend, "UTF-8"));
// Set up the header types needed to properly transfer JSON
httpPost.setHeader("Content-Type", "application/json");
httpPost.setHeader("Accept-Encoding", "application/json");
httpPost.setHeader("Accept-Language", "en-US");
// Execute POST
HttpResponse httpResponse = httpClient.execute(httpPost);
HttpEntity responseEntity = httpResponse.getEntity();
if (responseEntity != null) {
response = EntityUtils.toString(responseEntity);
} else {
response = "{\"NO DATA:\"NO DATA\"}";
}
} catch (ClientProtocolException e) {
response = "{\"ERROR\":" + e.getMessage().toString() + "}";
} catch (IOException e) {
response = "{\"ERROR\":" + e.getMessage().toString() + "}";
}
return response;
}
@Override
protected void onPostExecute(String result) {
Utilities.STATUS = result;
Log.i("FROM STATUS SERVICE: STATUS IS:", Utilities.STATUS);
super.onPostExecute(result);
}
}
}
Thank's alot Avi
Upvotes: 11
Views: 34823
Reputation: 1
put return START_STICKY; after while , and service will work properly
Upvotes: 0
Reputation: 437
Handler handler_proc = new Handler();
final Runnable runnable_proc = new Runnable() {
public void run() {
requestHttp(new OnFinish() {
@Override
public void onSuccess() {
// pause should be considered since the end of the previous http request.
// Otherwise, new requests will occur before the previous one is complete,
// if http_timeout > delay
handler_proc.postDelayed(runnable_proc, 10000);
}
});
}
}
handler_proc.post(runnable_proc);
Upvotes: 0
Reputation: 169
The reason the code only runs once is that there's a return
in your timer loop. However, I'd advise moving the loop to your background task or using one of the other answers given here rather than implementing a loop in onStartCommand()
that never returns.
Upvotes: 0
Reputation: 807
Put a handler inside onPostExecute to send a http request after 10 secs
new Handler().postDelayed(new Runnable() {
public void run() {
requestHttp();
}
}, 10000);
After 10 secs doInBackground will be executed again, after that onPostExecute again, handler again and so on and so on..
Upvotes: 19
Reputation: 3080
You can simply use CountDownTimer
class in android.
public class AlertSessionCountDownTimer extends CountDownTimer {
public AlertSessionCountDownTimer(long startTime, long interval) {
super(startTime, interval);
}
@Override
public void onFinish() {
//YOUR LOGIC ON FINISH 10 SECONDS
}
@Override
public void onTick(long millisUntilFinished) {
//YOUR LOGIC ON TICK
}
}
Hope it will help you.
Upvotes: 1