
在一个项目中使用了ViewPager+Fragment的组合,但是在实际使用中频繁的Crash,错误是在Fragment内部出现的NullPointException。
经过仔细排查,发现是因为我在Fragment内有一些AsyncTask联网操作,在网络链接失败的时候会弹出Toast消息提示。而生成Toast时传入的Context参数是getActivity() ,这里就有可能会出现问题了,只有Fragment附着(onAttach)在Activity上时 getActivity() 函数才会返回正确的对象,否则的话返回null。
因此,如果我正在联网,滑动ViewPager使得这个Fragment被销毁了或者点击Back键返回了,此时Fragment就会和Activity解除附着,当再试图弹出Toast的时候,getActivity() 返回null,于是就Crash了。
解决方案
在Fragment附着在Activity上时用一个变量保存引用。
1 2 3 4
|
public void (Activity activity){ this.mContext = activity; }
|
但是发现该方法在23已经标注过时。
于是,修改为:
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34
|
@TargetApi(23) public void (Context context) { super.onAttach(context); onAttachToContext(context); } * Deprecated on API 23 * Use onAttachToContext instead */ @SuppressWarnings("deprecation") public void (Activity activity) { super.onAttach(activity); if (Build.VERSION.SDK_INT < Build.VERSION_CODES.M) { onAttachToContext(activity); } } * Called when the fragment attaches to the context */ protected void onAttachToContext(Context context) { this.mContext = context; } public void onDetach() { super.onDetach(); this.mContext = null; }
|
当时stackFlow查了一会,以为这是最优的解决方案,记录下。
另,还看到另一种解决方案,使用全局Application获取Context:
1 2 3 4 5 6 7 8 9 10 11 12 13 14
|
public class MyApplication extends Application { private static MyApplication instance; public void onCreate() { super.onCreate(); instance = this; } public static MyApplication getInstance(){ return instance; } }
|
在程序任何地方,都可以使用Application来获取Context也不用担心内存泄露。
1 2
|
Context context = MyApplication.getInstance(); Toast.makeText(context, "Your Toast Message", Toast.SHORT_TOAST).show();
|
不过感觉前一种比较合适,毕竟这是全局的。
近期评论