Исключение нулевого указателя в Android при попытке настроить ход игрока с помощью onRestoreInstanceState

Question

Я делаю простую многопользовательскую игру socket.io tic tac toe. Пока все работает, однако, когда я пытаюсь повернуть экран, я получаю ошибку исключения нулевого указателя. У меня есть два метода переопределения для сохранения и восстановления значений логического значения хода моего игрока.

Когда я пытаюсь напрямую установить значение, как это player1.setMyTurn(savedInstanceState.getBoolean("playerTurn")) Это дает мне исключение нулевого указателя.

Кто-нибудь знает почему?

@Override
protected void onSaveInstanceState(Bundle savedInstanceState) {
    super.onSaveInstanceState(savedInstanceState);
    savedInstanceState.putBoolean("playerTurn", player1.isMyTurn());
}

@Override
protected void onRestoreInstanceState(Bundle savedInstanceState) {
    super.onRestoreInstanceState(savedInstanceState);
    player1.setMyTurn(savedInstanceState.getBoolean("playerTurn")); //this seems to cause the issue
}

Ошибка:

E/AndroidRuntime: FATAL EXCEPTION: main
              Process: com.twochicken.multiplayertictactoe, PID: 8612
              java.lang.RuntimeException: Unable to start activity ComponentInfo{com.twochicken.multiplayertictactoe/com.twochicken.multiplayertictactoe.MainActivity}: java.lang.NullPointerException
                  at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2195)
                  at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2245)
                  at android.app.ActivityThread.handleRelaunchActivity(ActivityThread.java:3738)
                  at android.app.ActivityThread.access$900(ActivityThread.java:135)
                  at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1202)
                  at android.os.Handler.dispatchMessage(Handler.java:102)
                  at android.os.Looper.loop(Looper.java:136)
                  at android.app.ActivityThread.main(ActivityThread.java:5017)
                  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:779)
                  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:595)
                  at dalvik.system.NativeStart.main(Native Method)
               Caused by: java.lang.NullPointerException
                  at com.twochicken.multiplayertictactoe.MainActivity.onRestoreInstanceState(MainActivity.java:416)
                  at android.app.Activity.performRestoreInstanceState(Activity.java:916)
                  at android.app.Instrumentation.callActivityOnRestoreInstanceState(Instrumentation.java:1138)
                  at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2173)
                  at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2245) 
                  at android.app.ActivityThread.handleRelaunchActivity(ActivityThread.java:3738) 
                  at android.app.ActivityThread.access$900(ActivityThread.java:135) 
                  at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1202) 
                  at android.os.Handler.dispatchMessage(Handler.java:102) 
                  at android.os.Looper.loop(Looper.java:136) 
                  at android.app.ActivityThread.main(ActivityThread.java:5017) 
                  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:779) 
                  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:595) 
                  at dalvik.system.NativeStart.main(Native Method) 

Answer 1

Попробуйте инвертировать строки в первом методе

@Override
protected void onSaveInstanceState(Bundle savedInstanceState) {
    savedInstanceState.putBoolean("playerTurn", player1.isMyTurn());
    super.onSaveInstanceState(savedInstanceState);
}

А во-вторых, убедитесь, что у вас есть данные, прежде чем использовать их.

@Override
protected void onRestoreInstanceState(Bundle savedInstanceState) {
    super.onRestoreInstanceState(savedInstanceState);
     if (savedInstanceState != null) {       
          player1.setMyTurn(savedInstanceState.getBoolean("playerTurn")); 
     }
}

Совет, который я даю вам, это создать константу вместо playerTurn, это может сделать ваш код более легким в обслуживании