NullPointerExceptionが出る原因が分からない

Question

エラー

java.lang.NullPointerException: Attempt to invoke interface method 'int android.content.SharedPreferences.getInt(java.lang.String, int)' on a null object reference
        at com.test.quizmaker.MainActivity$2.onClick(MainActivity.java:61)

やりたいこと

アプリを閉じてから更新をできるようにしたい。

試したこと

アプリを閉じずにbutton1を押し、更新（データ読み込み）することはできた。が、アプリを閉じてからbutton1を押すとNullPointerException

わかっていること

adbshellでSharedPreferenceでしっかり保存されていることが確認できた。

<map>
    <string name="title1">t</string>
    <string name="answer1">a</string>
    <int name="count" value="1" />
    <string name="quiz1">q</string>
</map>

また、MainActivity61行目

int updateInt = PreferenceManager.sp.getInt("count",10);

で、値がnullになっている。

知りたいこと

countというキー名でアプリを閉じる事前に保存されていることが確認できたのになぜヌルポが出てしまうのかを知りたいです。

コード

public class MainActivity extends AppCompatActivity {
    private ArrayList data = new ArrayList<String>();
    private static final int REQUEST_CODE = 1;
    private ArrayAdapter adapter;

    @Override
    protected void onActivityResult(int request_Code, int resultCode, Intent result){
        switch(request_Code){
            //MakeQuizActivityから戻ってきた場合
            case(REQUEST_CODE):
                //クイズを作るボタンを押して戻ってきたときの処理
                if(resultCode == RESULT_OK){
                        data.add(result.getStringExtra("INPUT_TITLE"));
                        adapter.notifyDataSetChanged();
                }
                //押されなかったときの処理
                else if(resultCode == RESULT_CANCELED){

                }
                break;
        }
    }
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        // リスト項目とListViewを対応付けるArrayAdapterを用意する
        adapter = new ArrayAdapter<>(this, android.R.layout.simple_list_item_1, data);
        // ListViewにArrayAdapterを設定する
        ListView listView = (ListView) findViewById(R.id.listView);
        listView.setAdapter(adapter);
        Button button = (Button) findViewById(R.id.makeQuiz_Button1);
        button.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent = new Intent(MainActivity.this, com.test.quizmaker.MakeQuizActivity.class);
                startActivityForResult(intent, REQUEST_CODE);
            }
        });
        Button button1 = (Button)findViewById(R.id.update_Button);
        button1.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                //現時点のコード
                int updateInt = PreferenceManager.sp.getInt("count",10);
                ArrayList updateString = new ArrayList<String>();
                updateString.add(PreferenceManager.sp.getString(DataKeys.TITLE.getValue() + String.valueOf(updateInt),"nothing"));
                data.add(updateString);
                adapter.notifyDataSetChanged();
                System.out.println(updateString);
            }
        });
    }
    //オプションメニューを作成
    public boolean onCreateOptionsMenu(Menu menu){
        //menuにcustom_menuレイアウトを適用
        getMenuInflater().inflate(R.menu.custom_menu, menu);
        //オプションメニューを表示する場合はtrue
        return true;
    }
    //メニュー選択時の処理　トースト表示
    @Override
    public boolean onOptionsItemSelected(MenuItem menuItem){
        //押されたメニューのIDで処理を振り分ける
        //if(menuItem.getItemId() == R.id.menuClear){
        switch (menuItem.getItemId()) {
            case R.id.menuSelect:

            case R.id.menuClear:
                adapter.clear();
                adapter.notifyDataSetChanged();
                PreferenceManager.clear();
                break;
        }
        return true;
    }
}

public class MakeQuizActivity extends AppCompatActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_make_quiz);

        Button button2 = (Button)findViewById(R.id.makeQuiz_Button2);
        button2.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent = new Intent(MakeQuizActivity.this,com.test.quizmaker.MainActivity.class);
                //PreferenceManager pm = new PreferenceManager();
                SharedPreferences sp = getSharedPreferences("DataStore",MODE_PRIVATE);
                SharedPreferences.Editor editor = sp.edit();
                PreferenceManager.init(sp,editor);
                String strTitle = ((EditText)findViewById(R.id.editText)).getText().toString();
                String strQuiz = ((EditText)findViewById(R.id.editText2)).getText().toString();
                String strAnswer = ((EditText)findViewById(R.id.editText3)).getText().toString();
                int count = 1;
                PreferenceManager.saveInt(count);
                PreferenceManager.saveString(strTitle, DataKeys.TITLE, String.valueOf(count));
                PreferenceManager.saveString(strQuiz, DataKeys.QUIZ, String.valueOf(count));
                PreferenceManager.saveString(strAnswer, DataKeys.ANSWER, String.valueOf(count));
                String title = PreferenceManager.sp.getString(DataKeys.TITLE.getValue() + String.valueOf(count), "Nothing");
                String quiz = PreferenceManager.sp.getString(DataKeys.QUIZ.getValue() + String.valueOf(count),"Nothing");
                String answer = PreferenceManager.sp.getString(DataKeys.ANSWER.getValue() + String.valueOf(count),"Nothing");
                intent.putExtra("INPUT_TITLE", title);
                intent.putExtra("INPUT_QUIZ",quiz);
                intent.putExtra("INPUT_ANSWER",answer);
                count ++;
                setResult(RESULT_OK, intent);
                finish();
            }
        });
    }
}

public class PreferenceManager{

    public static SharedPreferences sp;
    private static SharedPreferences.Editor spEditor;

    public static void init(SharedPreferences sp1, SharedPreferences.Editor editor1) {
        sp = sp1;
        spEditor = editor1;
    }

    public static void saveString(String s, DataKeys key, String c) {
        spEditor.putString(key.getValue() + c, s);
        spEditor.apply();
    }

    public static void saveInt(int c) {
        spEditor.putInt("count", c);
        spEditor.apply();
    }

    public static void clear(){
        spEditor.clear().commit();
    }
}

public enum DataKeys {
    TITLE("title"), QUIZ("quiz"), ANSWER("answer");

    private String value;

    private DataKeys(String value) {
        this.value = value;
    }

    public String getValue() {
        return this.value;
    }
}

Comments

[keicha_hrs]

コードを記述する部分の1行目の```の右側には、ファイル名ではなく言語名（javaとか）を記述してください。

Answer 1

PreferenceManagerクラスのフィールドのspはinit()メソッドが呼ばれたときに初期化されるようですが、このinit()メソッドを呼んでいるのはMakeQuizActivityでbutton2をクリックしたときだけのように思えます。MainActivityからアプリを再び起動し、MakeQuizActivityを経ることなくbutton1をクリックすると、init()メソッドが呼ばれる機会がないように思います。すると、spはnullのままということになるでしょう。そこで、

                //現時点のコード
                int updateInt = PreferenceManager.sp.getInt("count",10);

のように呼び出すと、中身がnullであるspを参照しているので、NullPointerExceptionになるのではないでしょうか。

検証するためには、この行にブレークポイントを設定してデバッグ実行し、spの値を確認してみることでしょう。

Comments

[j06110611]

なるほど！分かりやすい解説ありがとうございます！助かります！

Answer 2

実際に動かして試せてないのですが、アプリ起動直後にbutton1を押下すると、以下の処理がまだ行われていないのが原因ということはないですか？

※アプリ起動直後だと、PreferenceManagerクラスの"sp"という変数がインスタンス設定されておらず、NullPointerExceptionが発生しているのではないか。

SharedPreferences sp = getSharedPreferences("DataStore",MODE_PRIVATE);
SharedPreferences.Editor editor = sp.edit();
PreferenceManager.init(sp,editor);

Comments

[j06110611]

なるほど！初期化ができてないということですね。ありがとうございます！