fragment_num=1のフラグメント表示画面で、ダイアログを表示するとfragment_num=0となってしまう

Question

●実現したいこと
fragment_numを正しく取得したい。
fragment_num=1のフラグメント表示画面で保存ダイアログを出してボタンを押す（DialogFragmentKyoutuu.javaに入る（関係する））とfragment_numが0になってしまうため、ゲーム保存がfragment_num=1でできないです。
以前は変数の多くをstaticで行っていて、その時はfragment_num=1のまま保存できていて、保存取り出しをすると、ちゃんとその画面を出すことができていましたが、できなくなってしまいました。ここのところ行ったことは、viewmodelを使用し、ここに多くの変数の出し入れをするように変更したことです。

●試したこと
プログラムの各所にfragment_numを表示するトーストを書き入れ、どこからfragment_numが０になるかを探しました。
・フラグメント画面表示onViewCreated・・・fragment_num=1
・フラグメント画面表示状態で、 ダイアログ表示のためのボタンタップをしダイアログを表示・・・fragment_num=1
・ダイアログのボタン４つ「ゲームに戻る」「保存しないで終了」「保存して終了」「保存してゲームに戻る」のどれをタップしても・・・fragment_num=0

なので、DialogFragmentKyoutuu.javaに入る（関係する）とfragment_num=0になってしまうと考えています。
お助けください。

MainViewModel.javaの中のfragment_num

private  int fragment_num;
int getFragment_num(){return  fragment_num;}
void  setFragment_num(int fragment_num){this.fragment_num=fragment_num;}

フラグメントのonCreate　

public class S1UranaiFragment extends Fragment implements Runnable,DialogFragmentKyoutuu.ResultReceiver {
     
    //多いので省略

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        return inflater.inflate(R.layout.s1_fragment_uranai, container, false);
    }

    @Override
    public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);

        //「ほぞんしたところ」から来たなら
        flag_from_hozon=mainViewModel.getFlag_from_hozon();
        if(flag_from_hozon) {
            count_fr = mainViewModel.getCount_fr();
            flag_from_hozon = false;
            mainViewModel.setFlag_from_hozon(flag_from_hozon);
        }else {//stage1から来たなら
                  count_fr=0;
        }       
        fragment_num=1;
        mainViewModel.setFragment_num(fragment_num);
        aitemu_fr = (TextView) view.findViewById(R.id.aitemu_fr);
        TitleActivity.point_set();        
        thread=null;
       
        //省略

        start();

        ObjectAnimator animator_toumei = ObjectAnimator.ofFloat(uranaisi, "alpha", 0f, 0f).setDuration(3000);      
        batu_fr=view.findViewById(R.id.batu_fr);
        batu_fr.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                if(flag_batu_fr) {//保存ダイアログ表示                    
                    flag_count_fr=false;
                    DialogFragment dialog = new DialogFragmentKyoutuu();
                    dialog_num_kyoutuu = 1;
                    mainViewModel.setDialog_num_kyoutuu(dialog_num_kyoutuu);
                    dialog.show(getParentFragmentManager(), "dialog_kyoutuu"+dialog_num_kyoutuu);
                }
            }
        });

DialogFragmentKyoutuu.java

public class DialogFragmentKyoutuu extends DialogFragment {
   //多いので省略
    LayoutInflater inflater;
    View view;
    static boolean hozon_on
    Context context;
    SharedPreferences sharedPreferences;

    public Dialog onCreateDialog(Bundle savedInstanceState){
        AlertDialog.Builder builder = new AlertDialog.Builder(getActivity());
        sharedPreferences = context.getSharedPreferences("Data", Context.MODE_PRIVATE);

        dialog_num_kyoutuu=mainViewModel.getDialog_num_kyoutuu();       
        flag_count=false;
        mainViewModel.setFlag_count(flag_count);
        ResultReceiver resultReceiver = (ResultReceiver) getActivity();
        resultReceiver.onReturnValue(0,flag_count,0,0);//flagnum(0:flag_count(fragmentではflag_count_fr) 1:flag_from_dialog 2:)
        flag_count_fr=false;
        mainViewModel.setFlag_count_fr(flag_count_fr);
        switch(dialog_num_kyoutuu){
            case 0://省略
                break;
            case 1:inflater=LayoutInflater.from(getActivity());//保存画面ダイアログ
                view=inflater.inflate(R.layout.dialog_custom_kyoutuu_put_hozon,null);
                TextView hozon_tv=view.findViewById(R.id.hozon_tv);
                hozon_tv.setText(hozon_messege);
                builder.setView(view);
                Button hozon_bt_cansel=view.findViewById(R.id.hozon_bt_cansel);
                hozon_bt_cansel.setOnClickListener(new View.OnClickListener() {//ゲームに戻るボタン
                    @Override
                    public void onClick(View v) {
                        set_flag_count_true();//アクティビティのflag_count,フラグメントのflag_count_frをtrueにする
                        dismiss();                      
                    }
                });
                Button hozon_bt_syuuryou=view.findViewById(R.id.hozon_bt_syuuryou);
                hozon_bt_syuuryou.setOnClickListener(new View.OnClickListener() {//保存しないで終了ボタン
                    @Override
                    public void onClick(View v) {
                        hozon_on=false;                      
                        dialog_num_kyoutuu=2;
                        mainViewModel.setDialog_num_kyoutuu(dialog_num_kyoutuu);
                        DialogFragment dialog = new DialogFragmentKyoutuu();//終了していいか確認画面ダイアログ
                        dialog.show(getParentFragmentManager(), "dialog_kyoutuu"+dialog_num_kyoutuu);
                    }
                });
                Button hozon_bt_hozongosyuuryou=view.findViewById(R.id.hozon_bt_hozonngosyuuryou);
                hozon_bt_hozongosyuuryou.setOnClickListener(new View.OnClickListener() {//保存して終了ボタン
                    @Override
                    public void onClick(View v) {
                        put_hozon();//保存                        
                        hozon_on=true;
                        dialog_num_kyoutuu=2;
                        mainViewModel.setDialog_num_kyoutuu(dialog_num_kyoutuu);
                        DialogFragment dialog = new DialogFragmentKyoutuu();//終了していいか確認画面
                        dialog.show(getParentFragmentManager(), "dialog_kyoutuu"+dialog_num_kyoutuu);
                    }
                });
                Button hozon_bt_hozongomodoru=view.findViewById(R.id.hozon_bt_hozonngomodoru);
                hozon_bt_hozongomodoru.setOnClickListener(new View.OnClickListener() {//保存してゲームに戻るボタン
                    @Override
                    public void onClick(View v) {
                        put_hozon();//まず保存
                        hozon_on=true;               
                        set_flag_count_true();//アクティビティのflag_count,flag_count_frをtrueにする                        dismiss();
                    }
                });
                if(getActivity().getLocalClassName().equals("MainActivity")){//MainActivityでは2つのボタンを非表示
                    hozon_bt_hozongomodoru.setVisibility(View.INVISIBLE);
                    hozon_bt_hozongosyuuryou.setVisibility(View.INVISIBLE);
                }
                break;
            case 2://保存ありなしの終了確認ダイアログ                
                break;
            case 3://開始保存取り出しダイアログ
                break;
            default:
                throw new IllegalStateException("Unexpected value: " + dialog_num_kyoutuu);
        }
        AlertDialog dialog = builder.create();
        dialog.setCanceledOnTouchOutside(false);//ダイアログ以外の画面をタップしてもダイアログは消えない
        return dialog;
    }
    void change_activity(){
        stage_num=mainViewModel.getStage_num();
        fragment_num=mainViewModel.getFragment_num();
        flag_to_stage=mainViewModel.getFlag_to_stage();
        Intent intent = new Intent(getActivity(), ChangeActivity.class);        
        mainViewModel.setFlag_game_finish(flag_game_finish);        
        mainViewModel.setStage_num(stage_num);
        startActivity(intent);
        getActivity().finish();
    }
    //アクティビティ、フラグメント両用
    void put_hozon(){
        sharedPreferences = context.getSharedPreferences("Data", Context.MODE_PRIVATE);
        SharedPreferences.Editor editor = sharedPreferences.edit();
        stage_num=mainViewModel.getStage_num();
        switch (stage_num){
            case 1://Stage1
                Stage1 stage1=(Stage1)getActivity();
                stage1.stage1_set_originalData();//originalDataをviewmodelに入れるメソッド

                editor.putInt("Count",mainViewModel.getCount());
                editor.putBoolean("Flag_count",flag_count);
                editor.putInt("Step",mainViewModel.getStep());
                //フラグメントのみの値
                editor.putInt("Count_fr", mainViewModel.getCount_fr());
                fragment_num=mainViewModel.getFragment_num();
                switch (fragment_num) {
                    case 0:break;//フラグメントなし
                    case 1: //uranainoyakataのみの値
                        editor.putBoolean("Flag_uranau",mainViewModel.getFlag_uranau());
                        break;
                    case 2:
                }
            case  2://Stage2
                break;            
        }
        //共通のデータをviewmodelからsharedpreferencesに保存       
        editor.putString("Aitemu", mainViewModel.getAitemu());        
        editor.putInt("Stage_num",mainViewModel.getStage_num());
        editor.putInt("Fragment_num",mainViewModel.getFragment_num());
        editor.apply();//SharedPreferencesに保存
    }
    void get_hozon(){
    //省略
    }
    @Override
    public void onAttach(Context context) {//アクティビティと関連づいたとき
        super.onAttach(context);
        this.context = context;
    }

    public interface ResultReceiver {
        void onReturnValue(int flagnum,Boolean flag,int num0,int num1);
    }    
}

Comments

[PlugOut777]

省略された中にあるのだと思いますが、S1UranaiFragmentとDialogFragmentKyoutuu でmainViewModelはどのように定義されていますか？

> 以前は変数の多くをstaticで行っていて、その時はfragment_num=1のまま保存できていて、保存取り出しをすると、ちゃんとその画面を出すことができていましたが、できなくなってしまいました。
これが原因の可能性が高いです。

[aruko]

PlugOut777さん、ありがとうございます。
mainViewModelは、TitleActivityというアクティビティで下のように宣言し、S1UranaiFragmentとDialogFragmentKyoutuuではそれをインポートして使っています。

static MainVieModel mainVieModel;
mainViewModel=new ViewModelProvider(this).get(MainViewModel.class);

今の状態ではfragment_numだけがうまくいかず、stage_numや他の変数はすべて正しく取得できているのです。
なので、一番疑ったのは、MainVieModelの中で、fragment_numだけ二重に設定していないか、また書き込みミスをしていないかでしたが、そうなっていません。
アドバイスしていただければありがたいです。

[aruko]

static MainViewModel mainViewModelです。スペルミスですみません。

[PlugOut777]

> 今の状態ではfragment_numだけがうまくいかず、stage_numや他の変数はすべて正しく取得できているのです。
少し気になったのですが、fragment_numはDialogFragmentKyoutuuの中にフィールドとして定義されているのですかね？
また、値が0になってしまうのはput_hozon()の処理の中でしょうか？

[aruko]

fragment_numは、どのアクティビティ、フラグメントでも保存関係で使いますが、それぞれの場所で、int fragment_num;で宣言しています。
ですので、使用するときにfragment_num=mainViewModel.getFragment();をしてから使用しているつもりです。
DialigFragmentKyoutuuのどこかで、初期値のfragment_numのまま使えば0になるのは当然なので、どこかでそのようにしていないかも探したつもりです。
put_hozon()は、保存ダイアログの「保存して終了」「保存してゲームに戻る」には関係していますが、「ゲームに戻る」「保存しないで終了」には関係していません。
「ゲームに戻る」のボタンタップでもfragment_num=0になってしまうので、put_hozonが原因ではないように思います。
原因探しの質問ですみません。
アドバイスいただければ嬉しいです。

[jimbe]

直接は関係無いと思いますが、

> 以前は変数の多くをstaticで行っていて、(中略) ここのところ行ったことは、viewmodelを使用し、ここに多くの変数の出し入れをするように変更したことです。

> static MainViewModel mainViewModel
> mainViewModel=new ViewModelProvider(this).get(MainViewModel.class);
> それをインポートして使っています。

static 変数を ViewModel に入れても、 ViewModel を static にしてしまっては意味がありません。
ViewModel は ViewModelProvider によって管理されることが前提ですので、必要に応じて ViewModelProvider から取得して使用するべきと思います。
( getActivity が動作すれば mainViewModel=new ViewModelProvider(getActivity()).get(MainViewModel.class) で取り出せるはずです。 )

[jimbe]

> fragment_num=1のフラグメント表示画面

というのが S1UranaiFragment クラスのことでしょうか。そして

fragment_num=1;
mainViewModel.setFragment_num(fragment_num);

でビューモデルの fragment_num を 1 にしていると。
(なぜわざわざ 一度変数に入れているのか分かりませんが。)

> 保存ダイアログを出してボタンを押す

は(コードと逆のようで) batu_fr がボタンで、押すと

dialog_num_kyoutuu = 1;
mainViewModel.setDialog_num_kyoutuu(dialog_num_kyoutuu);

で mainViewModel.dialog_num_kyoutuu = 1 になって DialogFragmentKyoutuu の onCreateDialog で

dialog_num_kyoutuu=mainViewModel.getDialog_num_kyoutuu();

から switch(dialog_num_kyoutuu) で case 1 (//保存画面ダイアログ) に入り

> ダイアログのボタン４つ「ゲームに戻る」「保存しないで終了」「保存して終了」「保存してゲームに戻る」

の 4 つのボタンの設定があると・・・。

(なぜこのように複雑になってしまっているのでしょう。
dialog_num_kyoutuu は DialogFragmentKyoutuu のパラメータ (Fragment なのでコンストラクタは使えませんが ) にすれば良いだけでは。)

> ダイアログのボタン４つ「ゲームに戻る」「保存しないで終了」「保存して終了」「保存してゲームに戻る」のどれをタップしても・・・fragment_num=0

というのは、ご提示のコードのどの部分でトースト表示したでしょうか。

[jimbe]

MainViewModel の setFragment_num メソッドで、パラメータに 0 が指定されたら

Log.i("MainViewModel", "fragment_num is ZERO", new Throwable());

を実行するようにしてみるのは如何でしょうか。
このログでスタックトレースが出ますので、どこから 0 に設定するように呼ばれているかが分かると思います。

[aruko]

jimbeさん、ViewModelについての使い方を教えてくださり、ありがとうございます。
変えてみます。
PlugOut777さん、「ゲームに戻る」ボタンの、set_flag_count_trueメソッドの内容を載せます。（コードが長すぎて投稿できなかったので省略しました）
void set_flag_count_true(){
flag_from_dialog_hozon=true;
mainViewModel.setFlag_from_dialog_hozon(flag_from_dialog_hozon);
fragment_num=mainViewModel.getFragment_num();
if(fragment_num==0) {
flag_count = true;
ResultReceiver resultReceiver = (ResultReceiver) getActivity();
resultReceiver.onReturnValue(0,flag_count,0,0);//flagnum(0:flag_count(fragmentではflag_count_fr) 1:flag_from_dialog 2:)
}//フラグメントのflag_count_fr=trueの設定をここでしたい
else {
flag_count_fr = true;
mainViewModel.setFlag_count_fr(flag_count_fr);
}
}

[aruko]

jimbeさん、ありがとうございます。
まず、MainViewModel mainViewModel;のstaticをはずしたいのですが、宣言してあるTitleActivityに、point_set(),point_get(int,int,int)という２つのstaticメソッドがあり、その中で、mainViewModel.getFragment_num()を使っているので、staticをはずせないということでした。
これは、パラメータにfragment_numを入れればできそうなので、やってみます。

また、
>dialog_num_kyoutuu は DialogFragmentKyoutuu のパラメータ (Fragment なのでコンストラクタは使えませんが ) にすれば良いだけでは。)

DialogFragment dialog = new DialogFragmentKyoutuu();の()のところにパラメータが入るように設定すればよいということでしょうか。

>Log.i("MainViewModel", "fragment_num is ZERO", new Throwable());

やってみます。

・フラグメント表示画面・・・S1UranaiFragment
　・「保存ダイアログ表示ボタン」・・・batu_fr
・すべてのアクティビティ、フラグメントから使用するダイアログがまとめてある・・・DialogFragmentKyoutuu
　・「ゲームに戻るボタン」・・・hozon_bt_cansel

[PlugOut777]

うーん、個人的には各ボタンのsetOnClickListener()の中でnew DialogFragmentKyoutuu()しているあたりが怪しいような気はしますが……。

見落とし確認をするための情報が足りないような気もするので
もし可能であれば以下の情報も補完して頂いた方が、他の方からもっとヒントを貰えるかもしれません。
- mainViewModelの変数の定義
- fragment_numの変数の定義
- トーストを仕込んだ箇所

[jimbe]

> 宣言してあるTitleActivityに、point_set(),point_get(int,int,int)という２つのstaticメソッドがあり、その中で、mainViewModel.getFragment_num()を使っているので、staticをはずせないということでした。
> これは、パラメータにfragment_numを入れればできそうなので、やってみます。

以前変数につきましては static の是非のお尋ねがあったと思いますが、メソッドの static もこのように影響が波及してしまうと面倒になりますので、 point_set(), point_get(int,int,int) の static を外すというのも機会(?)が有りましたらお考えになっては如何でしょう。

> DialogFragment dialog = new DialogFragmentKyoutuu();の()のところにパラメータが入るように設定すればよいということでしょうか。

フラグメントはコンストラクタに直接パラメータを書いても、再生成時はパラメータの無いコンストラクタが使用されて誤動作の原因になりますので、コンストラクタでパラメータを指定するのはしないほうが良いです。
フラグメントの生成時にパラメータを使用するには、専用のメソッドが有ります。
[Fragmentで、パラメータ付きの画面遷移を実装する]
https://tech.mokelab.com/android/Fragment/argument.html

[aruko]

ネットが繋がらない、パソコン買い換えず等々で、遅くなり申し訳ありません。

1 MainViewModelをstaticにしないという件
　staticをなしにしてみましたが、それぞれのアクティビティ、フラグメントでnewされるためか、値が受け継がれずエラーが出ました。
最初は各アクティビティで宣言していましたが、なぜstaticに変えたのかの理由を忘れていましたが、そうでした。なので今は、とりあえずstaticに戻しましたが、またやり方を考えてみます。

2 DialogFragmentKyoutuuにパラメータをつける件
パラメータのあるインスタンスを作り、うまくいきました。
が、誤動作の原因になる、と教えてくださったのでこれも考えます。

3 mainViewModelを不必要に使い、複雑にしている件
　viewmodelを初めて使ったので、心配でその都度その都度set,getを繰り返していたので、すべて見直し、現アクティビティ（フラグメント）から出るとき以外はset,getを行わないこと、パラメータで値が引き継げるものはパラメータに変えることを行ったら、fragment_numも正しく取得できるようになりました。
fragment_num=0になってしまう原因は、多分ですが、提示したコード以外で省略した部分にありました。

買い替え前のパソコンで正しく保存、保存取り出しができている状態になりました。
PlugOut777さん、jimbeさん、一緒に考えて下さり本当にありがとうございました。
ところが、また別の問題が発生したので、新たに質問を立てたいと思います。

[jimbe]

> ネットが繋がらない、パソコン買い換え

リアルなイベントが重なってしまったようで、お疲れ様です(x_x

> fragment_numも正しく取得できるようになりました。
> fragment_num=0になってしまう原因は、多分ですが、提示したコード以外で省略した部分にありました。

変数を static やフィールド等にして"どこからでも使える"ようにした場合に一番問題になるのがまさにこのようなことです。
(人が一度に把握できる・していられる範囲は案外狭いのです。)

過去のシステム(を作るプログラマ)はそれによって大変苦労した為に、逆に変数が使える範囲(="スコープ")を極力小さくして不用意に(知らないうちにとか、過去に作って忘れたとかで)変更できないようにし、"変数の変化を想定内に収める"ことが出来るようにする手法を考えました。
それが「カプセル化」等と呼ばれ、"{" "}" の中で宣言した変数はその外に出たら無くなる(=その前後で変数がどうだったかを気にする必要が無くなる)レベルから、一連の関係データとその操作メソッドを1クラスに纏めて外部からは直接データを変更出来なくする(データの整合性をそのクラスだけで取れるようになる)、特定の情報を表示するビューのセットはフラグメント化して表示の整合性を取る等、さまざまなレベルで内外を分けるようになりました。

"変数の値を使いたい時にわざわざ get～ とか set ～ とか面倒なことしないでも static とか付ければいつでも使えるから便利" と思われがちですが、それは "包丁を使いたい時にいつでも使えるように机の上に出して置く" ような、もしくはそれ以上にプログラマ的には「実は危ない」行為となっています。
扱うデータをグループ化し、そのグループ単位でどのような操作があってどう変化し、いつ必要で不要になるかを見極めてカプセル化する必要があります。

[aruko]

こういう話をしてくださること、私がもっている本には書いてないことなので、ほんとに有難いです。
まだ全体の15%くらいの進捗状況ですが、基本的な部分をきちっと固めれば、あとはそれと同じように作ればいいので、今行っていることをしっかり考えてきちんと作っていきたいと思います。
ありがとうございました。
感謝です。^_^

Answer 1

PlugOutさん、jimbeさんのアドバイスにより解決しました。

Comments

[jimbe]

1 つのアクティビティ上で4つの画面(フラグメント)を表示して遷移しつつダイアログを表示して保存や終了(確認有)をするようなサンプルを作ってみました。
https://github.com/Jimbe-github/teratail.q_32pfw6d4u5wg8f

[aruko]

本当に申し訳ありません。
そして、とても嬉しいです。
最新android sutudioで動作できるようになり、まだ残して使っているstatic変数、メソッドを無くせるものは無くすよう、これから取り掛かろうと思ったところです。
パラメータのあるインスタンスをなくしたら、できるようになりました。
しかし、なんといってもお手本がないため、自分なりのコードです。
今後また根本的な部分でつまづくところが出てくるかもしれないと思いつつ・・なので、お手本として使わせてもらいます。
ありがとうございます。

お手本に