片段的布局字段在初始化时为 NULL答案

【问题标题】：Layout fields of fragment are NULL on initialization片段的布局字段在初始化时为 NULL
【发布时间】：2018-10-22 12:50:15
【问题描述】：

我在将数据传递给片段时遇到问题。它在生产中崩溃了 0.1%。假设在 100k 活动开始时，它发生了 100 次。它看起来不是很常见，但它非常困扰我，我认为我在使用数据初始化片段时做错了。问题是，我只创建一次片段，而所有其他时间我都需要将数据传递给他们，我正在做下一个方法：myFragmentInstance.setData(Object someData); 崩溃发生是因为它告诉片段中的那些视图元素没有找到并且它们是 NULL，但如果我没有重新创建它们，一切都应该没问题。我没有旋转手机，或者没有足够的内存。它发生在网络重新连接时，因为在网络重新连接时，我要去服务器获取新数据，然后将新数据设置到我的片段中。我有我使用的两个片段的字段的照片，也许你们中的一些人知道这些数据可以说明崩溃时片段的状态。我正在使用库ButterKnife 来初始化片段和活动的字段，而不是使用findById 来初始化它，可能它有一些影响或没有？

这里是简单项目的链接（github上只有这个问题）： https://github.com/yozhik/Reviews/tree/master/app/src/main/java/com/ylet/sr/review

说明：

CollapsingActivity - 与Collapsing AppBarLayout 的活动。它将一个或两个片段加载到“fragment_content_holder”中，并且它有TabLayout 在视图寻呼机中的片段之间切换。

在活动方法onCreate() - 我只是在模拟对服务器的请求 (loadData)，当加载了一些假数据时 - 我在第一次调用时在视图寻呼机中显示片段 - 我正在创建新的 @987654339 @ 扩展 FragmentPagerAdapter，用片段填充它并保存到实例的链接。在下一次通话中 - 我不会从头开始创建片段，而是用新数据填充它们。

MenuFragment1, MenuFragment1 - 两个片段。 MenuFragment1 - 有 public void setupData(SomeCustomData data) 方法，用于设置新数据，而不是在网络重新连接时重新创建片段。

NetworkStateReceiver - 监听网络变化并发送通知。

TabMenuAdapter - 只是保存片段的简单类。

05-11 18:11:05.088 12279-12279/com.myProjectName E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.myProjectName, PID: 12279
    java.lang.IllegalStateException: Fatal Exception thrown on Scheduler.
        at io.reactivex.android.schedulers.HandlerScheduler$ScheduledRunnable.run(HandlerScheduler.java:111)
        at android.os.Handler.handleCallback(Handler.java:739)
        at android.os.Handler.dispatchMessage(Handler.java:95)
        at android.os.Looper.loop(Looper.java:135)
        at android.app.ActivityThread.main(ActivityThread.java:5268)
        at java.lang.reflect.Method.invoke(Native Method)
        at java.lang.reflect.Method.invoke(Method.java:372)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:902)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:697)
     Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.TextView.setText(java.lang.CharSequence)' on a null object reference
        at com.yozhik.myProjectName.view.fragments.MyFinalTermsFragment.setupMyInformation(MyFinalTermsFragment.java:145)
        at com.yozhik.myProjectName.view.fragments.MyFinalTermsFragment.setupWithData(MyFinalTermsFragment.java:133)
        at com.yozhik.myProjectName.view.activity.MyFinalActivity.onDataLoaded(MyFinalActivity.java:742)
        at com.yozhik.myProjectName.presenter.MyFinalPresenter$1.onNext(MyFinalPresenter.java:55)
        at com.yozhik.myProjectName.presenter.MyFinalPresenter$1.onNext(MyFinalPresenter.java:47)
        at io.reactivex.internal.operators.observable.ObservableObserveOn$ObserveOnObserver.drainNormal(ObservableObserveOn.java:200)
        at io.reactivex.internal.operators.observable.ObservableObserveOn$ObserveOnObserver.run(ObservableObserveOn.java:252)
        at io.reactivex.android.schedulers.HandlerScheduler$ScheduledRunnable.run(HandlerScheduler.java:109)
        at android.os.Handler.handleCallback(Handler.java:739) 
        at android.os.Handler.dispatchMessage(Handler.java:95) 
        at android.os.Looper.loop(Looper.java:135) 
        at android.app.ActivityThread.main(ActivityThread.java:5268) 
        at java.lang.reflect.Method.invoke(Native Method) 
        at java.lang.reflect.Method.invoke(Method.java:372) 
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:902) 
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:697) 
05-11 18:11:07.953 2155-3877/? E/WifiStateMachine: Did not find remoteAddress {192.168.200.1} in /proc/net/arp
05-11 18:11:07.966 2155-3877/? E/WifiStateMachine: WifiStateMachine CMD_START_SCAN source -2 txSuccessRate=3800.62 rxSuccessRate=4732.06 targetRoamBSSID=any RSSI=-68
05-11 18:11:07.967 2155-3877/? E/WifiStateMachine: WifiStateMachine L2Connected CMD_START_SCAN source -2 2324, 2325 -> obsolete
05-11 18:11:08.021 2155-3896/? E/ConnectivityService: Unexpected mtu value: 0, wlan0
05-11 18:11:08.579 13514-13366/? E/WakeLock: release without a matched acquire!

在 setupData 方法中崩溃的片段，因为 data_1_txt 有时为 NULL。

public class MenuFragment1 extends Fragment {

    public SomeCustomData transferedDataFromActivity;
    private TextView data_1_txt;

    public static MenuFragment1 newInstance(SomeCustomData data) {
        Log.d("TEST", "MenuFragment1.newInstance");
        MenuFragment1 fragment = new MenuFragment1();

        Bundle args = new Bundle();
        args.putSerializable("DATA_FROM_ACTIVITY", data);
        fragment.setArguments(args);

        return fragment;
    }

    @Override
    public void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        Log.d("TEST", "MenuFragment1.onCreate");

        if (getArguments() != null) {
            this.transferedDataFromActivity = (SomeCustomData) getArguments().getSerializable("DATA_FROM_ACTIVITY");
        }
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        Log.d("TEST", "MenuFragment1.onCreateView");
        View v = inflater.inflate(R.layout.menu_fragment_1, container, false);
        data_1_txt = (TextView) v.findViewById(R.id.data_1_txt);

        setupInOnCreateView();

        return v;
    }

    protected void setupInOnCreateView() {
        Log.d("TEST", "MenuFragment1.setupInOnCreateView");
        //initialization of all view elements of layout with data is happens here.
        setupData(transferedDataFromActivity);
    }

    public void setupData(SomeCustomData data) {
        Log.d("TEST", "MenuFragment1.setupData");
        this.transferedDataFromActivity = data;
        if (transferedDataFromActivity != null) {
            data_1_txt.setText(transferedDataFromActivity.Name);
        }
    }
}

片段布局：

<?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:background="@color/green"
    android:orientation="vertical">

    <TextView
        android:id="@+id/data_1_txt"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:background="@color/yellow"
        android:text="Test"
        android:textSize="20sp" />

    <include layout="@layout/description_layout" />

</LinearLayout>

【问题讨论】：

在您的片段中，您实际上是在 onCreateView 中执行 findViewById。尝试在您的 setupData 方法中执行 findViewById（而不是使用变量）。如果它仍然为空，则意味着您的片段已被破坏。顺便说一句，出于这个确切原因，您需要格外小心使用 Thread - 它不支持生命周期
你试过我的答案了吗？只需将getMyData() 替换为您的textview.setText("your string")
@pouya 我还没有尝试过你的答案 - 因为它对我来说看起来像是 hack，我认为这次崩溃的原因是不同的，而且原因是这个 bug 很难解决重现，所以我无法重现它并告诉你它是否已修复。不过还是谢谢。
@DennisK 使用 Thread 是什么意思？我现在没有在代码中使用它。根据文档，关于“findViewById” - onCreateView 是查找视图并在变量中记住它们的地方。而且这个bug很难重现。
@yozhik 是的，事实上它是一种 hack，而且在这方面非常好。但是你能把你的FragmentManager的代码和你打电话给activity的地方贴出来吗？

标签： android android-fragments android-tablayout fragmentpageradapter butterknife

【解决方案1】：

根据我的经验，如果片段中有错误，通常是因为在 viewpager 和 TabMenu 中预加载了片段，所以我所做的并建议您做的是检查片段是否对用户可见如果是，请获取数据和其他内容，这是我的代码：

public class Fragment1 extends Fragment {
boolean visible = false;
public static Fragment1 newInstance() {
    return new Fragment1();
}
@Override
public View onCreateView(LayoutInflater inflater,
                         ViewGroup container, Bundle savedInstanceState) {
    if (visible && isResumed()) {
        onResume();
    } else if (visible) {
        getMyData();
    }
    return rootView;
}
@Override
public void setUserVisibleHint(boolean isVisibleToUser) {
    super.setUserVisibleHint(isVisibleToUser);
    visible = isVisibleToUser;
    if (isVisibleToUser) {
        getMyData();
    }
    else {
    }
}

@Override
public void onViewCreated(View view, @Nullable Bundle savedInstanceState) {
    super.onViewCreated(view, savedInstanceState);
}
}

这样，如果视图尚未创建并且对用户片段不可见，则不会执行任何操作。

希望这会有所帮助。

【讨论】：

@azizbekian 我实际上搜索了很多，以克服在 viewpager 中预加载片段的问题，但这是我想出的最好的。如果你有更好的方法请照亮我。就我而言，我的网络服务被一起调用了。
@azizbekian 是的，如果您有更好的解决方案，请与我们分享，谢谢。
您应该在getActivity() != null 的setUserVisibleHint() 中进行额外检查，因为如果您记录对setUserVisibleHint 的调用，您会看到它被多次调用，值为true，但如果你也记录getActivity()，你会看到当setUserVisibleHint为真时，很多时候getActivity()可能会返回null。
@parekhkruti26 什么样的检查比在实际中看到它更好，我已经做到了，这很有效。 getMyData() 只被调用一次。那是因为我已经检查了对用户的可见性，并且只有在活动不为空时才可见。此外，我正在使用此代码。这可能是一个黑客是的，但它对我来说工作正常。没有错误

【解决方案2】：

我很确定您的问题是由于将对片段的引用保留在数组中。片段具有生命周期，并且不保证引用会持续存在。正如您所说，很难重现和准确追踪出了什么问题，但也许您不需要这样做。关于如何解决此问题的一些建议：

不要存储对片段的引用。几乎遵循 Google 页面 (https://developer.android.com/training/animation/screen-slide) 上的示例，并在每次请求时实例化一个新片段。
如果您担心性能并且缓存正在解决它，请尝试使用 FragmentStatePagerAdapter - 它会缓存页面并管理片段的状态。
如果您需要从主片段（或活动）访问页面片段，而不是存储引用，请使用“findFragmentByTag”，它将始终返回片段的当前活动实例。
李>

【讨论】：

感谢您的回答，我也有这个想法，但是在stackoverflow上我也发现保留对片段的引用是可以的。因为我想知道为什么我必须始终使用“findFragmentByTag”来查找片段，这在我看来就像开销。
顺便说一句，如果在 FragmentStatePagerAdaper 中，我如何“findFragmentByTag” - 我只能提供带有新 MyFragment 的片段？我没有提到任何标签，它是在这个适配器的核心中实现的。而且我需要以某种方式为这些片段设置新的新数据，而不是重新创建它们。
这至少是问题的一部分，所以它得到了我的投票 - 如果你查看 OP 的链接 Github repo here 似乎有手动缓存片段作为活动的字段。您几乎不应该这样做 - Android 操作系统管理片段。为 Fragments 滚动你自己的适配器可能会让你像现在的那样陷入麻烦 ;-)

【解决方案3】：

我坚信事情并不像其他人解释的那样复杂。如果我理解正确的话，网络交易造成的延迟是罪魁祸首。

考虑这种情况。

您正在发出网络请求，该请求会在视图中进行一些更改最后。
您切换寻呼机。（片段是分离的，视图是销毁）
这是来自网络请求的回调。你猜怎么了！崩溃。

因此，在处理片段视图时，更加小心总是一个好主意。我通常这样做。

//in the base class
protected boolean isSafe()
    {
        return !(this.isRemoving() || this.getActivity() == null || this.isDetached()
                || !this.isAdded() || this.getView() == null);
    }

//usage in derived classes
onNewtworkResult(Result result) {
    if(!isSafe())
        return;
    //rest of the code
}

或者，您也可以将潜在代码包装在 try catch 中。但这更像是盲目的射击（至少在这种情况下）。

【讨论】：