SharedPreferencesの設計

Question

前提・実現したいこと

アプリ起動時に、SharedPreferencesの値を読み込んで、
アプリ終了時に、SharedPreferencesの値を書き込むことを考えています。
具体的には、前回のスクロールビューのスクロール位置を起動時に復元することを考えています。
スクロールビューの実装はまだしてませんが、
とりあえず、この用途でのSharedPreferencesの値の読み書きの設計についての質問です。

このようなとき、SharedPreferencesは、各メソッドごとに宣言して用意するのが適切でしょうか？
もしくはフィールドに宣言しておいた方がよいのでしょうか？

また、起動時の値の読み込みはonCreateメソッドで、終了時の値の書き込みはonDestroyメソッドで
実装するのは適切でしょうか？

また、今回はおそらくスクロール位置だけの単一のデータの読み書きとなると思いますが、
多めの複数データの読み書きとなってくると、SharedPreferencesの宣言方法はどちらが適切になるとか、
変わってきたりするものでしょうか？

試したこと

実際にそれぞれのコードを書いてみて、挙動を確認してみました。
予想に反して、データの書き込みすらできてませんでした
(既に、"string1", "value1"というキーと値のペアで保存していた場合、"string1", "value2"に書き直せないという挙動でした。)

すみません、質問が増えてしまいますが、
上記質問に加えて、今回試したコードでデータの書き換えができないのは何故でしょうか？

ご教示お願いします。

・各メソッドごとで宣言する場合。

public class MainActivity extends AppCompatActivity {

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

        SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(this);
        String s1 = prefs.getString("string1", "");
        Log.d("mytag", s1);
    }

    @Override
    public void onDestroy(){
        super.onDestroy();
        SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(this);
        SharedPreferences.Editor editor = prefs.edit();
        editor.putString("string1", "value1");
        editor.apply();
    }

}

・フィールドで宣言して使い回す場合。

public class MainActivity extends AppCompatActivity {

    SharedPreferences prefs;

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

        prefs = PreferenceManager.getDefaultSharedPreferences(this);
        String s1 = prefs.getString("string1", "");
        Log.d("mytag", s1);
    }

    @Override
    public void onDestroy(){
        super.onDestroy();
        SharedPreferences.Editor editor = prefs.edit();
        editor.putString("string1", "value1");
        editor.apply();
    }

}

Answer 1

多めの複数データの読み書きとなってくると、SharedPreferencesの宣言方法はどちらが適切になるとか、

変わってきたりするものでしょうか？

どちらでも構わないと思います。冗長性を考えると後者の方がいいように思いますが、「prefsに未代入のままメソッドを呼び出してぬるぽ」というチョンボを起こさないように気を配るという程度のことではないかと思います。

(既に、"string1", "value1"というキーと値のペアで保存していた場合、"string1", "value2"に書き直せないという挙動でした。)
上記質問に加えて、今回試したコードでデータの書き換えができないのは何故でしょうか？

これは意味がわかりません。後者のコードで一度実行しActivityを閉じて書き込みを行わせた後、onDestroy()で書き込む値を"value2"に書き換え、再度実行→Activityを閉じる→もう一度実行で"value2"を読み出せます（試すまでもないのですが、実際に試しました）。お示しいただいたコードに処理を追加したものでうまくいっていないものと思いますが、そのコードそのものを見なければ原因はわかりません。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
デバッグ時にonDestroy()が呼ばれていないのが原因でした(デバッグの仕方が悪かったようです)。
アプリ終了時の値保存はonPause()で行うことが推奨されているようでした。
https://sites.google.com/site/androidappdojo/home/devmemo/memo04
https://developer.android.com/guide/components/activities.html?hl=ja