RecyclerView へクエリの実行結果を引数として渡したい

Question

実現したいこと

フラグメントの中にRecyclerViewを入れています。
そしてRecyclerViewとしてデータベースからとってきているレコードを
繰り返し表示したいです。

ただ、「←ここが上手くいかない」のところが上手くいきません。
（DailyCalendarFragment.javaの56行目）
ライフサイクルメソッドで、順序を付けているつもりなんですが、
Timely_RecyclerView_Adapterの実行　→　クエリの実行
となってしまい、Timely_RecyclerView_Adapterへクエリの実行結果
を渡したいのにクエリがされていないため null で渡されてしまします。
（this.allTimeTable が null になる）

何か解決策ございましたら、ご教授よろしくお願いいたします。

申し訳ありませんが、もし情報が不足しているようでしたら
ご指摘いただけますと幸いです。
どうぞよろしくお願いいたします。

該当のソースコード

package com.example.RecordTime;

～省略～

public class DailyCalendarFragment extends Fragment {

    public static List<TimeTable_RoomEntity> allTimeTable;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        loadDB();
        Log.d("a", "a");
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        Log.d("b", "b");

        // Inflate the layout for this fragment
        return inflater.inflate(R.layout.daily_calendar_fragment, container, false);
    }


    @Override
    public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
        Log.d("c", "c");

        // RecyclerView を生成して各日にちを生成
        RecyclerView recyclerView = getActivity().findViewById(R.id.time_recycler_view_container);
        // ①recycleView に layoutManager をセット
        recyclerView.setLayoutManager(new LinearLayoutManager(getContext()));
        // ②Adapter を生成して RecyclerView にセット
        recyclerView.setAdapter(new Timely_RecyclerView_Adapter(getContext(), this.allTimeTable)); //←ここが上手くいかない
    }


    public void loadDB() {
        ExecutorService executor = Executors.newSingleThreadExecutor();

        executor.submit(new Runnable() {
            @Override
            public void run() {
                Database database = Room.databaseBuilder(getContext(), Database.class, "TimeTable").build();

                TimeTable_Dao dao = database.timeTable_dao();
                dao.insert(new TimeTable_RoomEntity("0番目のtitle"));
                dao.insert(new TimeTable_RoomEntity("1番目のtitle"));
                dao.insert(new TimeTable_RoomEntity("2番目のtitle"));

                List<TimeTable_RoomEntity> allTimeTable = dao.getAll();

                if(allTimeTable.isEmpty()) {
                    Log.d("allTimeTable========>", "empty == true");
                } else {
                    Log.d("aaaaaaaaaaaa========>", allTimeTable.get(0).title);
                }

                DailyCalendarFragment.allTimeTable = allTimeTable;
            }
        });
    }
}

package com.example.RecordTime;

～省略～

public class Timely_RecyclerView_Adapter extends RecyclerView.Adapter<Timely_RecyclerView_Adapter.ViewHolder> {


    //    ============= Adapter ===============
    List<TimeTable_RoomEntity> allTimeTable;
    Context context;

    Timely_RecyclerView_Adapter(Context context, List<TimeTable_RoomEntity> allTimeTable) {
        this.context = context;
        this.allTimeTable = allTimeTable;
    }

    @Override
    public int getItemCount() {
        return 3;
    }

    @Override
    public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
//        holder.textView.setText(this.allTimeTable.get(position).title);
        Log.d("e", "e");
    }

    @NonNull
    @Override
    public ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
        View view = LayoutInflater.from(context).inflate(R.layout.time_viewholder, parent, false);
        Log.d("bbb", "bbb");
        return new Timely_RecyclerView_Adapter.ViewHolder(view);
    }
    //    ============= Adapter ===============


    //    ============= viewHolder ============
    public class ViewHolder extends RecyclerView.ViewHolder {
        public TextView textView;
        public ViewHolder(@NonNull View itemView) {
            super(itemView);
            this.textView = itemView.findViewById(R.id.time_view_holder);
        }
    }
    //    ============= viewHolder =============
}

レイアウト追記

①今回メインとして扱うレイアウト

<?xml version="1.0" encoding="utf-8"?>
<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="300dp"
    android:layout_height="500dp"
    android:background="@drawable/border"
    tools:context=".DailyCalendarFragment">

    <androidx.recyclerview.widget.RecyclerView
        android:id="@+id/time_recycler_view_container"
        android:layout_width="match_parent"
        android:layout_height="match_parent" />
</FrameLayout>

②メインのフラグメント内のRecyclerViewのViewholder

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <TextView
        android:id="@+id/time_view_holder"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:layout_weight="1"
        android:ems="10"
        android:text="@string/time_view_holder" />
</LinearLayout>

※③今回のメインフラグメントを置くフラグメント

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <LinearLayout
        android:id="@+id/date"
        android:layout_width="match_parent"
        android:layout_height="25sp"
        android:layout_marginBottom="706dp"
        android:backgroundTint="@color/design_default_color_primary_dark"
        android:gravity="center"
        android:orientation="horizontal"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent">

        <TextView
            android:id="@+id/selected_month"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:text="@string/fuu" />

        <TextView
            android:id="@+id/selected_date"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:text="@string/selected_date" />
    </LinearLayout>

    <androidx.fragment.app.FragmentContainerView
        android:id="@+id/fragmentContainerView"
        android:name="com.example.RecordTime.DailyCalendarFragment"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/date" />

</androidx.constraintlayout.widget.ConstraintLayout>

★★★引数自体は渡せたかと思いましたがエラーが出るパターン★★★
エラー：No adapter attached; skipping layout

package com.example.RecordTime;
～省略～

public class DailyCalendarFragment extends Fragment {

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
    }

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

    public void loadDB(View view) {
        ExecutorService executor = Executors.newSingleThreadExecutor();

        executor.submit(new Runnable() {
            @Override
            public void run() {
                Database database = Room.databaseBuilder(getContext(), Database.class, "TimeTable").build();

                TimeTable_Dao dao = database.timeTable_dao();
                dao.insert(new TimeTable_RoomEntity("0番目のtitle"));
                dao.insert(new TimeTable_RoomEntity("1番目のtitle"));
                dao.insert(new TimeTable_RoomEntity("2番目のtitle"));

                List<TimeTable_RoomEntity> allTimeTable = dao.getAll();

                setRecyclerView(view, allTimeTable);
            }
        });
    }


    public void setRecyclerView(View view, List<TimeTable_RoomEntity> allTimeTable) {
        // RecyclerView を生成して各時間を生成
        RecyclerView recyclerView = view.findViewById(R.id.time_recycler_view_container);
        // ①recycleView に layoutManager をセット
        recyclerView.setLayoutManager(new LinearLayoutManager(getContext()));
        // ②Adapter を生成して RecyclerView にセット
        recyclerView.setAdapter(new Timely_RecyclerView_Adapter(getContext(), allTimeTable));
    }
}

Comments

[hoshi-takanori]

DB アクセスは別スレッドで実行されてるので、その結果が得られたら表示を更新する必要があるかと。
また、定数ではない static 変数を使うのはお勧めしません。

[jimbe]

どこに『「←ここが上手くいかない」』があるのか探してしまいました。ファイル名と行番号を書いて頂ければ探す必要が無くて良いと思います。

それと、フラグメントとしてレイアウトも設定しているのに
RecyclerView recyclerView = getActivity().findViewById(R.id.time_recycler_view_container);
というのはどういうことでしょう。

[tetejiro]

お二方ともコメント、回答をいただき大変ありがとうございます。
修正をさせていただきました。

また、回答をいただく前に自分で試行錯誤していたので、そちらもせっかくなので
追記をさせていただきました。（★★★引数自体は渡せたかと思いましたがエラーが出るパターン★★★の部分）
※エラー（No adapter attached; skipping layout）が出ており失敗しています。

いただきました回答は、今拝見せさていただいておりますが
まだ理解するのに時間をいただきたいと思います。
見終わり次第アクションさせていただきます。
よろしくお願いいたします。

[jimbe]

ここは質問へのコメントですので、回答へのコメントは各回答にある「コメント」にお書きください。

追加されたコードの "No adapter attached; skipping layout" は、 onCreateView 後に RecyclerView を表示しようとした段階でアダプタが設定されていないからです。
それを防ぐ為、私の回答では空のアダプタを設定しておき、 loadDB のデータを後から設定しています。

Answer 1

アダプタにデータをセットするメソッドを作り、 loadDB でデータが揃ったら Handler 経由でそのメソッドを呼び出します。

なお、この構造は若干簡易版です。極稀な落ちるタイミングがあります。
もっとちゃんとするなら ViewModel で LiveData を利用することになるでしょう。

(行レイアウトは手抜きのため android の一行レイアウトを使っています)

DailyCalendarFragment.java

import android.annotation.SuppressLint;
import android.os.*;

import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.fragment.app.Fragment;
import androidx.recyclerview.widget.RecyclerView;

import android.util.Log;
import android.view.*;
import android.widget.TextView;

import java.util.*;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;

public class DailyCalendarFragment extends Fragment {
  private static final String LOG_TAG = "DailyCalendarFragment";

  private final ExecutorService executor = Executors.newSingleThreadExecutor();
  private Handler handler;
  private Adapter adapter;

  DailyCalendarFragment() {
    super(R.layout.fragment_daily_calendar);
  }

  @Override
  public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
    adapter = new Adapter();

    RecyclerView recyclerView = view.findViewById(R.id.recycler_view);
    recyclerView.setAdapter(adapter);

    handler = new Handler(Looper.getMainLooper());

    loadDB();
  }

  private void loadDB() {
    executor.submit(() -> {
      Log.d(LOG_TAG, "loadDB start.");
      try {
        Thread.sleep(3*1000L); //時間が掛かる処理のつもり(分かり易いよう長め)
      } catch(InterruptedException ignore) {
        //no process
      }
      List<Item> itemList = new ArrayList<>();
      itemList.add(new Item("0番目のtitle"));
      itemList.add(new Item("1番目のtitle"));
      itemList.add(new Item("2番目のtitle"));

      handler.post(() -> adapter.set(itemList));
      Log.d(LOG_TAG, "loadDB end.");
    });
  }

  private static class Adapter extends RecyclerView.Adapter<Adapter.ViewHolder> {
    private final List<Item> itemList = new ArrayList<>();

    @SuppressLint("NotifyDataSetChanged")
    void set(List<Item> newItemList) {
      Log.d(LOG_TAG, "Adapter#set start. newItemList.size()=" + newItemList.size());
      itemList.clear();
      itemList.addAll(newItemList); //防御コピー
      notifyDataSetChanged();
      Log.d(LOG_TAG, "Adapter#set end.");
    }

    @Override
    public int getItemCount() {
      return itemList.size();
    }

    @Override
    public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
      holder.bind(itemList.get(position));
    }

    @NonNull
    @Override
    public ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
      return new ViewHolder(parent);
    }

    static class ViewHolder extends RecyclerView.ViewHolder {
      private final TextView textView;

      private ViewHolder(@NonNull ViewGroup parent) {
        super(LayoutInflater.from(parent.getContext()).inflate(android.R.layout.simple_list_item_1, parent, false));
        textView = itemView.findViewById(android.R.id.text1);
      }

      private void bind(Item item) {
        textView.setText(item.title);
      }
    }
  }
}

class Item {
  final String title;
  Item(String title) {
    this.title = title;
  }
}

res/layout/fragment_daily_calendar.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.recyclerview.widget.RecyclerView xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    android:id="@+id/recycler_view"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    app:layoutManager="androidx.recyclerview.widget.LinearLayoutManager" />

実行結果

実行時 Logcat

2023-12-19 03:55:44.953 23160-23185 DailyCalendarFragment   com.teratail.q_nq6yuripcws7qn        D  loadDB start.
2023-12-19 03:55:47.955 23160-23160 DailyCalendarFragment   com.teratail.q_nq6yuripcws7qn        D  Adapter#set start. newItemList.size()=3
2023-12-19 03:55:47.955 23160-23160 DailyCalendarFragment   com.teratail.q_nq6yuripcws7qn        D  Adapter#set end.
2023-12-19 03:55:47.955 23160-23185 DailyCalendarFragment   com.teratail.q_nq6yuripcws7qn        D  loadDB end.

Comments

[tetejiro]

ご回答いただきありがとうございました。
理解に時間がかかり返信が遅くなってしまい、大変申し訳ございませんでした。
質問を重ねてしまい大変恐縮ですが、
以下を確認したく、ご回答をお願いできますでしょうか？

①"No adapter attached; skipping layout" は、 onCreateView 後に RecyclerView を表示しようとした段階でアダプタが設定されていないからです。
それを防ぐ為、私の回答では空のアダプタを設定しておき、 loadDB のデータを後から設定しています。

この言葉が長い間理解できずにいたのですが、以下のような意味で合っていますでしょうか？
RecyclerViewで表示するデータの中身を空にしておかないと、
"No adapter attached; skipping layout"エラーが発生する。
そのためRecyclerViewで表示するデータは最初は空にしておき、
その後RecyclerViewクラスのsetメソッドを用いて、表示を更新するという処理にした。
（https://www.jianshu.com/p/da2fda474624　中国語？の記事でした、すみません。この記事と同じことをご指摘いただいていると考えました）

②Thread.sleep(3*1000L);
に関してですが、この処理は通常行う、いわゆるベストプラクティスなのでしょうか？
今回扱うデータは小さいデータを想定しているのですが
その場合も必要なのでしょうか？

③handler.post(() -> adapter.set(itemList));
に関してですが、handler.post では、キューに積むのと実行を両方する処理ということで合っていますでしょうか？
（スレッドを初めて扱うため初歩的な質問ですみません。）

④itemList.addAll(newItemList); //防御コピー
に関してですが、以下のサイトの内容のことということで合っていますでしょうか？
（受け取る側で new して改ざん不可にすると認識しています。）
※https://thekingsmuseum.info/entry/2016/06/02/231750

たくさん質問をしてしまい大変申し訳ございません。
もしよろしければご回答をよろしくお願いいたします。

[jimbe]

>RecyclerViewで表示するデータの中身を空にしておかないと、
>"No adapter attached; skipping layout"エラーが発生する。

"データの中身" という抽象的(?) な話では無く、単に ”RecyclerView が表示される段階でまだ setAdapter が呼ばれてないから、何をどう表示するのか全く分からないから表示出来ないよ” ということです。
リンク先の内容も(google翻訳が中国語での表現をうまく翻訳出来ているのか分かりませんが)そういうことでしょう。

loadDB で sleep しているのはコメントに書いている通り、 データベースアクセス(は時に時間が掛かるモノなのでそれ)をシミュレートしているだけです。

Android で データベースにアクセスするのを(EDTとは)別スレッドにするというのは、本件のアプリの場合はともかくとして大量のデータがある場合は時間が掛かるものであり、その間アプリが反応しなくなるのを防ぐためです。(本件のコードもその意図を汲んで executor でやっているのではないですか? )
そして Android のライフサイクルのメソッドが呼ばれるよりも時間が掛かっているからこそ本件の問題が発生しています。
逆に言うと、もしデータベースのデータが少なくてライフサイクルのメソッドが呼ばれるよりも時間が掛からず(見た目)正常に動作してしまうと、データが増えて時間が掛かるようになった時に表示がされなくなって何が悪いのか分かり難くなるという面倒なバグになります。
そのため、データベースの読み込みに時間が掛かっても表示がされるということを確認する意味で、データベースに大量のデータを入れるよりも簡単に sleep させています。
回答はあくまで動作確認のサンプルであり、実際のアプリでは当然 sleep は入れません。

>handler.post では、キューに積むのと実行を両方する処理

Android は EDT と呼ばれる単一スレッドが全体の動作を支配しています。EDT の動作自体は簡単で『キューからイベントを取り出して実行』を繰り返すだけです。(つまり Android 自体の中核はシングルスレッドです。)
Looper.getMainLooper() から作った Handler はその EDT のキューに対して処理することが出来、 post はキューへの追加です。
キューは常に多くのプログラムからイベントが入れられており EDT は常にそれらを取り出しては実行していますので、やがて post したイベントが EDT に取り出されれば、その時に実行されます。

"防御コピー"とコメントしている個所の意図は、まさにリンク先のことを意識しています。
ビアソンエデュケーションの『Effective Java』は(もう古い初版と第2版ですが)私も持っています。

[tetejiro]

ご丁寧にご回答をしていただき誠にありがとうございます。
質問への対応プラスアルファでこのようにお付き合いいただき
大変感謝しております。
追記もありがとうございました。

質問の内容を、①以外は理解できたと思います。

しつこく聞いてしまってすみませんが
もしよろしければ①について教えてくださいますでしょうか？
本当に何度もすみません。
（もし別で質問を立てたほうがよろしければそのようにさせていただきますので、おっしゃってください。）

＞”RecyclerView が表示される段階でまだ setAdapter が呼ばれてないから、何をどう表示するのか全く分からないから表示出来ないよ” ということです。

onCreateView()でそのフラグメントの View が生成されるので、
それまでに setAdapter をしておけば問題ないのかと思い、
onAttach や onCreate などで事前に setAdapter の処理をしてみましたが
それではできませんでした。
このことで、頭が混乱してしまっている状況です。

また、このエラー(No adapter attached; skipping layout)が発生するタイミングをログで出力してみたのですが
※★★★引数自体は渡せたかと思いましたがエラーが出るパターン★★★での実行です。

public void run() {
Log.d("ccccccccccc=======>", "cccccccccccc");　<=======C
Database database = Room.databaseBuilder(getContext(), Database.class, "TimeTable").build();

TimeTable_Dao dao = database.timeTable_dao();
dao.insert(new TimeTable_RoomEntity("0番目のtitle"));
dao.insert(new TimeTable_RoomEntity("1番目のtitle"));
dao.insert(new TimeTable_RoomEntity("2番目のtitle"));

List<TimeTable_RoomEntity> allTimeTable = dao.getAll();

Log.d("ddddddddddd=======>", "ddddddddd");　<=======D
setRecyclerView(view, allTimeTable);
Log.d("eeeeeeeee=======>", "eeeeeeeeee");　<=======E
}

の、　c と d の間で No adapter attached; skipping layout が出ていました。
d と e の間で行われないということで、
私が漏らしている観点のようなものがあるのでしょうか？

大変恐縮ですが、
もしよろしければご回答よろしくお願いします。

[jimbe]

私の回答に関するものであればこちらに書いて頂くのが妥当と思いますし、ご理解頂けなければ回答の意味がありませんので質問頂くのも問題ありません。

>それまでに setAdapter をしておけば

該当の例外は EDT 上で発生しているものですので、別のスレッドの(ログの出力との)タイミングは関係ありません。
EDT は onViewCreated が終わり次第 view を表示しようとします。 onViewCreated の最後の loadDB は「データベースから読み込んで setAdapter を実行するスレッド」(長いので以降 DBRT と呼びます)を起動します。つまり loadDB は即戻ってくるので onViewCreated も早々に終わり、 EDT が view を表示しようとするのと DBRT は同時に実行されていて、しかもどのスレッドが何時何をしているかは別のスレッドは分かりません。マルチスレッドでは(意図して同期処理を書かない限り)あるスレッドのある段階で他のスレッドが何かをし終わっている等は保証されません。

起きている現象は、** たまたま ** DBRT の C と D のログの間くらいの時間に EDT が(レイアウトに定義されている) RecyclerView を表示しようとしており、『あれ？ Adapter が無いじゃん』と気が付いて No adapter～ と言っているのです。

[tetejiro]

＞loadDB は即戻ってくる
ここが理解ができずにいました。
ようやく理解ができたと思います。
本当にありがとうございました。

最初から最後まで本当にご丁寧に回答をしていただき
大変感謝しております。

質問への回答だけでなく、他の学びも大いにあり
勉強になりました。
改めて、ありがとうございました。