Android Studio:Dialogの表示について

Question

前提

Android StudioでJavaを使ってメモ帳アプリを作っています。作成してメモを削除するときに、本当に削除するか確認するダイアログを出したいのですがjava.lang.IllegalStateExceptionが出て、うまくいきません。

実現したいこと

• ダイアログの実装

発生している問題・エラーメッセージ

E/AndroidRuntime: FATAL EXCEPTION: main
    Process: jp.halo_sf.haloint, PID: 22128
    java.lang.IllegalStateException: FragmentManager has not been attached to a host.
        at androidx.fragment.app.FragmentManager.enqueueAction(FragmentManager.java:1880)
        at androidx.fragment.app.BackStackRecord.commitInternal(BackStackRecord.java:329)
        at androidx.fragment.app.BackStackRecord.commit(BackStackRecord.java:294)
        at androidx.fragment.app.DialogFragment.show(DialogFragment.java:260)
        at jp.halo_sf.haloint.RecyclerViewAdapter.lambda$onBindViewHolder$0(RecyclerViewAdapter.java:42)
        at jp.halo_sf.haloint.RecyclerViewAdapter$$ExternalSyntheticLambda0.onClick(Unknown Source:0)
        at android.view.View.performClick(View.java:7463)
        at android.view.View.performClickInternal(View.java:7440)
        at android.view.View.access$3600(View.java:825)
        at android.view.View$PerformClick.run(View.java:28414)
        at android.os.Handler.handleCallback(Handler.java:938)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loop(Looper.java:223)
        at android.app.ActivityThread.main(ActivityThread.java:8240)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:979)

該当のソースコード

package jp.halo_sf.haloint;

import android.database.Cursor;
import android.os.Bundle;

import androidx.appcompat.app.AppCompatActivity;
import androidx.recyclerview.widget.LinearLayoutManager;
import androidx.recyclerview.widget.RecyclerView;

public class MainActivity extends AppCompatActivity {
    private DatabaseHelper _helper;
    private static RecyclerView rvNote;
    private static RecyclerViewAdapter adapter;

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

        //DatabaseHelperの生成処理。
        _helper = new DatabaseHelper(MainActivity.this);

        //RecyclerViewの設定。
        rvNote = findViewById(R.id.rvNote);
        rvNote.setLayoutManager(new LinearLayoutManager(MainActivity.this));
//DataBaseからデータを取得。
        Cursor cursor = _helper.getData();
        NoteManager noteManager = new NoteManager();
        while (cursor.moveToNext()) {
            noteManager.addNoteItem(String.valueOf(cursor.getInt(0)), cursor.getString(1), cursor.getString(2));
        }
        noteManager.addToMapData();
        cursor.close();
        adapter = new RecyclerViewAdapter(noteManager.getNoteItems());
        rvNote.setAdapter(adapter);
    }

package jp.halo_sf.haloint;

import androidx.fragment.app.FragmentActivity;

public class RecyclerViewAdapter extends RecyclerView.Adapter<RecyclerViewAdapter.RecyclerListViewHolder> {
    @Override
    public void onBindViewHolder(@NonNull RecyclerListViewHolder holder, int position) {
        holder._tvTitle.setText(titles.get(position));
        holder._tvText.setText(texts.get(position));

        holder._iBtnDeleteL.setOnClickListener(view -> {
            DialogFragmentCheckingDelete dialogFragment = new DialogFragmentCheckingDelete();
//エラー文に記載されている箇所。
            dialogFragment.show(new FragmentActivity().getSupportFragmentManager(),
"DialogFragmentCheckingDelete");
        });
    }
}

試したこと

getSupportFragmentManagerをFragmentActivityをインスタンス化して取得しているからだと思い、MainActivityにダイアログを表示するメソッドを記入し、そこで処理を行ってみましたが全く状況が変わらなかったので元に戻しました。
おそらくgetFragmentManagerが関係していると思うのですが…何か分かることがあれば、些細な事でも良いのでお教えいただけますと幸いです。何卒宜しくお願い致します。

補足情報（FW/ツールのバージョンなど）

Android Studio Bumblebee | 2021.1.1 Patch 2
Build #AI-211.7628.21.2111.8193401, built on February 17, 2022
ランタイム・バージョン: 11.0.11+9-b60-7590822 amd64
VM: OpenJDK 64-Bit Server VM by Oracle Corporation
Windows 10 10.0
GC: G1 Young Generation, G1 Old Generation
Memory: 1280M
Cores: 8
Registry: external.system.auto.import.disabled=true
Non-Bundled Plugins: dev.polek.adbwifi (1.2.4), org.jetbrains.kotlin (211-1.6.10-release-923-AS7442.40)

Comments

[hoshi-takanori]

> MainActivityにダイアログを表示するメソッドを記入
でいけると思いますけど。(まさか new MainActivity() してそのメソッドを呼び出したとか？)

[s.s]

そのまさかです。本当に未熟なものですみません。私もそれが問題なのではないかと思い、requireActivity();等のメソッドを試しても見たのですがこのメソッドがどうもRecyclerView.Adapterでは使えないみたいで…もし良ければ、どうすれば良いのかお教え願えませんでしょうか？

[hoshi-takanori]

RecyclerViewAdapter に MainActivity の参照を持たせれば良いかと。
(具体的なコードを知りたければ、MainActivity で RecyclerViewAdapter を生成してるあたりのコードをお示しください。)

[s.s]

遅くなってしまってすみません。今コードを追加したのでご覧くださると幸いです。

Answer 1

new FragmentActivity().getSupportFragmentManager()

先のご質問でも少し書きましたが、ここでも new FragmentActivity() は意味がありません。
根本として、 アクティビティはアプリのコードが new で作っても動作しないモノです。
アクティビティはただ作れば良いものでは無く、 Android OS と結びついて初めて動作する存在です。ですのでアクティビティが必要であれば、既に動作しているアプリのアクティビティを何とか取得する必要があります。
ついでに、フラグメントマネージャはアクティビティ毎(ついでにフラグメント毎にも)にありますので、どのアクティビティ(もしくはフラグメント)でも良いという訳にもいきません。

ご質問のコードの場合は、Adapter 内でクリック時の処理を行うよりも、アクティビティにクリックされたことを伝えて後の処理を任せるほうが Adapter の役割としては適当に思います。

public class MainActivity extends AppCompatActivity {
  @Override
  protected void onCreate(Bundle savedInstanceState) {
//略
    adapter = new RecyclerViewAdapter(noteManager.getNoteItems());
    adapter.setDeleteListener((view, position) -> {
      DialogFragmentCheckingDelete dialogFragment = new DialogFragmentCheckingDelete();
      dialogFragment.show(getSupportFragmentManager(), "DialogFragmentCheckingDelete");
    });
    rvNote.setAdapter(adapter);
//略
  }
}

public class RecyclerViewAdapter extends RecyclerView.Adapter<RecyclerViewAdapter.RecyclerListViewHolder> {
//略
  interface DeleteListener {
    void onDelete(View view, int position);
  }

  private DeleteListener deleteListener;
  void setDeleteListener(DeleteListener deleteListener) {
    this.deleteListener = deleteListener;
  }

  @Override
  public void onBindViewHolder(@NonNull RecyclerListViewHolder holder, int position) {
    holder._tvTitle.setText(titles.get(position));
    holder._tvText.setText(texts.get(position));
    holder._iBtnDeleteL.setTag(position);
  }

  public class RecyclerListViewHolder extends RecyclerView.ViewHolder {
    final TextView _tvTitle;
    final TextView _tvText;
    final Button _iBtnDeleteL;

    public NoteViewHolder(View itemView) {
      super(itemView);
      _tvTitle= itemView.findViewById(R.id.tvNoteTitle);
      _tvText= itemView.findViewById(R.id.tvNoteText);
      _iBtnDeleteL= itemView.findViewById(R.id.dtnDelete);
      _iBtnDeleteL.setOnClickListener(view -> {
        if(deleteListener != null) deleteListener.onDelete(view, (int)_iBtnDeleteL.getTag());
      });
    }
  }
//略

Comments

[s.s]

jimbeさん、またもご回答くださりありがとうございます。お恥ずかしい話、まだアンドロイドアプリにおけるnewというものがいまいちどころか全然理解できていません。
>アクティビティはアプリのコードが new で作っても動作しないモノです。アクティビティはただ作れば良いものでは無く、 Android OS と結びついて初めて動作する存在です。
とのことでしたが、この場合、newはいつ使えるのでしょうか？newを使って良い場面とダメな場面について、よろしければご教授願えませんでしょうか？

[jimbe]

new はオブジェクトを作るものですので、 java 言語としてはいつでも使えます。
ただ、Android として new で作ってもそのままでは動かない特殊(?)なものがあるということで、その1つがアクティビティです。
他にどんなものがあるのかは、それらを列挙した資料というのは見たことがありませんのでお答えし兼ねます。
目安的には、アクティビティやコンテキストから get～ で取得するような (フラグメントマネージャとか) は大抵 new で作っても動かないと思いますが、逆の現象として例外的に動くもの(コンストラクタのパラメータにアクティビティやコンテキストを指定するものとか)もあるかもしれません。
何かのクラスやメソッドを初めて使おうという場合は、ドキュメントやサンプルをよく読んで、new するのか何かから get するのかを確認しておくと良いでしょう。

[s.s]

ご返答本当にありがとうございます。自分なりに考えてみたのであっているか確かめていただけると幸いです。
今回の場合でいうと、getSupportFragmentManager();を、ダイアログを表示するアクティビティ上で行わないと、正しいFragmentManagerが取得できないため、ダイアログがうます表示できないということであっていますか？
また、「試したこと」に書いた
>getSupportFragmentManagerをFragmentActivityをインスタンス化して取得しているからだと思い、MainActivityにダイアログを表示するメソッドを記入し、そこで処理を行ってみましたが全く状況が変わらなかった
というのはメソッドを呼び出すときにnew MainActivity();をしてしまっているため、今表示しているMainActivity（MainActivity1とする）とは別のMainActivityが作られてしまいそこで処理を行っても、MainActivity1には反映されないため、表示ができないということであっていますか？

[s.s]

何度も質問してすみません。これで大丈夫だと思ったのですが、実はダイアログにRecyclerViewのpositionを引き渡したいのです。positionがonBindHolder内なら取得できるのですが、MainActivityからはおそらく取得できません。なので、onBindHolderのフィールドにpositionを保存してgetter経由で取得しようと思ったのですが、setonClickListener();の中でpositionを取得しなければいけません。しかし、このメソッドのなかにはすでにdeleteLitenerが入っているのでどうすれば良いのか、さっぱり分かりません。どうかお教え願いませんでしょうか？

[jimbe]

> 今回の場合でいうと、getSupportFragmentManager();を、ダイアログを表示するアクティビティ上で行わないと、正しいFragmentManagerが取得できないため、ダイアログがうまく表示できないということであっていますか？

合っています。

> new MainActivity();をしてしまっているため、今表示しているMainActivity（MainActivity1とする）とは別のMainActivityが作られてしまいそこで処理を行っても、MainActivity1には反映されないため、表示ができないということであっていますか？

ほぼ合っています。
より正確に言うならば、new MainActivity() として作成したオブジェクトは回答に書きましたように"アクティビティとしての意味は無い"モノですので、それを用いて何かをしようとしても大方は(ご経験されているようにどこかで例外が発生するなどして)処理が出来ないと思います。
たまたま例外が発生しないような処理をしたとして「表示ができない(されない)」という結果になります。

[jimbe]

> 実はダイアログにRecyclerViewのpositionを引き渡したいのです。

回答のコードでは DeleteListener として OnCLickListener を流用(?) していますが、専用リスナを作ってパラメータとして渡すのがスマートに思います。
回答を編集します。

[s.s]

返信ありがとうございます！インスタンス化についてだいぶ理解を深められました。jimbeさんのおかげです。本当に申し訳ないのですが、上の質問についてもご回答いただけると幸いです。

[s.s]

回答編集、本当にありがとうございました。今回は、この質問に丁寧にご回答くださり、根本的な原因を教えてくださったjinbeさんをベストアンサーとさせていただきました。コメントしてくださったhoshi-takanoriさん、この質問を呼んでくださった多くの方々にも感謝を述べたいと思います。本当にありがとうございました。また分からないことがあれば質問させていただくと思うので、これからも何卒宜しくお願い致します。

[jimbe]

ViewHolder を前のご質問から持ってきましたので view の id とかイロイロ違うかもしれませんが、必要なのは コンストラクタでの setOnClickListener の部分だけです。
DeleteListener.onDelete に View パラメータを入れていますが、使う予定が無ければ(恐らく使うことは無いと思いますが^^;) potision だけにしても大丈夫でしょう。

[s.s]

ありがとうございます！