チェックboxの有り/無しでテキストとボタンの有効/無効を操作しチェック状態も保存したい。

Question

簡単なスケジュールメモを作成しており
チェックboxの有り/無しでテキストとボタンの有効/無効を操作しチェック状態も保存したいのですが
コードは完成させたのですがうまい事動きません。。。。
詳細仕様は以下です。
･)textedit2つに予定内容、予定日時を入力し保存ボタンで入力状態を保存する。
※checkboxのcheckが無しだとtexteditへの入力/保存ボタン・クリアボタンの操作は無効。
※checkboxにcheckが入っていればtexteditへの入力/保存ボタン・クリアボタンの操作は有効。
･)保存ボタンでtexteditの内容/check状態を保存し、次回アプリ起動時には読みだして表示する。
エラーは出ていないのでコードの内容におかしい箇所があると思うのですが、どう修正すればいいか悩んでおります。。。。宜しければアドバイス頂きたいです。

final SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(this);
final boolean checked = prefs.getBoolean("chkbox01",false);
final String schetext = prefs.getString("schetext", "");
final String schetime = prefs.getString("schetime", "");

final CheckBox chkbox01 = (CheckBox)findViewById(R.id.checkbox01);
chkbox01.setChecked(checked);

//読み込んだ値に応じて文字列の初期状態をセット
if (!checked){
   chkbox01.setText("予定:No.01を入力する場合はチェックして下さい。");
   EditText scheedit01 = findViewById(R.id.schedule_text01);
   scheedit01.setEnabled(false);  // チェック無しのケースは入力禁止
   EditText schetime01 = findViewById(R.id.schedule_time01);
   schetime01.setEnabled(false);//チェック無しの場合は予定時間も入力禁止
   Button button1 = (Button)findViewById(R.id.button1);
   button1.setEnabled(false);//チェック無しの場合はボタンも使用できない
   Button button2 = (Button)findViewById(R.id.button2);
   button2.setEnabled(false);//チェック無しの場合はボタンも使用できない

}else {
   chkbox01.setText("予定:No.01が入力されています。");
   TextView sche01 = findViewById(R.id.schedule01);
   sche01.setText("Schedule No.01");
   EditText scheedit01 = findViewById(R.id.schedule_text01);
   scheedit01.setText(schetext);
   EditText schetime01 = findViewById(R.id.schedule_time01);
   schetime01.setText(schetime);
}

chkbox01.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View view) {
        //チェック状態を取得する
        final boolean checked = chkbox01.isChecked();
        final EditText scheedit01 = findViewById(R.id.schedule_text01);
        scheedit01.setEnabled(true);  // チェックが有りなら入力を許可する
        scheedit01.setText(schetext);
        final EditText schetime01 = findViewById(R.id.schedule_time01);
        schetime01.setEnabled(true);//チェックが有りなら入力を許可する。
        schetime01.setText(schetime);
        Button button1 = (Button)findViewById(R.id.button1);
        button1.setEnabled(true);//チェック有ならボタン使用を許可
        Button button2 = (Button)findViewById(R.id.button2);
        button2.setEnabled(true);//チェック有ならボタン使用を許可

        //予定クリアボタンで入力内容をクリアする
        button2.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
               scheedit01.getEditableText().clear();
               schetime01.getEditableText().clear();
            }
         });

         button1.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View view) {
                //予定内容/予定日時のテキストを取得
                String savetext = scheedit01.getText().toString();
                String savetime = schetime01.getText().toString();
                //SharedPreferencesに書き込む
                SharedPreferences.Editor editor = prefs.edit();
                editor.putBoolean("chkbox01", checked);
                editor.putString("schetext", savetext);
                editor.putString("schetime", savetime);
                //editor.commit()
                editor.apply();
              }
         });

         if(!checked) {
             chkbox01.setText("予定:No.01を入力する場合はチェックして下さい。");
         }
         else{
             chkbox01.setText("予定:No.01が入力されています。");
         }

     }
});

Comments

[keicha_hrs]

前の質問でも指摘されているはずですが、コードの範囲にはMarkdownを用いてください。

[rockey_7625]

先ほど修正を行いました。
コード貼り付けになれておらず、申し訳ありません。
投稿の際にコードの挿入の機能に気付いていませんでした。。
機能を使えばMarkdown記法で貼り付け可能なようですので今後機能を使用して貼り付けるように致します。※Markdown記法に関して初心者で申し訳ありません。

[keicha_hrs]

それと、質問内容からは「うまくいっていないこと」が何なのかよくわかりません。「チェック状態も保存したい」というのがうまくいっていないことですか？

[rockey_7625]

うまくいっていない事は以下です。
①チェック状態の保存は想定通りの動きですが、texteditの書き込みを行い保存ボタンで保存しアプリ終了→アプリを再度起動した時にクリアボタンが機能しない。【チェック有りなのにクリアボタンが効かない。。。】
②再度アプリを起動した時にチェックをはずしても保存/クリアボタンが有効になってしまう。
【チェック無しでは無効にしたつもりなのですが。。。】

[keicha_hrs]

「アプリを再起動」というのは、例えばホームボタンを押してランチャー画面に戻った後に、ランチャーからアプリアイコンを叩く操作を含んでいますか？その場合、onCreate()は実行されないので、別の対応をする必要があります。

[rockey_7625]

はい、一旦アプリを終了させてホーム画面に戻り再度アプリアイコンを叩いて起動させますので含んでいます。現状のコードでは不十分という事でしょうか？

[keicha_hrs]

チェックボックスの状態を保存したいのは、チェックをON/OFFしたときですか？それともbutton1を押したときのみですか？

[rockey_7625]

はい、チェックboxの状態保存はbutton1を押したときのみでよいです。

Answer 1

findViewById()を何度も繰り返しているところをまとめたり、同じ処理をするところを関数化したりしてまとめてみましたが、これで想定通りの動きになっているでしょうか？違うところがあればまた調整します。

public class MainActivity extends AppCompatActivity {
    private CheckBox chkbox01;
    private EditText scheedit01;
    private EditText schetime01;
    private Button button1;
    private Button button2;
    private TextView sche01;

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

        final SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(this);
        final boolean checked = prefs.getBoolean("chkbox01", false);
        final String schetext = prefs.getString("schetext", "");
        final String schetime = prefs.getString("schetime", "");

        chkbox01 = findViewById(R.id.checkbox01);
        scheedit01 = findViewById(R.id.schedule_text01);
        schetime01 = findViewById(R.id.schedule_time01);
        button1 = findViewById(R.id.button1);
        button2 = findViewById(R.id.button2);
        sche01 = findViewById(R.id.schedule01);

        chkbox01.setChecked(checked);

        viewEnable(checked);

        //読み込んだ値に応じて文字列の初期状態をセット
        if (checked) {
            sche01.setText("Schedule No.01");
            scheedit01.setText(schetext);
            schetime01.setText(schetime);
        }

        chkbox01.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                //チェック状態を取得する
                boolean checked = chkbox01.isChecked();
                viewEnable(checked);
            }
        });

        button1.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                //予定内容/予定日時のテキストを取得
                boolean checked = chkbox01.isChecked();
                String savetext = scheedit01.getText().toString();
                String savetime = schetime01.getText().toString();
                //SharedPreferencesに書き込む
                SharedPreferences.Editor editor = prefs.edit();
                editor.putBoolean("chkbox01", checked);
                editor.putString("schetext", savetext);
                editor.putString("schetime", savetime);
                //editor.commit()
                editor.apply();
            }
        });

        //予定クリアボタンで入力内容をクリアする
        button2.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                scheedit01.getEditableText().clear();
                schetime01.getEditableText().clear();
            }
        });
    }

    void viewEnable(boolean checked) {
        if (!checked) {
            chkbox01.setText("予定:No.01を入力する場合はチェックして下さい。");
            scheedit01.setEnabled(false);  // チェック無しのケースは入力禁止
            schetime01.setEnabled(false);//チェック無しの場合は予定時間も入力禁止
            button1.setEnabled(false);//チェック無しの場合はボタンも使用できない
            button2.setEnabled(false);//チェック無しの場合はボタンも使用できない
        } else {
            chkbox01.setText("予定:No.01が入力されています。");
            scheedit01.setEnabled(true);  // チェックが有りなら入力を許可する
            schetime01.setEnabled(true);//チェックが有りなら入力を許可する。
            button1.setEnabled(true);//チェック有ならボタン使用を許可
            button2.setEnabled(true);//チェック有ならボタン使用を許可
        }
    }
}

Comments

[rockey_7625]

先ほど確認しました。
ありがとうございます！この動きがほしかったです！
それと自分が書いたコードはかなり無駄が多かったようですね。。。【javaをもっと勉強します。】
また、申し訳ないのですがもう1点質問よろしいでしょか？
今回のコードがおおまか1件分のプログラムになるので、これをもう数件分【5件ぐらい】実装する予定です。
なのでこの一連のコードを配列化して5件分入力/表示したいのですがこれは可能なのでしょうか？

[keicha_hrs]

今回作った画面を2画面目として、1画面目として5件分をリストアップする画面を作ればいいのではないでしょうか。1画面目で選択した番号をインテントで2画面目に送り、2画面目ではそれを受け取ってそれに基づいてSharedPreferencesのキー名を決めたり、表示する文字列の文言を変えたりする工夫を盛り込めばいいと思います。

[rockey_7625]

なるほど！
そういう工夫が必要なのですね。
アドバイス誠にありがとうございました！