RecyclerViewの更新が反映されない(Firebaseから取得したデータリストを並べる)

Question

前提・実現したいこと

チーム管理系のアプリを作成しています。
アプリを起動すると、Firebaseへ自動ログインされ、その後Firebaseから、ユーザー固有のデータ（その人が所属しているチーム一覧データ）を取得し、RecyclerViewに反映させます。

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

起動しても、RecyclerViewが空のままで、反映されません。
しかし、FABをクリックして、ダイアログを表示させ、そこのEditTextをタップして、キーボードが出てくると、後ろのほうでチーム一覧が出現し、その後ダイアログを閉じてもそのまま残っています。（アプリを再起動するとまた最初、空のまま表示される。）

該当のソースコード

class TeamActivity : AppCompatActivity() {


    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)

        //端末内に保存されているメアドパスデータを取得
        val data = getSharedPreferences("Data", Context.MODE_PRIVATE)
        var Emaildata = data.getString("Email", null)
        var Passdata = data.getString("Passwoed", null)

        if ((Emaildata != null) && (Passdata != null)) {
            val mAuth = FirebaseAuth.getInstance()
            mAuth.signInWithEmailAndPassword(Emaildata, Passdata)
                .addOnCompleteListener {
                    if (it.isSuccessful) {
                        Toast.makeText(applicationContext, "ログインしました", Toast.LENGTH_LONG).show()
                        // 成功処理
                    } else {
                        Toast.makeText(
                            applicationContext,
                            "ログインに失敗しました\n申し訳ありませんがもう一度\nもう一度登録しなおしてください",
                            Toast.LENGTH_LONG
                        ).show()
                        // 失敗処理
                    }
                }

            // [START check_current_user]
        }


        val user = Firebase.auth.currentUser
        if (user != null) {
            // User is signed in
        } else {
            // No user is signed in
            val intent = Intent(applicationContext, LoginActivity::class.java)
            //第2画面に送るデータを格納。
            startActivity(intent)
        }
        // [END check_current_user]
        setContentView(R.layout.activity_team)

        val layoutManager = GridLayoutManager(this, 2, GridLayoutManager.VERTICAL, false)

        // アダプターとレイアウトマネージャーをセット
        val TeamRecyclerView = findViewById<RecyclerView>(R.id.TeamRecyclerView)
        val adapter = CustomAdapter(TeamList())
        TeamRecyclerView.layoutManager = layoutManager
        TeamRecyclerView.setHasFixedSize(true)
        TeamRecyclerView.adapter = adapter

        val makefab = findViewById<FloatingActionButton>(R.id.sub_btn1)
        val joinfab = findViewById<FloatingActionButton>(R.id.sub_btn2)
        makefab.setOnClickListener(MakeFabClickListener())
        joinfab.setOnClickListener(JoinFabClickListener())
    }

    private inner class MakeFabClickListener : View.OnClickListener {
        override fun onClick(v: View?) {
            val dialogFragment = MakeTeamDialogFragment()
            dialogFragment.show(supportFragmentManager, "MakeTeamDialogFragment")
        }
    }

    private inner class JoinFabClickListener : View.OnClickListener {
        override fun onClick(v: View?) {
            val dialogFragment = JoinTeamDialogFragment()
            dialogFragment.show(supportFragmentManager, "JoinTeamDialogFragment")
        }
    }
}
fun TeamList(): MutableList<MutableMap<String, String>> {
    val uid = Firebase.auth.currentUser?.uid.toString()
    val dbc = FirebaseFirestore.getInstance()
    val teamList: MutableList<MutableMap<String, String>> = mutableListOf()
    dbc.collection("Users").document(uid).get()
        .addOnSuccessListener {
            val dataList = it.data
            var team = mutableMapOf<String,String>()
            dataList?.forEach {
                team = mutableMapOf("team" to it.value.toString())
                teamList.add(team)
            }
        }
    return teamList
}
// ViewHolderクラスを作成
class CustomViewHolder(val view: View) : RecyclerView.ViewHolder(view) {
    val tvTeamname = view.tvteamname
}

class CustomAdapter(private val customList: MutableList<MutableMap<String,String>>) : RecyclerView.Adapter<CustomViewHolder>() {

    // 上記のViewHolderクラスを使ってViewHolderを作成
    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): CustomViewHolder {
        val Inflater = LayoutInflater.from(parent.context)
        val item = Inflater.inflate(R.layout.grid_view, parent, false)
        return CustomViewHolder(item)
    }

    // ViewHolderに表示するデータを挿入
    override fun onBindViewHolder(holder: CustomViewHolder, position: Int) {
        val item = customList[position]
        val teamname = item["team"] as String
        holder.view.tvteamname.text = teamname
    }

    // getItemCount onCreateViewHolder onBindViewHolderを実装
    // recyclerViewのコンテンツのサイズ
    override fun getItemCount(): Int {
        return customList.size
    }
}

activity_team.xml

grid_view.xml

class JoinTeamDialogFragment: DialogFragment() {
    private val TAG = "JoinFragment"

    override fun onCreateDialog(savedInstanceState: Bundle?): Dialog {
        val activity = requireActivity()
        val builder = AlertDialog.Builder(activity)
        val inflater = activity.layoutInflater
        val teamjoinview = inflater.inflate(R.layout.team_join_dialog,null)


        builder.setView(teamjoinview)
            .setTitle("チームを追加")
            .setPositiveButton("参加"){dialog,id->
                //入力欄の有無を判別する//
                val teamid =teamjoinview.findViewById<EditText>(R.id.etteamid)
                val mynamej = teamjoinview.findViewById<EditText>(R.id.etmynamej)
                val teamidEdi = teamid.text
                val mynamejEdi = mynamej.text

                //チームに参加（チームにuid追加）して、uidに所属チームを紐づける
                if(!TextUtils.isEmpty(teamidEdi) && !TextUtils.isEmpty(mynamejEdi)){
                    val uid = Firebase.auth.currentUser?.uid.toString()
                    val teamidStr = teamidEdi.toString()
                    val mynamejStr = mynamejEdi.toString()
                    val dbc = FirebaseFirestore.getInstance()


                    dbc.collection("Teams")
                        .document(teamidStr)
                        .collection("members")
                        .document("members")
                        .set(mapOf(uid to mynamejStr), SetOptions.merge())

                    dbc.collection("Teams")
                        .document(teamidStr)
                        .get()
                        .addOnCompleteListener { task ->
                            if (task.isSuccessful) {
                                val document = task.result
                                if (document != null && document.data != null) {
                                    val tname = document.data?.get("teamname")
                                    dbc.collection("Users")
                                        .document(uid)
                                        .set(mapOf(teamidStr to tname), SetOptions.merge())
                                }else {
                                }
                            }else {
                            }
                        }
                        .addOnFailureListener { e -> }

                    dbc.collection("Users").document(uid).get()
                        .addOnSuccessListener {
                            val dataList = it.data
                            val teamlist: MutableList<MutableMap<String, String>> = mutableListOf()
                            dataList?.forEach {
                                var team = mutableMapOf("team" to it.value.toString())
                                teamlist.add(team)
                            }

                            val adapter = CustomAdapter(teamlist)
                            adapter.notifyDataSetChanged()
                        }
                }else{
                    Toast.makeText(activity,"チームコードとあなたのお名前を\n入力してください。",Toast.LENGTH_LONG).show()
                }
                //入力欄有無判別終了
            }
            .setNegativeButton("キャンセル"){dialog,id->}
        return builder.create()
    }


}

画面がこうなります

↓

↓

↓ここでうしろにうっすらとチームのカードが出現している。

↓

試したこと

Firebaseからデータを持ってくる前に描画してしまっているのかなとは思いましたが、なんとも原因がわかりません。TeamActivityでadapterを設定した直後にnotifydatasetchangedを呼んでもみましたが、最初に描画はされないままでした。
ちなみにアダプタに渡すリストを固定の用意したリストにしたら、普通に最初から描画されました。ので、firebaseからとってきたデータをリストにして、アダプタに渡すところに問題があるのかなと思いました。

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

ちなみに、もうひとつのエラーとして、ダイアログにチームコードと名前を入力して、参加を押して、Firebaseのユーザーの所属チーム一覧に新しいチームのデータを追加して、notifydatasetchangedでRecyclerViewを更新しても、ダイアログが消えた後、RecyclerViewは更新されてないまま、前のチーム一覧が載っていたので、そちらもわかるかた、解決法ご教授いただきたいです。

何卒、皆様のお力をお貸しくださいmm

Comments

[hoshi-takanori]

teamList の内容が変化した時に、CustomAdapter の notifyDataSetChanged() を呼んで表示を更新する必要がある気がします。

[joseryo]

それは、foreachでaddしていくたびにnotifyをよぶということでしょうか。
試したことに追記しましたが、TeamActivityでアダプタ呼んでセットした後にnotifyしても、変化はありませんでした。

Answer 1

JoinTeamDialogFragment で notifyDataSetChanged 自体は呼んでますが、

                            val adapter = CustomAdapter(teamlist)
                            adapter.notifyDataSetChanged()

としているので、RecyclerView に設定された adapter ではなく、新しく別の CustomAdapter を作ってそれに対して呼んでいるので、これでは意味がありません。

とりあえず adapter を TeamActivity のプロパティにして、

 class TeamActivity : AppCompatActivity() {
 
+    lateinit var adapter: CustomAdapter
 
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
 
         // 略
 
         // アダプターとレイアウトマネージャーをセット
         val TeamRecyclerView = findViewById<RecyclerView>(R.id.TeamRecyclerView)
-        val adapter = CustomAdapter(TeamList())
+        adapter = CustomAdapter(TeamList())

CustomAdapter の customList を外から設定できるようにして、

- class CustomAdapter(private val customList: MutableList<MutableMap<String,String>>) : RecyclerView.Adapter<CustomViewHolder>() 
+ class CustomAdapter(var customList: MutableList<MutableMap<String,String>>) : RecyclerView.Adapter<CustomViewHolder>() 

JoinTeamDialogFragment で呼び出し元の Activity を取得して、その adapter に対して操作を行うようにすればいいと思います。

-                            val adapter = CustomAdapter(teamlist)
-                            adapter.notifyDataSetChanged()
+                            val adapter = (activity as TeamActivity).adapter
+                            adapter.customList = teamlist
+                            adapter.notifyDataSetChanged()

参考: Android - DialogFragmentでどのボタンがクリックされたか、呼び出し元のActivityに知らせたい。｜teratail

Comments

[joseryo]

hoshi様、補足の方のエラーの解決策ということで、ご回答ありがとうございます。
早速ご教授いただいた、コードを試してみましたが、ダイアログに入力し参加を押し、ダイアログが閉じたあと、以前と同様にRecyClerViewが変化することはありませんでした。
やはり、今回の質問のメインエラーである、アプリ立ち上げ時にリストが反映されない（EditTextwをいじると出現するという変なタイミングで反映される）の方に根本的な問題があるように思えます。
そちらの方でなにか心当たりのあることありましたら、ご教授お願いしたく存じますm(_ _)m

Answer 2

TeamRecyclerView.layoutManager = layoutManager
TeamRecyclerView.setHasFixedSize(true)
adapter = CustomAdapter(TeamList())
Handler().postDelayed(Runnable {
adapter.notifyDataSetChanged()
TeamRecyclerView.adapter = adapter
}, 3000)

このように、一部を3秒の遅延処理にすることで、更新されました。
補足での、ダイアログフラグメントからのアダプタ呼び出しについても、hoshi様のご回答により解決できましたが、

遅延処理はその場しのぎの解決方法なので、どなたか、読み込み次第更新されるような解決策がわかるかた、ご教授お願いいたします。