【问题标题】:NullPointerException on Fragment Listener片段侦听器上的 NullPointerException
【发布时间】:2016-05-09 13:48:32
【问题描述】:

我有一个带有 ViewPager 的 Activity,在 ViewPager 适配器中我为每个位置提供 Fragment。

一个示例 Fragment 是 DebugFragment。我已经写了下面的源代码。

public class DebugFragment extends android.support.v4.app.Fragment {

private OnFragmentInteractionListener mListener;

public interface OnFragmentInteractionListener {
    void onFragmentInteraction(int someValue);
}

public static DebugFragment newInstance() {
    DebugFragment fragment = new DebugFragment();
    Bundle args = new Bundle();
    fragment.setArguments(args);
    return fragment;
}

private BroadcastReceiver mMessageReceiver = new BroadcastReceiver() {
    @Override
    public void onReceive(Context context, Intent intent) {
        mListener.onFragmentInteraction(0);
    }
};

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    LocalBroadcastManager.getInstance(getContext()).registerReceiver(mMessageReceiver,
            new IntentFilter("com.android.example.INITIAL_REQUEST"));
}

@Override
public void onDestroy() {
    LocalBroadcastManager.getInstance(getContext()).unregisterReceiver(mMessageReceiver);
    super.onDestroy();
}

@Override
public void onAttach(Context context) {
    super.onAttach(context);
    if (context instanceof OnFragmentInteractionListener) {
        mListener = (OnFragmentInteractionListener) context;
    } else {
        throw new RuntimeException(context.toString()
                + " must implement OnFragmentInteractionListener");
    }
}

@Override
public void onDetach() {
    super.onDetach();
    mListener = null;
}

@Override
public void onResume() {
    super.onResume();
    getUserData();
}

public void getUserData() {
 // Inside Background Thread
    if (getActivity() == null) {
        return;
    }
    getActivity().runOnUiThread(new Runnable() {
        @Override
        public void run() {
            mListener.onFragmentInteraction(0); // This line throws NPE
        }
    });
}

下面是我的 Activity 实现。

public class DebugActivity extends AppCompatActivity implements
    DebugFragment.OnFragmentInteractionListener {

    // Other Activity Callback

    @Override
    public void onActivityResult(int requestCode, int resultCode, Intent data) {
        super.onActivityResult(requestCode, resultCode, data);
        if (requestCode == REQUEST_CODE) {
            if (resultCode == Activity.RESULT_OK) {
                DebugFragment debugFragment = ((DebugFragment) mViewPagerAdapter.getRegisteredFragment(2));
                if (debugFragment != null) {
                    debugFragment.getUserData();
                }
            }
        }
    }
}

我从 Fragment 的 onResume、BroadcastReceiver、Activity 的 OnActivityResult 调用我的 DebugFragment 的 getUserData。

有时我在尝试访问 FragmentListener(即 mListener)时在 getUserData 中得到 NullPointerException。 我想知道为什么?

因为我已经在检查 Activity 是否为空。这还不够吗。我还必须检查 mListener 的 null 吗?如果有人能向我解释活动不会为空但我的 mListener 将为空的情况,那就太好了。我只将我的活动保持在纵向模式。

编辑

我的适配器代码

public abstract class TabPagerAdapter extends FragmentPagerAdapter {

    public TabPagerAdapter(FragmentManager fm) {
        super(fm);
    }

    public abstract View getTabView(int position);
}

public class SecondaryPagerAdapter extends TabPagerAdapter {

    private static final int NUM_PAGES = 5;

    private String tabTitles[] = new String[] { "Today", "New", "Calendar", "In-progress", "Invoices" };
    private int[] imageResId = { R.drawable.ic_tab_hired_pro, R.drawable.ic_tab_history,
            R.drawable.ic_tab_today, R.drawable.ic_tab_inprogress, R.drawable.ic_tab_invoices };
    SparseArray<Fragment> registeredFragments = new SparseArray<Fragment>();

    public SecondaryPagerAdapter(FragmentManager fm) {
        super(fm);
    }

    @Override
    public Fragment getItem(int position) {
        switch (position) {
            case 0:
            case 1:
            case 3:
                return ServiceRequestFragment.newInstance(tabTitles[position]);
            case 2:
                return DebugFragment.newInstance();
            case 4:
                return InvoicesFragment.newInstance();
            default:
                throw new RuntimeException("No fragment for this position");
        }
    }

    @Override
    public int getCount() {
        return NUM_PAGES;
    }

    @Override
    public View getTabView(int position) {
        CustomTab customTab = new CustomTab(DashBoardActivity.this);
        customTab.bindWith(imageResId[position], tabTitles[position]);
        return customTab;
    }

    @Override
    public Object instantiateItem(ViewGroup container, int position) {
        Fragment fragment = (Fragment) super.instantiateItem(container, position);
        registeredFragments.put(position, fragment);
        return fragment;
    }

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        registeredFragments.remove(position);
        super.destroyItem(container, position, object);
    }

    public Fragment getRegisteredFragment(int position) {
        return registeredFragments.get(position);
    }
}

从我的活动中,我称之为

mSecondaryPagerAdapter = new SecondaryPagerAdapter(getSupportFragmentManager());
mSecondaryPager = (ViewPager) findViewById(R.id.dashboard_pager);
mSecondaryPager.setOffscreenPageLimit(4);
mSecondaryPager.setAdapter(mSecondaryPagerAdapter);

【问题讨论】:

  • 您的片段是如何创建的?你可以发布代码吗?我怀疑在您的活动被系统杀死后重新启动您的应用程序时您有 NPE。是这样吗?
  • @Elye 我已经添加了片段创建的代码。我使用 viewPager 适配器中的静态方法 NewInstance()。我从用户的设备收到崩溃报告。我不能确切地说出用户是什么导致了这次崩溃。你说的可能就是这样。
  • 也许你可以分享你的适配器代码?
  • @Elye 我已经添加了我的适配器代码以及我如何从活动中使用它。
  • 我觉得没问题。没看出有什么毛病。你可以查看stackoverflow.com/questions/7951730/… 看看你是否能发现任何东西。一种选择是退出日志并查看崩溃发生时特定片段再次实例化的次数。如果不止一次,那么也许您已经“重新创建”了一个片段,而系统已经恢复了一个“保存的实例”片段......那么问题几乎就在这条线上。

标签: android android-fragments android-fragmentactivity android-lifecycle fragment-lifecycle


【解决方案1】:

当您调用 getActivity().runOnUiThread(new Runnable() {}) 时,这会将 Runnable 排入队列以在 UI 线程上运行之后所有其他已在 UI 线程上排入队列的内容,其中可能包括对 onDestroy() 的调用,这将将mListener 设置为空。这意味着您的应用程序可能会在广播进入时关闭。虽然这在正常情况下应该不是问题,因为您在清除 mListener 之前取消注册您的接收器,但 Runnable 可能已在之后排队this,这就是监听器在执行时为空的原因。

为避免 NPE,您应该在 Runnable 中检查 mListener == null。但是,这仍然意味着回调是在您的 Fragment 被销毁后安排的,因此,您的 Fragment 实例被泄露。最好的办法是创建一个 Handler 并将 Runnable 发布给它,而不是调用runOnUiThread()。然后,在onDestroy() 中调用mHandler.removeCallbacksAndMessages(null),这实质上是清空了队列,因此根本不会调用Runnable。

【讨论】:

    【解决方案2】:
    mMessageReceiver = new BroadcastReceiver() {
        @Override
        public void onReceive(Context context, Intent intent) {
          if(mListener != null){
            mListener.onFragmentInteraction(0);
          }
        }
    };
    

    当app接收到广播时,可能是这个activity当时已经关闭了

    【讨论】:

      【解决方案3】:

      @Jschools 有最好的答案。但作为替代方案,您也可以使用 Eventbus 代替侦听器模式。 eventbus 只会广播事件,如果 Activity 不活跃,则不会有任何事件到达,也不会抛出 NPE。

      【讨论】:

        【解决方案4】:

        根据文档,onAttach 在片段首次附加到其上下文时被调用。现在,由于您不是直接在 Activity 中而是通过适配器实例化片段,因此您确定调用了 DebugFragment 的 onAttach(Context context) 方法,因为这是您将上下文分配给 mListener 的地方吗?

        已经报告了未调用 onAttach(Context context) 方法的情况。请看https://code.google.com/p/android/issues/detail?id=183358

        您一起跳过 Fragment 生命周期方法,并在 newInstance() 中设置 mListener。

        public static DebugFragment newInstance(Context context) {
            DebugFragment fragment = new DebugFragment();
            fragment.setListener(context)
            Bundle args = new Bundle();
            fragment.setArguments(args);
            return fragment;
        }
        
        private void setListener(Context context){
        if (context.instanceof(OnFragmentInteractionListener) {
                mListener = (OnFragmentInteractionListener) context;
             } 
             else {
                throw new RuntimeException(context.toString()
                        + " must implement OnFragmentInteractionListener");
             }
        }
        

        【讨论】:

          猜你喜欢
          • 2023-01-05
          • 1970-01-01
          • 1970-01-01
          • 1970-01-01
          • 1970-01-01
          • 2013-05-11
          • 2017-11-09
          • 1970-01-01
          相关资源
          最近更新 更多