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下面的片段中，将启动一个Observable计时器，在该计时器到达3秒钟时，将实例化适配器issueAdapter。然后，在调用onDestroyView时，它将进行一些清理（例如，将issueAdapter.setOnKeyListener设置为null）

以编程方式，仅应在30秒后销毁该片段（实例化给issueAdapter的时间）。这是大约一年的生产以来我第一次收到该异常。

我的问题是，是否可能缺少某些生命周期感知的代码处理程序？

不好。：该设备不允许旋转。

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

旋转并不是唯一可能触发您的Fragment销毁和重新创建的配置更改。此外，即使不进行配置更改，也可以销毁Fragment。也许用户打开此屏幕，然后立即接到电话，他们的设备终止了您的应用程序保存资源的过程。

总而言之，即使您认为onDestroyView()在三秒钟结束之前都不应被调用，您仍然需要处理事实。

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

Answer 2

您可以为适配器设置一个安全条件吗，因为在所有过程成功时该对象都存在，但在错误情况下issueAdapter为空，因为未找到实例。我建议您将onDestroy放进去

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

Answer 3

正如Ben P.所说，空检查将是简化列表解决方案。如果需要确保未清除issueAdapter，可以将以下行添加到片段的OnCreate方法中。

setRetainInstance(true);

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

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

3 个答案: