フラグメントのonViewCreatedメソッドでfindViewByIdがnullになる。

Question

前提・実現したいこと

お世話になっております。
Android初学者で書籍を使用して勉強している者です。
現在Kotlinでフラグメントについて学習しており、画面をタップした際の動きとして2種類の動きを実装しています。
スマホサイズでは第2のアクティビティが起動し、タブレットサイズではフラグメントが追加される動きになります。

ですが書籍の情報が少し古く、書籍のフラグメントのソースコードで使用しているonActivityメソッドが、Fragmentライブラリのバージョン1.3.0以降になった際に非推奨になっているようです。

そのため色々検索していたところ、現在はonViewCreatedメソッドを使うことが原則推奨されているようで、その中に書籍のonActivityメソッドと同じ処理を記述したのですが、デバッグしてみたところタブレットサイズの際にactivity?.findViewById<View>(R.id.menuThanksFrame)の処理で変数menuThanksFrameがnullになってしまいタブレットサイズだと認識されない状態です。

あまり知識が足りていなく、どのように解決して良いかわからないです。
是非お力をお借りしたく今回質問させていただきます。
よろしくお願いいたします。

該当のソースコード1

MenuListFragment.kt

package com.websarva.wings.android.fragmentsample

import android.content.Intent
import android.os.Bundle
import androidx.fragment.app.Fragment
import android.view.LayoutInflater
import android.view.View
import android.view.ViewGroup
import android.widget.AdapterView
import android.widget.ListView
import android.widget.SimpleAdapter

class MenuListFragment : Fragment() {
    // 大画面かどうかの判定フラグ。
    private var _isLayoutXLarge = true

    // onActivityCreatedメソッドはver1.3以降非推奨になった。
    // 以下、書籍ではonActivityCreatedメソッドを使用する。
    override fun onViewCreated(view: View,savedInstanceState: Bundle?) {
        // 親クラスのメソッド呼び出し。
        super.onViewCreated(view,savedInstanceState)
        // 自分が所属するアクティビティからmenuThanksFrameを取得。
        // ここの処理でnullになる。
        val menuThanksFrame = activity?.findViewById<View>(R.id.menuThanksFrame)
        // menuThanksFrameがnull,つまり存在しないなら・・・。
        if (menuThanksFrame == null) {
            // 画面判定フラグを通常画面とする。
            _isLayoutXLarge = false
        }
    }


    override fun onCreateView(
        inflater: LayoutInflater, container: ViewGroup?,
        savedInstanceState: Bundle?
    ): View? {
        // フラグメントで表示する画面をXMLファイルからインフレートする。
        val view = inflater.inflate(R.layout.fragment_menu_list, container, false)
        // 画面部品ListViewを取得。
        val lvMenu = view.findViewById<ListView>(R.id.lvMenu)

        // SimpleAdapterで使用するMutableListオブジェクトを用意。
        val menuList: MutableList<MutableMap<String, String>> = mutableListOf()

        // 「から揚げ定食」のデータを格納するMapオブジェクトの用意とmenuListへのデータ登録。
        var menu = mutableMapOf("name" to "から揚げ定食", "price" to "800円")
        menuList.add(menu)
        // 「ハンバーグ定食」のデータを格納するMapオブジェクトの用意とmenuListへのデータ登録。
        menu = mutableMapOf("name" to "ハンバーグ定食", "price" to "850円")
        menuList.add(menu)
        // 以下データ登録の繰り返しのため省略。

        // SimpleAdapter第4引数from用のデータの用意。
        val from = arrayOf("name", "price")
        // SimpleAdapter第5引数to用のデータの用意。
        val to = intArrayOf(android.R.id.text1, android.R.id.text2)
        // SimpleAdapterを生成。
        val adapter = SimpleAdapter(activity, menuList,
            android.R.layout.simple_list_item_2, from, to)
        // アダプタの登録。
        lvMenu.adapter = adapter

        // リスナの登録。
        lvMenu.onItemClickListener = ListItemClickListener()

        // インフレートされた画面を戻り値として返す。
        return view
    }

    private inner class ListItemClickListener : AdapterView.OnItemClickListener {
        override fun onItemClick(parent: AdapterView<*>, view: View, position: Int, id: Long) {
            // タップされた行のデータを取得。SimpleAdapterでは1行分のデータはMutableMap型!
            val item = parent.getItemAtPosition(position) as MutableMap<String, String>
            // 定食名と金額を取得。
            val menuName = item["name"]
            val menuPrice = item["price"]

            // 引継ぎデータをまとめて格納できるBundleオブジェクトを生成。
            val bundle = Bundle()
            // Bundleオブジェクトに引継ぎデータを格納。
            bundle.putString("menuName", menuName)
            bundle.putString("menuPrice", menuPrice)

            // 大画面の場合。
            if (_isLayoutXLarge) {
                // フラグメントトランザクションの開始。
                // 以下のfragmentManagerは非推奨になった。
                // val transaction = fragmentManager?.beginTransaction()
                // parentFragmentManagerを使うことが推奨になった。
                // parentFragmentManagerはnullチェックがある為。
                val transaction = parentFragmentManager?.beginTransaction()
                // 注文完了フラグメントを生成。
                val menuThanksFragment = MenuThanksFragment()
                // 引継ぎデータを注文完了フラグメントに格納。
                menuThanksFragment.arguments = bundle
                // 生成した注文完了フラグメントをmenuThanksFrameレイアウト部品に追加（置き換え）。
                transaction?.replace(R.id.menuThanksFrame, menuThanksFragment)
                // フラグメントトランザクションのコミット。
                transaction?.commit()
            }
            // 通常画面の場合。
            else {
                // インテントオブジェクトを生成。
                // activityは現在所属するアクティビティを示す（今回はMainActivity）。
                val intent2MenuThanks = Intent(activity, MenuThanksActivity::class.java)
                // 第2画面に送るデータを格納。
                intent2MenuThanks.putExtras(bundle)
                // 第2画面の起動。
                startActivity(intent2MenuThanks)
            }
        }
    }
}

該当のソースコード2

activity_main.xml(xlarge)

<?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"
    android:baselineAligned="false"
    android:orientation="horizontal">

    <fragment
        android:id="@+id/fragmentMenuList"
        android:name="com.websarva.wings.android.fragmentsample.MenuListFragment"
        android:layout_width="0dp"
        android:layout_height="match_parent"
        android:layout_weight="0.4"/>

    <FrameLayout
        android:id="@+id/menuThanksFrame"
        android:layout_width="0dp"
        android:layout_height="match_parent"
        android:layout_marginRight="10dp"
        android:layout_marginLeft="50dp"
        android:layout_weight="0.6"
        android:background="?android:attr/detailsElementBackground" />
</LinearLayout>

試したこと

検索して以下のサイトを見つけたので、onActivityメソッドではなく、onViewCreatedメソッドで実装しました。
https://minpro.net/onactivitycreated-deprecated

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

Fragmentライブラリのバージョンは1.3.6で使用しています。

（追記）スマートフォンサイズのレイアウトファイルは以下の通りになります。
activity_main.xml

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

    <fragment
        android:id="@+id/fragmentMenuList"
        android:name="com.websarva.wings.android.fragmentsample.MenuListFragment"
        android:layout_width="match_parent"
        android:layout_height="match_parent"/>
</FrameLayout>

Comments

[jimbe]

どこで「タブレットサイズだと認識」しているのでしょうか。

[timobol]

質問ありがとうございます。
_isLayoutXLargeという変数がtrueの場合、タブレットサイズであるという認識です。

[jimbe]

> _isLayoutXLargeという変数がtrueの場合
それは MenuListFragment 内でのお話ですね。

通常(?)、画面の大きさに合わせて画面構造を変える場合、予めリソースに各大きさ(や向き)に合わせたレイアウトを作成しておくと、OS が自動でレイアウトを選択してくれます。

ですので、もし「スマホサイズ」のレイアウトには R.id.menuThanksFrame が無く、「タブレットサイズ」のレイアウトには R.id.menuThanksFrame が有るというのであれば、タブレットで実行しているのに R.id.menuThanksFrame が無い=そのタブレットのサイズに合うレイアウトを OS が見つけられず、デフォルトとして「スマホサイズ」のレイアウトが使用されている・・・と見ることが出来ます。

どのような環境で実行されての結果か分かりませんが、その環境に合うタブレットサイズのレイアウトは定義されているのでしょうか。

[timobol]

今回該当のソースコード2に記載されているXMLがactivity_main.xml(xlarge)となっており、タブレットサイズ用のレイアウトになっております。
スマホサイズ用には別途activity_main.xmlのレイアウトファイルがあり、その中ではmenuThanksFrameの項目はありません。

[timobol]

補足情報の欄にスマホサイズのレイアウトファイルactivity_main.xmlを追記いたしました。

[timobol]

実行については、タブレットはPixel Cのエミュレータを起動させて動作確認しております。
スマホはPixel 4で実行しております。

Answer 1

再現しました。

結論から言いますと、レイアウトにフラグメントを直接書いた場合、フラグメントの onViewCreated の実行される段階では、まだアクティビティにレイアウトが全ては設定されていません。

ですので、レイアウトにフラグメントを直接書くのではなくホルダだけとし、アクティビティの onCreate で FragmentManager を利用してホルダにフラグメントを展開しますと、onViewCreated でアクティビティの他の View を獲得できます。

ここからは私の考えですが、折角フラグメントを利用するのですから、スマホサイズ時も別アクティビティに遷移するのではなくフラグメントの切り替えで対応し、かつ、フラグメントから他の View に直接アクセスするのではなく、 ViewModel を用いて間接的に処理を依頼・データを取得するほうが、他の View から独立しますのでコード上分かり易くなるかと思います。

Comments

[timobol]

>結論から言いますと、レイアウトにフラグメントを直接書いた場合、フラグメントの onViewCreated の実行される段階では、まだアクティビティにレイアウトが全ては設定されていません。
なるほど。そうなのですね…。

初歩的な質問で申し訳ないのですが、ホルダとはどういうものなのでしょうか？
調べてみたのですがわかなかったです。
MenuListFragment.ktの中でonCreateをoverrideして、FragmentManagerを使うという認識なのですが、合っていますでしょうか？ホルダの展開方法がわからなかったので参考になるものがあるとありがたいです。

[jimbe]

ホルダと言ったのは、ご質問のコードでも使われている「Fragment を置く場所として書いているモノ」のことです。id:menuThanksFrame の役割のことですね。

id:menuThanksFrame に MenuThanksFragment を replace しているのと同じように、 今はスマホ/タブレットとも直接(fragment タグの) MenuListFragment を書いている所を例えば FrameLayout タグの id:menuListFrame とし、onCreate で MenuListFragment を replace する形とすれば、ご質問の状態にはならないと思います。

[jimbe]

以下に「私の考え」で改造した AndroidStudio プロジェクトを入れてみましたので、良かったら参考にしてみてください。

https://github.com/Jimbe-github/teratail-q362992

コード: https://github.com/Jimbe-github/teratail-q362992/tree/master/app/src/main/java/com/teratail/q362992
レイアウト: https://github.com/Jimbe-github/teratail-q362992/tree/master/app/src/main/res/layout
https://github.com/Jimbe-github/teratail-q362992/tree/master/app/src/main/res/layout-xlarge

[timobol]

少し内容が理解できました。
アクティビティのonCreateを以下のようにしたところ、変数menuThanksFrameがnullにならず想定していた動きになりました！
MainActivity.kt
```
package com.websarva.wings.android.fragmentsample

import androidx.appcompat.app.AppCompatActivity
import android.os.Bundle
import androidx.fragment.app.FragmentManager

class MainActivity : AppCompatActivity() {
override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState)
setContentView(R.layout.activity_main)
if (savedInstanceState == null) {
val fragmentManager = supportFragmentManager
val transaction = fragmentManager.beginTransaction()
transaction?.replace(R.id.fragmentMenuList,MenuListFragment())
transaction?.commit()
}
}
}
```
（最初MenuListFragmentクラスのonCreateメソッドをオーバーライドするのだと勘違いしていました。申し訳ありません。）

タブレット時にフラグメントを使った処理に現在はなっていますが、jimbeさんのおっしゃる通りにスマホ時もフラグメントでの処理するように統一した方がいいと思いました。
まだ書籍で学習中の身なので、ひと段落したら色々改良してみたいと思います！
私のような初心者に様々なアドバイスをしていただきありがとうございました。
今後ともよろしくお願いいたします。
今回jimbeさんの回答をベストアンサーにさせていただきます！

[timobol]

jimbeさんのコードも後で見てみて参考にしてみたいと思います。
まだViewModelの概念も理解できていないので、これからも勉強を続けていきたいと思います。
何から何までありがとうございます！