getViewLifecycleOwner()でLifecycleOwnerオブジェクトが取得できない

Question

前提・実現したいこと

Androidアプリを開発しています。
ViewModelとLiveDataを実装してデータ共有したいです。

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

LiveDataのObserverの設定が上手くいきません。
具体的にはMutableLiveDataのobseveメソッドの第一引数にLifecycleOwnerオブジェクトを入れるため
getViewLifecycleOwner()にて取得を試みているのですがNullが返ってくるのっだと思います。(CenterFragment.javaのコメント行の箇所)
なぜNullが返ってくるのでしょうか？
ライフサイクル的にNullが返る状況ではないと思いますが、、、

エラーメッセージ
2020-05-05 17:33:21.125 10181-10181/? E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.example.myapplication_ver2, PID: 10181
    java.lang.IllegalStateException: Can't access the Fragment View's LifecycleOwner when getView() is null i.e., before onCreateView() or after onDestroyView()
        at androidx.fragment.app.Fragment.getViewLifecycleOwner(Fragment.java:328)
        at com.example.myapplication_ver2.CenterFragment.onCreate(CenterFragment.java:25)
        at androidx.fragment.app.Fragment.performCreate(Fragment.java:2684)
        at androidx.fragment.app.FragmentStateManager.create(FragmentStateManager.java:270)
        at androidx.fragment.app.FragmentManager.moveToState(FragmentManager.java:1173)
        at androidx.fragment.app.FragmentTransition.addToFirstInLastOut(FragmentTransition.java:1255)
        at androidx.fragment.app.FragmentTransition.calculateFragments(FragmentTransition.java:1138)
        at androidx.fragment.app.FragmentTransition.startTransitions(FragmentTransition.java:136)
        at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:1987)
        at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:1945)
        at androidx.fragment.app.FragmentManager.execPendingActions(FragmentManager.java:1847)
        at androidx.fragment.app.FragmentManager.dispatchStateChange(FragmentManager.java:2621)
        at androidx.fragment.app.FragmentManager.dispatchActivityCreated(FragmentManager.java:2569)
        at androidx.fragment.app.FragmentController.dispatchActivityCreated(FragmentController.java:247)
        at androidx.fragment.app.FragmentActivity.onStart(FragmentActivity.java:541)
        at androidx.appcompat.app.AppCompatActivity.onStart(AppCompatActivity.java:201)
        at android.app.Instrumentation.callActivityOnStart(Instrumentation.java:1391)
        at android.app.Activity.performStart(Activity.java:7157)
        at android.app.ActivityThread.handleStartActivity(ActivityThread.java:2937)
        at android.app.servertransaction.TransactionExecutor.performLifecycleSequence(TransactionExecutor.java:180)
        at android.app.servertransaction.TransactionExecutor.cycleToPath(TransactionExecutor.java:165)
        at android.app.servertransaction.TransactionExecutor.executeLifecycleState(TransactionExecutor.java:142)
        at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:70)
        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1808)
        at android.os.Handler.dispatchMessage(Handler.java:106)
        at android.os.Looper.loop(Looper.java:193)
        at android.app.ActivityThread.main(ActivityThread.java:6669)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)
    
    

該当のソースコード

CenterFragment.java

package com.example.myapplication_ver2;

import android.graphics.Color;
import android.os.Bundle;
import android.util.Log;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.Button;
import android.widget.LinearLayout;
import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.fragment.app.Fragment;
import androidx.lifecycle.LifecycleOwner;
import androidx.lifecycle.MutableLiveData;
import androidx.lifecycle.Observer;
import androidx.lifecycle.ViewModelProviders;

public class CenterFragment extends Fragment {

    @Nullable
    @Override
    public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) {

        LinearLayout linearLayout = new LinearLayout(getContext());
        linearLayout.setBackgroundColor(Color.BLUE);
        container.addView(linearLayout);

        Button button = new Button(getContext());
        linearLayout.addView(button);
        return super.onCreateView(inflater, container, savedInstanceState);
    }

    @Override
    public void onActivityCreated(@Nullable Bundle savedInstanceState) {

        super.onActivityCreated(savedInstanceState);
        ViewModelA viewModelA = ViewModelProviders.of(this).get(ViewModelA.class);
        viewModelA.getList().observe(getViewLifecycleOwner(), new Observer<Integer>() { //恐らくgetViewLifecycleOwner()でnullが返ってくる事によるエラー
            @Override
            public void onChanged(Integer s) {
            }
        });

    }
}

MainActivity.java

package com.example.myapplication_ver2;

import androidx.appcompat.app.AppCompatActivity;
import androidx.constraintlayout.widget.ConstraintLayout;
import androidx.fragment.app.Fragment;
import androidx.fragment.app.FragmentManager;
import androidx.fragment.app.FragmentTransaction;

import android.os.Bundle;
import android.widget.LinearLayout;

public class MainActivity extends AppCompatActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        FragmentManager fragmentManager = getSupportFragmentManager();
        FragmentTransaction fragmentTransaction = fragmentManager.beginTransaction();
        fragmentTransaction.add(R.id.layout,new CenterFragment());
        fragmentTransaction.commit();
    }
}

ViewModelA.java

package com.example.myapplication_ver2;

import androidx.lifecycle.LiveData;
import androidx.lifecycle.MutableLiveData;
import androidx.lifecycle.ViewModel;

public class ViewModelA extends ViewModel {
    private MutableLiveData<Integer> liveData;

    public MutableLiveData<Integer> getList(){
        if(liveData==null){
            liveData = new MutableLiveData();
        }
        return liveData;
    }

}

Comments

[nakasho_dev]

FragmentのViewがnullだからエラーになっていると思われます。エラーが発生しているタイミングが分かりませんが、起動時ならばonCreateViewの結果がnullの可能性もありそうです。デバッグ実行で確認してみてはいかがでしょうか？そもそも＠Nullableって必要でしたっけ？

[I_LIKE_EGGS]

問題点は私もそこだと思ってます。
観察すると
FragmentのgetViewLifecycleOwner()で最初のif (mViewLifecycleOwner == null)でtrueの為IllegalStateExceptionを吐いているようでした。
mViewLifecycleOwner がnullという事はViewが作成されていないのだと思います。

---Fragment.java---326行目
public LifecycleOwner getViewLifecycleOwner() {
if (mViewLifecycleOwner == null) {
throw new IllegalStateException("Can't access the Fragment View's LifecycleOwner when "
+ "getView() is null i.e., before onCreateView() or after onDestroyView()");
}
return mViewLifecycleOwner;
}
------

＠NullableはonActivityCreatedをoverrideする際、Androidstudioが勝手につけるのでいつもそのまま使っています。ここに問題がありますかね、、？

[nakasho_dev]

＠Nullableは問題なかったようです。すいません。
デバッグ実行してonCreateViewの最終行の実行がnullになるのであればContainerに格納しているViewに問題がありそうですね。

[I_LIKE_EGGS]

デバックで追ってみたら　super.onCreateView(inflater, container, savedInstanceState)でnullが返っている事によりonCreateViewの返り値がnullになり、その後mViewLifecycleOwner = nullが挿入され問題を引き起こしていました><
nullを返らせない為にはFragment作成時（つまりnew Fragment()）引数を与えなければならないようです。
いつもFragment作成時引数なしでやっていた為、何を与えたらいいのかわかりません。
もしわかればよろしくお願いいたしますm(__)m

[nakasho_dev]

根本的な解決になるか分かりませんがレイアウトXMLファイルをinflaterで読み込む方式だとどうですか？

[I_LIKE_EGGS]

すみません。今androidデベロッパーViewModelのサンプルを見ていたら
ViewModelA viewModelA = ViewModelProviders.of(this).get(ViewModelA.class);
のof(this)の部分がof(getActivity())になっており、
observe()の第一引数もgetActivity()になっておりました。

最初個人ブログのサンプルで ViewModelA viewModelA = ViewModelProviders.of(this).get(ViewModelA.class);となっており、
そちらを入れたら、その事が原因かわかりませんが、
observe()の第一引数にthisが入らず、AndroidstudioにgetViewLifecycleOwner()に置き換えろと怒られた為、getViewLifecycleOwner()を挿入し、今回のエラーにぶち当たり、質問しました。
もしかしたら根本的違うかもしれません。
質問はまだ閉じませんが、色々試行錯誤してみます。

[I_LIKE_EGGS]

すみません、自己解決しました為解決方法を自己解決欄に記載しておきます。
ありがとうございます。

[nakasho_dev]

自己解決したようで良かったです。ちなみにViewModelProviders.ofはDeprecatedになっているので気をつけてください。

[I_LIKE_EGGS]

ですねf^^いつかここの部分でエラーを起こすかもしれないので気を付けるようにしますf^^

Answer 1

すでに解決済みにされていますが、気になったので回答します。

Fragment::getViewLifecycleOwnerがnullを返す場合、2通りの理由が考えられます。

1. onCreateViewでnullを返している（UIを持たないFragment）
2. onCreateViewの前、もしくはonDestroyView後にgetViewLifecycleOwnerを呼び出している

ご提示のコードの場合は、1.のパターンになっています。

public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) {
    ...
    container.addView(linearLayout);
    ...
    return super.onCreateView(inflater, container, savedInstanceState);
}

上記のコードはFragmentの使い方として不適切ですので、container.addView(linearLayout)は削除して、linearLayoutをreturnするように変更してください。そうすればlinearLayoutがFragmentのViewとして設定され、getViewLifecycleOwnerが利用できるようになるはずです。

また、Fragment内でobserveの第1引数にgetActivity()を渡すのはメモリリークの原因となるのでご注意ください。基本的には自身のライフサイクルに基づくLifecycleOwnerを渡すべきです。

Comments

[I_LIKE_EGGS]

container.addView(linewarLayout)は不適切な使い方なんですね！参考になりました。ありがとうございます。
LinearLayoutをreturnする方法もあるんですね（っていうかそれが本来の使い方か…）
observeの第1引数の件も参考にさせて頂きます。ありがとうございます！

Answer 2

そもそも今回FragmentAとFragmentBとFragmentが二つありFragment間でデータを連動させていました。
そもそもFragment間でデータをやり取りする際ViewModel取得の設定でof(this)ではなくof(getActivity())としなければならなかったようです。

of(this)としてその後、observe()の第一引数をgetActivity()としていた為、エラーを吐かずデータが連動しなかった為はまってしまいました。

そしてgetActivity()の部分が違うのかと思い込みgetViewLifecycleOwner()にしたところエラーを吐き今回の質問させて頂きました。

そもそも問題点がobserve()の設定ではあらず、ViewModel取得の設定でof()部分だった為、みるところを根本的に間違えてしまいました。
以上のような経緯により問題の解決にいたりました。

ちなみに
of(this)でもof(getActivity())でもその後observeメソッドの第一引数にgetViewLifecycleOwner()を入れてしまうとエラーを吐きます。

public LifecycleOwner getViewLifecycleOwner() {
if (mViewLifecycleOwner == null) {
throw new IllegalStateException("Can't access the Fragment View's LifecycleOwner when "
+ "getView() is null i.e., before onCreateView() or after onDestroyView()");
}
return mViewLifecycleOwner;
}
でmViewLifecycleOwner が nullになっている事によりif文に捕まってしまうためです。

第一引数にgetActivity()を入れるとエラーは吐きませんが、Fragment間のデータの連動が上手くいきません。
ActivityとViewModelの連携の場合of(this)にするのだと思います。