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 Build a Weather App (2015) Concurrency and Error Handling Executing an Asynchronous Call

can anyone help me to clean up this code. where am i get it wrong

Continuing with our movie app, we are now ready to asynchronously execute the request we previously built. Start by creating a Call variable and set it using the newCall() method from the OkHttpClient. Hint: newCall() requires a parameter of the Request type.

MovieActivity.java
import android.app.Activity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import java.io.IOException;

public class MovieActivity extends Activity {

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

        // Get some movie information!
        String apiUrl = "http://api.rottentomatoes.com/api/public/v1.0/movies.json?apikey=xyz&q=hobbit";
        OkHttpClient client = new OkHttpClient();
        Request request = new Request.Builder()
                .url(apiUrl)
                .build();
      Call call = client.newCall(request);
      call.enqueue(new newCall(){
         public void onFailure(Request.request,IOException e){
         }
      });
    }

}

4 Answers

Hello,

You're so close on this one. All this task wants you to do is create the Call object. Remove your code after that

call.enqueue(new newCall(){
         public void onFailure(Request.request,IOException e){
         }
      });

and you will pass the task.

All he wants you to do is remove

call.enqueue(new newCall(){ public void onFailure(Request.request,IOException e){ } });

And You're done

All he wants you to do is remove

call.enqueue(new newCall(){ public void onFailure(Request.request,IOException e){ } });

And You're done

thanks James, but am still having some errors. whats missing on this code

Could you post your current code?