Блог

Я пытаюсь создать игру для Android, и я следую нескольким примерам кода, чтобы заставить мой игровой цикл работать. Это включает в себя создание нового потока. В run() методе у меня есть блок try / finally. После выполнения блока finally выдается NullPointerException . Я понятия не имею, почему, ничто не кажется нулевым, и даже если это так, ничто не ссылается ни на что нулевое. Я думал, что, возможно, this было null, но, похоже, это не так. Вот код, который, я думаю, имеет отношение к делу:

 public class MainThread extends Thread {
private boolean running;
private final SurfaceHolder holder;

private boolean GameIsRunning = false;

private int mMode;
public static final int STATE_LOSE = 1;
public static final int STATE_PAUSE = 2;
public static final int STATE_READY = 3;
public static final int STATE_RUNNING = 4;
public static final int STATE_WIN = 5;

private MainGame game;

public MainThread(SurfaceHolder holder, MainGamePanel panel) {
    super();
    this.holder = holder;
    game = new MainGame(panel.getContext());
    mMode = STATE_RUNNING;
}

@Override
public void run() {
    while (running) {
        Canvas c = null;
        try {
            c = holder.lockCanvas(null);
            synchronized (holder) {
                if (mMode == STATE_RUNNING) {
                    updateAll();
                }
                drawAll(c);
            }
        } finally {
            // do this in a finally so that if an exception is thrown
            // during the above, we don't leave the Surface in an
            // inconsistent state
            if (c != null) {
                holder.unlockCanvasAndPost(c);
            }
        } // <<<<<<<<<<<<< After this line executes a null pointer exception is thrown
    } 
}
  

Создание потока:

 public class MainGamePanel extends SurfaceView implements SurfaceHolder.Callback{

private MainThread thread;

public MainGamePanel(Context context) {
    super(context);
    getHolder().addCallback(this);

    // create the game loop thread
    thread = new MainThread(getHolder(), this);
    setFocusable(true);
}


@Override
public void surfaceCreated(SurfaceHolder holder) {
    thread.setRunning(true);
    thread.start();
}
  

Спасибо!

NPE генерируется внутри блока try и становится видимым после выполнения блока finally.

Судя по коду, это происходит, скорее всего, потому, что c это null происходит, когда вы передаете его в updateAll . У вас есть null проверка внутри блока finally — так что, я полагаю, вы ожидаете, что оно может быть нулевым. Добавьте еще одну проверку в блок try и обработайте c == null там тоже.

Из Android API ( SurfaceHolder#lockCanvas ):

Возвращается null, если поверхность не была создана или иным образом не может быть отредактирована

Попробуйте заменить c = holder.lockCanvas(null); на c = holder.lockCanvas();

И, конечно, вы должны опубликовать свой stacktrace.