FragmentActivity?のgetActivityがnullになる

Question

最近、Firebaseを使ったアプリを開発中です。（ボードゲームアプリです）

実は、FragmentActivity?のactivityが、フラグメントが削除されてないにもかかわらずnullになる時があります。(ブレークポイントで確認済みなのでnullになってるのは確かです）

Fragmentが開いている状態で、このようなことが起こることがあるのでしょうか？コードを載せようにも、全て載せるには多すぎて載せれず、絞って載せるにもどこを載せればいいのかわからないので、GitHubリンクを載せておきます。問題の箇所はGameSettingFragmentのsetSpinnerメソッドです。バリバリの初心者なのでコード読みにくかったらすみません。
GitHub

毎回必ずしもnullになるというわけでなく、ある条件下でなります。
①ルームを新しく作成し（SetRoomInfoFragmentにて）、（WaitingMembersFragmentを通り）GameSetting画面で最初にチーム分けをするとき
チーム分けをする前はmember情報がnameしかない。チーム分けした後にmember情報がhost,team,name,vote(null)と増える。エラーで強制終了させられても、チーム分けはされており(member情報がfirebaseに保存される）、再度アプリを立ち上げ、参加者としてGameSettingFragment画面まで進むと、エラーは起きない。

問題があった箇所(GameSettingFragment)だけ載せます。他はGitHubを参照していただくとありがたいです。

 private fun setSpinner(teamRed: MutableList<Uid>, teamBlue: MutableList<Uid>) {

.
.
.

        adapter= // activityがnull -> adapterもnullになり強制クローズ
            when (isMyTeam) {
                Team.RED -> ArrayAdapter(activity!!.baseContext,
                    R.layout.spiner_item,
                    teamRedNameList)
                Team.BLUE -> ArrayAdapter(activity!!.baseContext,
                    R.layout.spiner_item,
                    teamBlueNameList)
            }
        adapter?.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item)
        spinner.adapter = adapter
        
.
.
.
      
    }

Comments

[hoshi-takanori]

setSpinner は Firestore の listener から呼ばれてるようですが、fragment を閉じた後も listener は残るので、その場合 activity が null になります。activity と fragment の lifecycle を意識する必要がありますね。その上で、ViewModel とかを使うといいと思います。あと、isMyTeam (is で始まると boolean かと思っちゃいます) はグローバル変数のようですが、Android でグローバル変数を使うのは良くないです…。

[hoshi-takanori]

↑よく見たら onDetach で remove してましたね。かつ、フラグメントも削除してないなら別の問題ですね。失礼しました。(動かして確認しようと思ったけど、Android が 4 つないと試せない…。)

[yukari00]

返信遅れてすみません！
グローバル変数使うのは良くないんですね汗
知りませんでした！ご指摘ありがとうございます。
あと、試そうとしてくださってありがとうございます（４つないと試せない状態ですみませんんん）一旦、原因が分からなかったのでif(activity != null)の後に処理を書き、保留状態にして開発を進めています。そしたら、また別の壁にぶち当たったのでこちらの方も確認してくださると嬉しいです！

https://teratail.com/questions/273173

Answer 1

GameSettingFragmentのコードを拝見しました。色々とアドバイスしたくなる感じのコードですが、とりあえずご質問の点（FragmentやActivityのライフサイクル関連）に絞って回答します。

実は、FragmentActivity?のactivityが、フラグメントが削除されてないにもかかわらずnullになる時があります。

Fragmentが開いている状態で、このようなことが起こることがあるのでしょうか？

提示された現状のコードでは、問題の箇所でgetActivity()がnullにならないことは保証できません。setSpinnerの呼び出し元であるlisteningMembers含め、FragmentおよびViewの破棄時にリスナーの解除が正しく行われていないためです。（購読を行うaddSnapshotListenerだけでなく、1度だけ呼び出されるaddOnSuccessListenerの方もFragmentの破棄後にコールバックが実行される可能性があります）

---

これらの問題を解消する方法として、Android Architecture ComponentsのLifecycleが提供する機能を利用することから始めてみるのをおすすめします。AndroidXのFragmentやAppCompatActivityをお使いなら、すでにLifecycleを利用する準備は整っています。

1. FragmentのonDestroyView時に自動的に解放されるリスナーメソッドを実装する

現状のコードではFirestoreへのリスナーを登録した後、自分でリスナー解除のメソッドを呼ぶようになっていますが、これの呼び忘れを防ぐためにonDestroyView()で自動的に解除されるように実装します。（onDetach()で解除するのはViewへのアクセスで落ちる可能性があるので、アンチパターンです）

まずはFirestoreの拡張メソッドを実装します。

• FirestoreExts.kt

fun DocumentReference.addSnapshotListener(
    owner: LifecycleOwner,
    listener: (DocumentSnapshot?, FirebaseFirestoreException?) -> Unit
): ListenerRegistration {
    val registration = addSnapshotListener(listener)
    owner.lifecycle.addObserver(object : LifecycleObserver {
        @OnLifecycleEvent(Lifecycle.Event.ON_DESTROY)
        fun onDestroy() {
            registration.remove()
            owner.lifecycle.removeObserver(this)
        }
    })
    return registration
}

Fragment内で呼んでいるaddSnapshotListenerを上記の拡張に置き換えます。
onDestoryView()で解放してほしいので、viewLifecycleOwnerを利用しています。

listeningMembers = database.collection(dbCollection)
    .document(keyword)
    .collection("members")
    .addSnapshotListener(viewLifecycleOwner) { it, e ->
        // ViewやActivityにアクセスする任意の処理...
    }

以上で、ライフサイクルに関するリスナーの解放漏れを防ぐことができます。
長くなるのでaddOnSuccessListener等の実装については割愛します。同じように実装してみてください。

2. 特定のライフサイクル内でコードが実行されるようにする

上記の1.だけでも問題は解消されると思いますが、さらにライフサイクル関連の対策を確実にするために、特定のライルサイクル内（例えば、onCreateView()〜onDestoryView()の間）だけ実行されるような処理を書くことができます。
この実装にはLifecycle KTXの導入が必要になります。

implementation "androidx.lifecycle:lifecycle-runtime-ktx:$lifecycle_version")

setSpinnerを以下のように変更することで、コメントアウトした部分の処理はonCreateView()〜onDestroyView()の間だけ実行が開始されることが保証されます。

private fun setSpinner(...) {
    viewLifecycleOwner.lifecycleScope.launchWhenCreated {
        // ViewやActivityにアクセスする任意の処理...
    }
}

---

こんな感じでライフサイクルに関する問題はAAC Lifecycleを利用することでだいぶ扱いやすくなります。AAC Lifecycleに慣れてきたら、次のステップとして関連する機能であるLiveDataやViewModelの導入に取り組んでみるのも検討してみてください。

Comments

[yukari00]

素晴らしい回答ありがとうございます！！！自分の次の課題が見えてきました！
一旦指摘していただいたところ、調べてみて試してみます！