[assigned] FireBaseAnalytics.setCurrentScreen() on Fragment may not be called

Overview (Required)

(日本語で失礼します)

一部のFragmentでFireBaseAnalytics.setCurrentScreenが呼ばれないケースがあります。 そのFragmentとは、AllSessionsFragmentやSearchSessionsFragmentと、ViewPagerの中の最初のFragmentです。 一例としてSearchSessionsFragmentのライフサイクルは以下のようになります。

1. アプリを起動
2. bottom navigationで検索を選択
3. SearchSessionsFragmentインスタンス生成
4. setUserVisibleHint()が呼ばれるが ログは送れない
5. onAttach()が呼ばれる
6. onActivityCreated()が呼ばれる（fireBaseAnalyticsインスタンスの初期化）
7. トピックタブ(SearchTopicsFragment)に移動、トピックタブのログは送れる
8. 再度セッションタブ(SearchSessionsFragment)に移動、この時のログは送れる

現状setUserVisibleHint()でログを送っていますが、 上記の 3 の時点だとfireBaseAnalytics はまだnullなので実行されません。 たとえfireBaseAnalyticsの初期化タイミングを早めて実行できたとしても、まだこのときはAttachされておらずgetActivity()がnullなのでsetCurrentScreen()を実行できません。

代わりにViewPager.OnPageChangeListener を使って試そうとしましたが、あまりいいやり方にできなかったのでissueにしました 🤔

Links

• https://developer.android.com/reference/android/app/Fragment.html#setUserVisibleHint(boolean)

[eneim]

Assigned されたが、コメントだけさせてください 😄, setUserVisibleHint(true) でしかログを送信するようであれば、FragmentStatePagerAdapterを実装する時に、FragmentStatePagerAdapter#setPrimaryItem() をOverrideして、何とかできるかもしれません 😄 (setUserVisibleHint()に「true」が渡されたのはこのメソッドでしか呼ばれないわけですね)

そして fireBaseAnalytics のインスタンスは事前で生成すべきかと思います。SearchFragmentで生成して、PagerAdapterで使うなど 😄

[kakajika]

少し冗長かもしれませんが、Rxの withLatestFrom を使ってイベントの待ち合わせをするという手もあります。

private val onActivityCreatedStream = PublishSubject.create<Activity>()
private val setUserVisibleHintStream = PublishSubject.create<Boolean>()

override fun onCreate(...) {
    super.onCreate(...)

    // onActivityCreated() が呼ばれた状態で、setUserVisibleHint() が呼ばれるたびに実行
    setUserVisibleHintStream
            .withLatestFrom(onActivityCreatedStream)
            .filter { (isVisibleToUser, _) -> isVisibleToUser }
            .subscribe { (isVisibleToUser, activity) ->
                fireBaseAnalytics?.setCurrentScreen(activity, null, this::class.java.simpleName)
            }
            .addTo(...)
}

override fun onActivityCreated(...) {
    super.onActivityCreated(...)
    activity?.let { onActivityCreatedStream.onNext(it) }
}

override fun setUserVisibleHint(isVisibleToUser: Boolean) {
    super.setUserVisibleHint(isVisibleToUser)
    setUserVisibleHintStream.onNext(isVisibleToUser)
}

ご参考までに 😃