NullPointerException в HandlerThread

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

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

Журнал ошибок выглядит следующим образом:

FATAL EXCEPTION: main
Process: com.myproject.android, PID: 22175
java.lang.NullPointerException
    at com.myproject.android.ImageDownloaderThread.queueImage(ImageDownloaderThread.java:74)
    at com.myproject.android.NewsItemPagerActivity$NewsItemFragmentStatePagerAdapter.getItem(NewsItemPagerActivity.java:325)
    at android.support.v13.app.FragmentStatePagerAdapter.instantiateItem(FragmentStatePagerAdapter.java:109)
    at android.support.v4.view.ViewPager.addNewItem(ViewPager.java:832)
    at android.support.v4.view.ViewPager.populate(ViewPager.java:982)
    at android.support.v4.view.ViewPager.populate(ViewPager.java:914)
    at android.support.v4.view.ViewPager.onMeasure(ViewPager.java:1436)
    at android.view.View.measure(View.java:16497)
    at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:5125)
    at android.widget.FrameLayout.onMeasure(FrameLayout.java:310)
    at android.view.View.measure(View.java:16497)
    at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:5125)
    at com.android.internal.widget.ActionBarOverlayLayout.onMeasure(ActionBarOverlayLayout.java:327)
    at android.view.View.measure(View.java:16497)
    at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:5125)
    at android.widget.FrameLayout.onMeasure(FrameLayout.java:310)
    at com.android.internal.policy.impl.PhoneWindow$DecorView.onMeasure(PhoneWindow.java:2291)
    at android.view.View.measure(View.java:16497)
    at android.view.ViewRootImpl.performMeasure(ViewRootImpl.java:1912)
    at android.view.ViewRootImpl.measureHierarchy(ViewRootImpl.java:1109)
    at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:1291)
    at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:996)
    at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:5600)
    at android.view.Choreographer$CallbackRecord.run(Choreographer.java:761)
    at android.view.Choreographer.doCallbacks(Choreographer.java:574)
    at android.view.Choreographer.doFrame(Choreographer.java:544)
    at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:747)
    at android.os.Handler.handleCallback(Handler.java:733)
    at android.os.Handler.dispatchMessage(Handler.java:95)
    at android.os.Looper.loop(Looper.java:136)
    at android.app.ActivityThread.main(ActivityThread.java:5001)
    at java.lang.reflect.Method.invokeNative(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:515)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:785)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:601)
    at dalvik.system.NativeStart.main(Native Method)

И код, в котором это происходит, показан ниже:

package com.myproject.android;

import java.io.IOException;

import android.graphics.Bitmap;
import android.graphics.BitmapFactory;
import android.os.Handler;
import android.os.HandlerThread;
import android.os.Message;
import android.util.Log;

/*
 * This class is used to download images in the background thread
 */
public class ImageDownloaderThread<Token> extends HandlerThread {

    private static final String TAG = "ImageDownloader";
    private static final int MESSAGE_DOWNLOAD = 0;

    // This is the handler attached to the looper
    Handler mHandler; 






    // The is used as a reference to the main UI thread's handler
    Handler mResponseHandler;

    // This is a listener object that is used to update the main UI thread with the image that is downloaded
    Listener mListener;

    // This is the interface needed when a listener is created. It forces an implementation of the callback in the main UI thread
    public interface Listener {
        void onImageDownloaded(Bitmap image, int pos);
    }

    // Set the listener
    public void setListener(Listener listener) {
        mListener = listener;
    }





    // Constructor
    public ImageDownloaderThread(Handler responseHandler) {
        super(TAG);
        mResponseHandler = responseHandler; // Set the response handler to the one passed from the main thread
    }


    // This method executes some setup before Looper loops for each message
    @Override
    protected void onLooperPrepared() {

        // Create a message handler to handle the message queue
        mHandler = new MessageHandler(ImageDownloaderThread.this);
    }


    // This method is used to add a message to the message queue, so that it can be handled later
    // ... this method is called by the main UI thread to add the message to the queue of the current thread to be handled later
    public void queueImage(String url, int pos) {

        mHandler
            .obtainMessage(MESSAGE_DOWNLOAD, pos, 0, url)
            .sendToTarget();
    }





    // This method is used to download the image  
    private void handleRequest(String url, int pos) {

        try {

            // first check if the url is empty. if it is, then return
            if (url == null) {
                return;
            }

            // Download the image
            byte[] bitmapBytes = new NewsItemsFetcher().getUrlBytes(url);

            // Generate a bitmap
            final Bitmap bitmap = BitmapFactory.decodeByteArray(bitmapBytes, 0, bitmapBytes.length);

            // Set position as 'final'
            final int position = pos;


            // We are using mResponseHandler.post(Runnable) to send a message to the response handler
            // This message will eventually result in the main thread updating the UI with the image
            mResponseHandler.post(new Runnable() {
                @Override
                public void run() {                 
                    mListener.onImageDownloaded(bitmap, position);

                }
            });

        }

        catch (HttpResponseException httpe) {
            // TODO: Handle http response not OK
            Log.e(TAG, "Error in server response", httpe);
        }

        catch (IOException ioe) {
            // TODO: Handle download error
            Log.e(TAG, "Error downloading image", ioe);
        }

    }


    class MessageHandler extends Handler {

        private final ImageDownloaderThread<Token> mImageDownloader;

        MessageHandler(ImageDownloaderThread<Token> imageDownloader) {
            mImageDownloader = imageDownloader;
        }

        // This method is used to process the message that is waiting in the queue 
        @Override
        public void handleMessage(Message msg) {

            // First, check if the message is to download an image
            if (msg.what == MESSAGE_DOWNLOAD) {

                // Call the handleRequest() function which will eventually download the image
                String url = (String)msg.obj;
                int pos = msg.arg1;


                if (mImageDownloader != null) {
                    mImageDownloader.handleRequest(url, pos);
                }

            }
        }

    }

}

Если вам интересно, строка 74 в журнале ошибок (точнее, эта at com.myproject.android.ImageDownloaderThread.queueImage(ImageDownloaderThread.java:74) ссылается на .obtainMessage(MESSAGE_DOWNLOAD, pos, 0, url) строку кода в queueImage()


РЕДАКТИРОВАТЬ

Согласно предложению в ответе Loop, mHandler равно null при вызове queueImage(). Итак, как я могу гарантировать, что mHandler будет инициализирован onLooperPrepared() перед выполнением любого вызова queueImage()?




Ответы (2)


Единственная причина для меня заключается в том, что метод queueImage() вызывается перед onLooperPrepared(), поэтому mHandler не инициализируется.

Обновить

HandlerThread — это просто Thread с реализацией метода run(), где вызывается onLooperPrepared().

@Override
public void run() {
    mTid = Process.myTid();
    Looper.prepare();
    synchronized (this) {
        mLooper = Looper.myLooper();
        notifyAll();
    }
    Process.setThreadPriority(mPriority);
    onLooperPrepared();//It's HERE
    Looper.loop();
    mTid = -1;
}

Поэтому, когда он вызывается, зависит от запуска этого потока. Если вы запустите его и немедленно вызовете публичный метод по ссылке этого потока, вы можете столкнуться с состоянием гонки, и mHandler не будет инициализирован вовремя.

Одним из решений может быть задержка начала обработки изображений или игра с методами синхронизации. Однако я бы использовал гораздо более простой способ.

Просто для ясности: вы хотите, чтобы ваш mHandler был инициализирован сразу после создания HandlerThread, и вы не хотите делать это явно из основного действия, где создается HandlerThread.

Обновление 2

Просто придумайте следующее решение.

queueImage() предоставляет простые и легкие данные. Вы можете проверить, является ли mHandler нулевым, если это правда, добавьте параметры queueImage() в эту очередь. Когда вызывается onLoopPrepared(), проверьте, есть ли что-нибудь в этой очереди, и обработайте эти данные.

private LinkedBlockingQueue<Pair<String,Integer>> mQueue = new LinkedBlockingQueue<Pair<String,Integer>>();

public void queueImage(String url, int pos) {
    if (mHandler == null) {
        mQueue.put(new Pair<String,Integer>(url, pos));
        return;
    }
    mHandler
        .obtainMessage(MESSAGE_DOWNLOAD, pos, 0, url)
        .sendToTarget();
}

@Override
protected void onLooperPrepared() {

    // Create a message handler to handle the message queue
    mHandler = new MessageHandler(ImageDownloaderThread.this);
    //TODO check the queue here, if there is data take it and process
    //you can call queueImage() once again for each queue item
    Pair<String, Integer> pair = null;
    while((pair = mQueue.poll()) != null) {
        queueImage(pair.first, pair.second);
    }
}
person Damian Petla    schedule 23.08.2014
comment
Но, согласно документации Android, onLooperPrepared() вызывается перед циклами Looper. Точная формулировка: Call back method that can be explicitly overridden if needed to execute some setup before Looper loops. (для получения более подробной информации перейдите по ссылке developer.android.com/reference/android/os/). Теперь, почему queueImage() вызывается раньше onLooperPrepared()? - person Greeso; 23.08.2014
comment
Кстати, мне прикольно, что у тебя ID Loop, а мы обсуждаем Loopers :) - person Greeso; 23.08.2014
comment
Чтобы вы знали, я провел несколько тестов. И вы правы, mHandler ЕСТЬ null. Я предпочитаю ваше второе предложенное решение, но я не хочу пропускать некоторые queueImage() вызовы. Как я могу заставить queueImage() ждать завершения выполнения onLooperPrepared(), чтобы завершить инициализацию? - person Greeso; 23.08.2014
comment
Можете ли вы показать, где и когда вызывается queueImage()? - person Damian Petla; 23.08.2014
comment
Я обновил вопрос. Кстати, спасибо за помощь. - person Greeso; 23.08.2014
comment
Мне нравится обновление 2, но не могли бы вы объяснить его немного подробнее, пожалуйста? в какой очереди вы стоите, когда mHandler становится null? Спасибо - person Greeso; 23.08.2014
comment
Кроме того, просто к вашему сведению, решение для обновления 1 не сработает. Теперь это вызывает android.os.NetworkOnMainThreadException. - person Greeso; 23.08.2014
comment
Да, я забыл, что обработчик прикрепляется к петлителю текущего потока. Мы могли бы попробовать вызвать getLooper() для HandlerThread, но в конструкторе он вернет null. Вскоре я обновлю свой ответ, удалив обновление 1, и приведу пример для обновления 2. - person Damian Petla; 23.08.2014
comment
Большое спасибо. Вы очень полезны. Я так благодарен вам. Это сработало. Я бы дал вам десять голосов, но сайт позволяет только один. Кстати, я использовал очередь LinkedList вместо LinkedBlockingQueue, поскольку доступ к этой очереди осуществляется только через один поток (поток фонового зацикливания). Кроме этого, все отлично. Спасибо. - person Greeso; 23.08.2014
comment
Будьте осторожны, так как queueImage и onLooperPrepared обычно вызываются из разных потоков, поэтому сообщения могут быть потеряны из-за условий гонки. В этом случае необходима правильная синхронизация. - person fdermishin; 06.08.2017

Я встречаю тот же вопрос. Я решил, вызвав wait() перед тем, как поставить сообщение в очередь, и вызвав call notifyAll() в onLooperPrepared. Для этого не нужны дополнительные переменные, в которых хранятся ожидающие сообщения.

person robert    schedule 19.10.2016
comment
Это отличный ответ, если вы можете позволить себе заблокировать ветку. Я испытываю это состояние гонки в потоке пользовательского интерфейса Android, которое я не могу заблокировать. - person Joe Lapp; 05.02.2017