Ответные обратные вызовы Okhttp в основной теме

Я создал вспомогательный класс для обработки всех моих HTTP-вызовов в моем приложении. Это простая одноэлементная оболочка для okhttp, которая выглядит так (я опустил некоторые несущественные части):

public class HttpUtil {

    private OkHttpClient client;
    private Request.Builder builder;

    ...

    public void get(String url, HttpCallback cb) {
        call("GET", url, cb);
    }

    public void post(String url, HttpCallback cb) {
        call("POST", url, cb);
    }

    private void call(String method, String url, final HttpCallback cb) {
        Request request = builder.url(url).method(method, method.equals("GET") ? null : new RequestBody() {
            // don't care much about request body
            @Override
            public MediaType contentType() {
                return null;
            }

            @Override
            public void writeTo(BufferedSink sink) throws IOException {

            }
        }).build();

        client.newCall(request).enqueue(new Callback() {
            @Override
            public void onFailure(Request request, Throwable throwable) {
                cb.onFailure(null, throwable);
            }

            @Override
            public void onResponse(Response response) throws IOException {
                if (!response.isSuccessful()) {
                    cb.onFailure(response, null);
                    return;
                }
                cb.onSuccess(response);
            }
        });
    }


    public interface HttpCallback  {

        /**
         * called when the server response was not 2xx or when an exception was thrown in the process
         * @param response - in case of server error (4xx, 5xx) this contains the server response
         *                 in case of IO exception this is null
         * @param throwable - contains the exception. in case of server error (4xx, 5xx) this is null
         */
        public void onFailure(Response response, Throwable throwable);

        /**
         * contains the server response
         * @param response
         */
        public void onSuccess(Response response);
    }

}

Затем в моем основном действии я использую этот вспомогательный класс:

HttpUtil.get(url, new HttpUtil.HttpCallback() {
            @Override
            public void onFailure(Response response, Throwable throwable) {
                // handle failure
            }

            @Override
            public void onSuccess(Response response) {
                // <-------- Do some view manipulation here
            }
        });

onSuccess генерирует исключение при запуске кода:

android.view.ViewRootImpl $CalledFromWrongThreadException: только оригинальный поток, создавший иерархию представлений, может коснуться его представлений.

С моей точки зрения, обратные вызовы Okhttp работают в основном потоке, так почему я могу получить эту ошибку?

** Как и примечание, я создал интерфейс HttpCallback для оболочки класса Okhttp Callback, потому что я хотел изменить поведение onResponse и onFailure, чтобы я мог объединить логику обработки неудачных ответов из-за исключений i/o и неудачных ответов из-за проблем с сервером.

Спасибо.

Ответ 1

С моей точки зрения, обратные вызовы Okhttp запускаются в основном потоке, так почему я могу получить эту ошибку?

Это неверно. Обратные вызовы выполняются в фоновом потоке. Если вы хотите немедленно обработать что-то в пользовательском интерфейсе, вам нужно будет отправить сообщение в основной поток.

Поскольку у вас уже есть обертка вокруг обратного вызова, вы можете сделать это внутри вашего помощника, чтобы все удобства HttpCallback вызывались в основном потоке для удобства.

Ответ 2

Как предложил Джейк Уортон, я должен был явно запустить обратные вызовы в основном потоке.

Итак, я завернул вызовы обратным вызовам с помощью Runnable следующим образом:

private void call(String method, String url, final HttpCallback cb) {
    ...

    client.newCall(request).enqueue(new Callback() {
            Handler mainHandler = new Handler(context.getMainLooper());

            @Override
            public void onFailure(Request request,final Throwable throwable) {
                mainHandler.post(new Runnable() {

                    @Override
                    public void run() {
                        cb.onFailure(null, throwable);
                    }
                });

            }

            @Override
            public void onResponse(final Response response) throws IOException {
                mainHandler.post(new Runnable() {

                    @Override
                    public void run() {
                        if (!response.isSuccessful()) {
                            cb.onFailure(response, null);
                            return;
                        }
                        cb.onSuccess(response);
                    }
                });

            }
        });
 }

Ответ 3

Я знаю, это старый вопрос, но в последнее время я столкнулся с той же проблемой. Если вам нужно обновить любое представление, вам нужно будет использовать runOnUiThread() или опубликовать результат в основном потоке.

HttpUtil.get(url, new Callback() { //okhttp3.Callback
   @Override
   public void onFailure(Call call, IOException e) { /* Handle error **/ }

   @Override
   public void onResponse(Call call, Response response) throws IOException {

      String myResponse =  response.body().string();
      //Do something with response
      //...

      MyActivity.this.runOnUiThread(new Runnable() {
            @Override
            public void run() {
               //Handle UI here                        
               findViewById(R.id.loading).setVisibility(View.GONE);                
            }
        });
   }
});