4

我有一个带有 ViewPager 的活动，在 ViewPager 适配器中我为每个位置提供片段。

一个示例片段是 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
        }
    });
}

我的活动实现如下。

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 null 了。这还不够吗。我还必须检查 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);
4

4 回答 4

12

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

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

于 2016-05-11T17:00:10.877 回答
2 
mMessageReceiver = new BroadcastReceiver() {
    @Override
    public void onReceive(Context context, Intent intent) {
      if(mListener != null){
        mListener.onFragmentInteraction(0);
      }
    }
};

当app接收广播时，可能是这个活动当时已经关闭

于 2016-05-12T09:21:27.697 回答
1

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

于 2016-05-17T12:42:04.047 回答
0

根据文档，onAttach 在片段首次附加到其上下文时被调用。现在，由于您不是直接在活动中实例化片段，而是通过适配器，您确定调用了 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");
     }
}
于 2016-05-18T10:06:59.007 回答