Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

Android Threads and Services Introducing Services Creating a Service

Posted by Mackenzie Shonayi
Mackenzie Shonayi
23,069 Points

I need help guys

I cant see whats wrong with the sendMessage code

TwitterService.java 
public class TwitterService extends Service {
  TwitterClient twitterClient = new TwitterClient();    
  TwitterThread twitterThread;
  TwitterHandler mHandler;
  @Override
  public void onCreate(){
    twitterThread = new TwitterThread();
    twitterThread.setName("TwitterThread");
    twitterThread.start();

    mHandler = twitterThread.twitterHandler;
    mHandler.setService(this);
  }
  @Override
  public IBinder onBind(Intent intent){
    return null;
  }
  public int onStartCommand(Intent intent, int flags, int startId){
    Message message = Message.obtain();
    mHandler.sendMessage(message);
  }


  class TwitterThread extends Thread {
    TwitterHandler twitterHandler;
    @Override
    public void run() {
      Looper.prepare();
      twitterHandler = new TwitterHandler();
      Looper.loop();
    }
  }    

  class TwitterHandler extends Handler {
    @Override
    public void handleMessage(Message msg) {
      twitterClient.update();
    }
  }
}
MainActivity.java 
public class MainActivity extends Activity {
  Button button;   

  @Override
  public void onCreate(Bundle savedInstanceState) {
    setContentView(R.layout.activity_main);

    button = (Button) findViewById(R.id.update_button);
    button.setOnClickListener(new View.OnClickListener() {
      @Override
      public void onClick(View v){

      }
    });
  }
}

2 Answers

Ben Deitch
STAFF
Ben Deitch
Treehouse Teacher
Ben Deitch
Ben Deitch
Treehouse Teacher

Hey Mackenzie! It looks like OnStartCommand isn't returning :)

Mackenzie Shonayi
Mackenzie Shonayi
23,069 Points
Mackenzie Shonayi
Mackenzie Shonayi
23,069 Points

Thanx Ben but after adding the following code i am still getting an error.

public int onStartCommand(Intent intent, int flags, int startId){ Message message = Message.obtain(); mHandler.sendMessage(message); return Service.START_REDELIVER_INTENT; }

Unsubscribed User
Unsubscribed User
48,988 Points

worked for me :

public class TwitterService extends Service {
  TwitterClient twitterClient = new TwitterClient();    
  TwitterThread twitterThread;

  @Override 
  public void onCreate() {
    twitterThread = new TwitterThread();
    twitterThread.start();
  }

  @Override
  public IBinder onBind(Intent intent) {
    return null;
  }

  @Override
  public int onStartCommand(Intent intent, int flags, int startId){
      Message message = Message.obtain();
      twitterThread.twitterHandler.sendMessage(message);
      return Service.START_REDELIVER_INTENT;
  }

   class TwitterThread extends Thread {
    TwitterHandler twitterHandler;
    @Override
    public void run() {
      Looper.prepare();
      twitterHandler = new TwitterHandler();
      Looper.loop();
    }
  }    

  class TwitterHandler extends Handler {
    @Override
    public void handleMessage(Message msg) {
      twitterClient.update();
    }
  }
}```