片段调用onDestroyView，但对适配器的引用为空

Question

java.lang.NullPointerException: Attempt to invoke virtual method 'void my.app.ui.fragment.adapter.IssueAdapter.setOnKeyListener(my.app.fragment.adapter.IssueAdapter$OnItemListener)' on a null object reference
    at MyFragment.onDestroyView(MyFragment.java:429)
    at android.support.v4.app.Fragment.performDestroyView(Fragment.java:2565)
    at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:1512)
    at android.support.v4.app.FragmentManagerImpl.moveFragmentToExpectedState(FragmentManager.java:1759)
    at android.support.v4.app.BackStackRecord.executeOps(BackStackRecord.java:792)
    at android.support.v4.app.FragmentManagerImpl.executeOps(FragmentManager.java:2596)
    at android.support.v4.app.FragmentManagerImpl.executeOpsTogether(FragmentManager.java:2383)
    at android.support.v4.app.FragmentManagerImpl.removeRedundantOperationsAndExecute(FragmentManager.java:2338)
    at android.support.v4.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:2245)
    at android.support.v4.app.FragmentManagerImpl$1.run(FragmentManager.java:703)
    at android.os.Handler.handleCallback(Handler.java:739)
    at android.os.Handler.dispatchMessage(Handler.java:95)
    at android.os.Looper.loop(Looper.java:145)
    at android.app.ActivityThread.main(ActivityThread.java:6939)
    at java.lang.reflect.Method.invoke(Method.java:-2)
    at java.lang.reflect.Method.invoke(Method.java:372)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1404)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1199)

在下面的片段中，onCreateView将启动一个可观察的计时器，当计时器达到3秒时，适配器issueAdapter将被实例化。然后，当调用onDestroyView时，它将执行一些清理(例如，将issueAdapter.setOnKeyListener设置为null )

在编程上，片段应该只在30秒后被销毁(将issuesAdapter实例化的时间留下来)。这可能是我在一年的生产中第一次收到这样的异常。

我的问题是，有没有可能我遗漏了一些生命周期感知的代码处理程序？

Obs:该设备不允许旋转。

private final CompositeDisposable disposables = new CompositeDisposable();
private IssueAdapter issueAdapter;    

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
    ViewGroup rootView = (ViewGroup) inflater.inflate(my_fragment, container, false);
    unbinder = ButterKnife.bind(this, rootView);

    disposables.add(
            Observable
                    .timer(3, TimeUnit.SECONDS)
                    .subscribeOn(Schedulers.io())
                    .observeOn(AndroidSchedulers.mainThread())
                    .subscribe(__ -> {
                        List<Issue> issues = AppRepository.getInstance().getIssues();
                        issueAdapter = new IssueAdapter(issues);
                        issueAdapter.setOnKeyListener(position -> {
                            this.issueSelected = issues.get(position);
                        });
                    }, ErrorService::notify)
    );

    return rootView;
}

@Override
public void onDestroyView() {
    super.onDestroyView();

    issueAdapter.setOnKeyListener(null);
    unbinder.unbind();
}

@Override
public void onDestroy() {
    super.onDestroy();

    disposables.clear();
}

谢谢

Answer 1

旋转不是唯一可能触发碎片销毁和重新创建的配置更改。此外，即使没有配置更改，碎片也可能被销毁；可能用户打开此屏幕，然后立即收到电话，他们的设备终止了您的应用程序的进程，以节省资源。

总而言之，即使你认为在三秒钟之前“不应该”调用onDestroyView()，你仍然需要处理它可以被调用的事实。

在这种情况下似乎很简单:只需检查issueAdapter是否为null，然后再尝试清空其键侦听器。

Answer 2

你能给适配器设置一个安全条件吗，因为在所有进程都成功的时候对象是存在的，但是在错误的情况下，issueAdapter是一个空的，因为找不到实例。我建议你在你的onDestroy put中

if(issueAdapter!=null){
   issueAdapter.setOnKeyListener(null);
}

Answer 3

正如Ben P.所说，空检查将是最简单的解决方案。如果需要确保issueAdapter没有被清除，可以在片段中的OnCreate方法中添加以下代码行。

setRetainInstance(true);

这将确保在重新创建片段时使用相同的实例(具有实例变量)。如果您不需要确保您的issueHandler不会被设置为null，那么我会避免使用它。