アプリ起動時のonLayoutChildの挙動について

Question

前提・実現したいこと

ViewPagerの実装を勉強しながらそれっぽいアプリのような形になるまでどんどん機能を追加していこうという勉強をしています。
ViewPagerが形になったのでBottomNavigationViewを追加して、こちらのサイトの「■スクロール時に隠す」の項を参考にしてBottomNavigationViewをスクロール時に隠すようにしようとしています。

BottomNavigationViewのカスタマイズ - Re.Ra.Ku アドベントカレンダー day 18

実装はサイトにあるものほぼそのまま(サイト上ではRecycleViewがあるところにViewPagerを指定してなかに長文TextViewを仕込んでいる)で、実際に動かしてみたところ下スクロールすると消えることは消えるのですが上スクロールしても表示が元に戻りません。
おかしいなと思い色々試していると、"起動時だけ"、childのgetTop, getBottomが0になっていることがわかりました。
しかし、なぜそうなっているのか、どう回避すれば良いかがわかりません。
ご教示お願いします。

試したこと

• 起動時に発動する時のchildがBottomNavigationViewではない別のなにかを拾っているのかと思ったのですが、childの中身はきちんとBottomNavigationViewであることをgetIdで確認しています。
• 起動後にBottomNavigationViewの最初以外のメニュー(※)をタップすると、正常にgetTop等が働いて目的通りの動作をします。

※メニューA,B,Cとある時、AをタップしてもonLayoutChildは発動せず、BまたはCをタップした時のみonLayoutChildが発動しています。なお各種メニューはイベント未設定のため現状見かけだけです。

該当のソースコード

activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout 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:id="@+id/ll"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical"
    tools:context="com.hoge.viewpager.MainActivity">

    <android.support.design.widget.CoordinatorLayout
        android:id="@+id/cl"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:fitsSystemWindows="true">

        <android.support.v4.view.ViewPager
            android:id="@+id/viewpager"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            app:layout_behavior="@string/appbar_scrolling_view_behavior">

        </android.support.v4.view.ViewPager>

        <android.support.design.widget.BottomNavigationView
            android:id="@+id/bottomNavigationView"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:layout_gravity="bottom"
            app:itemBackground="@color/colorPrimaryDark"
            app:itemIconTint="@android:color/white"
            app:itemTextColor="@android:color/white"
            app:layout_behavior="com.hoge.viewpager.BottomNavigationBehavior"
            app:layout_constraintBottom_toTopOf="@+id/viewpager"
            app:layout_constraintEnd_toEndOf="@+id/viewpager"
            app:layout_constraintHorizontal_bias="1.0"
            app:layout_constraintStart_toStartOf="@+id/viewpager"
            app:layout_constraintTop_toTopOf="parent"
            app:menu="@menu/navigation" />

    </android.support.design.widget.CoordinatorLayout>

</LinearLayout>

問題の箇所

    public boolean onLayoutChild(CoordinatorLayout parent, BottomNavigationView child, int layoutDirection) {
        defaultTop = child.getTop();
        defaultBottom = child.getBottom();
        defaultHeight = defaultBottom - defaultTop;
        Log.d(TAG, "onLayoutChild: "+defaultTop);　//起動時0  メニューB,Cタップ時は正常に取得
        Log.d(TAG, "onLayoutChild: "+defaultBottom); //起動時0　メニューB,Cタップ時は正常に取得
        Log.d(TAG, "onLayoutChild: "+defaultHeight); 
        Log.d(TAG, "onLayoutChild: "+ _context.getResources().getResourceEntryName(child.getId()) ); //起動時ちゃんとbottomNavigationViewが表示される

        return super.onLayoutChild(parent, child, layoutDirection);
    }

Answer 1

CoordinatorLayoutのソースコードを読んでみた限りだと、初回はレイアウトがされていない状態でbehavior.onLayoutChildが呼ばれるようなので、自分でレイアウト用のメソッドを呼んであげる必要がありそうです。

public boolean onLayoutChild(CoordinatorLayout parent, BottomNavigationView child, int layoutDirection) {
    // 自分でレイアウト用のメソッドを呼ぶ
    parent.onLayoutChlid(child, layoutDirection);

    defaultTop = child.getTop();
    defaultBottom = child.getBottom();
    defaultHeight = defaultBottom - defaultTop;

    // 自分でレイアウト用のメソッドを呼んだことを上に伝える
    return true;
}

このようにしてみてはいかがでしょうか？

Comments

[inzs]

ありがとうございます。回答頂いた方法で目的通り動作しました。
一応初回だけでいいかと思って以下のようにしてみました。

if (child.getTop()== child.getBottom()) parent.onLayoutChild(child,layoutDirection);

[kakajika]

そうですね、たしかに無駄なレイアウト処理を行うようになっていました。
ソースコードによると、behavior.onLayoutChildでtrueを返すことにより、レイアウト処理が重複して走ってしまわないように指定できるようです。そのように回答を修正しておきました。

[inzs]

なるほど！それだけで問題なさそうですね。

私が先に書いた　if (child.getTop() == child.getBottom()) とした場合、return super.onLayoutChild～で再レイアウト処理されていましたが、レイアウト処理前にTop,Bottomを取得してしまっているため、BottomNavigationViewを隠した後にViewPagerをスライドさせた時にonLayoutChildが発動して、BottomNavigationViewが隠れてしまっている状態の座標を取得するため正しく表示されないようです。

よって、編集回答頂いた内容そのままが一番良いようです。
ありがとうございました。