每一次犯错,都是一次蜕变;每一个bug,都是一次成长。
- 入参处,对参数进行合法性校验;
- 使用不可靠的数据时,对该数据进行校验;
在入参处,一定要对参数进行合法性校验。且看如下代码,看看有什么问题!
public class InitChatActionDataUtil {
public static ArrayList<String> initGridListData(String type, String remark, int isShowReserve, String doctorBed) {
ArrayList<String> gridActionList = new ArrayList();
gridActionList.add("相册");
gridActionList.add("拍照");
gridActionList.add("表情");
if (type.equals("2")) {
//群聊,生成GridView数据: 相册,拍照,表情,红包
gridActionList.add("红包");
} else {
//与医生,相册,拍照,表情,评价,(预约,加床====随机,根据传过来数据判断)
if (remark.equals("医生")) {
gridActionList.add("评价");
if (isShowReserve == 2) {
gridActionList.add("预约");
if (doctorBed.equals("2")) {
gridActionList.add("加床");
}
}
}
//小瑞:相册,拍照,表情
//个管:相册,拍照,表情,评价,红包
if (remark.equals("个管师")) {
gridActionList.add("评价");
gridActionList.add("红包");
}
// 与个人:相册,拍照,表情,红包
if (remark.equals("患者")) {
gridActionList.add("红包");
}
}
return gridActionList;
}
}
在上述代码中,我通过入参:
- type,传入会话类型,是单聊,还是群聊;
- remark,备注,会话对象是医生,还是个管师,还是客服,or 患者;
- doctorBed,医生是否提供加床服务;
如果,入参按照我的设想,都是正确的传入,那么这段代码可以正常运行,代码如下。
InitChatActionDataUtil.initGridListData(
"2",
"医生",
1,
"2")
但是,如果入参变成如下这样,会怎样呢?
InitChatActionDataUtil.initGridListData(
null,
null,
1,
null)
在传入null后,程序抛出异常:
reason: java.lang.RuntimeException: Unable to start activity ComponentInfo{com.xxxx/com.xxxx.activity.ChatMessageActivity}: java.lang.NullPointerException: Attempt to invoke virtual method 'boolean java.lang.String.equals(java.lang.Object)' on a null object reference
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2359)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2422)
at android.app.ActivityThread.access$800(ActivityThread.java:159)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1319)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:135)
at android.app.ActivityThread.main(ActivityThread.java:5313)
at java.lang.reflect.Method.invoke(Native Method)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1116)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:809)
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'boolean java.lang.String.equals(java.lang.Object)' on a null object reference
at com.xxxx.utils.InitChatActionDataUtil.initGridListData(InitChatActionDataUtil.java:24)
at com.xxxx.activity.ChatMessageActivity.findView(ChatMessageActivity.java:375)
at com.xxxx.activity.ChatMessageActivity.onCreate(ChatMessageActivity.java:313)
at android.app.Activity.performCreate(Activity.java:6036)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1106)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2302)
... 10 more
可以看到结果:
java.lang.NullPointerException
因为,代码变成了这样:
null.equals("xxx")
至此,算是清楚了。如果不对入参进行校验,程序会抛出异常。这里,只是罗列了一个极其简单的场景。在实际开发中,这种情况非常多见。一定要对入参进行校验。
上面代码改成,如下所示,才会比较安全:
public class InitChatActionDataUtil {
public static ArrayList<String> initGridListData(String type, String remark, int isShowReserve, String doctorBed) {
ArrayList<String> gridActionList = new ArrayList();
gridActionList.add("相册");
gridActionList.add("拍照");
gridActionList.add("表情");
//对入参进行校验
if (type != null && remark != null && doctorBed != null) {
......
}
return gridActionList;
}
下面再看另一中情形:
我从Activity A 中传过来两个Extra : rubyCode 、type。
rubyCode = getIntent().getStringExtra("rubyCode");
UIType = getIntent().getIntExtra("type", 2);
在当前Activity B中直接使用这两个参数,会发生什么呢?
if(rubyCode.equals("EXfsGSEs")){
......
}
两种情况:
- 如果rubyCode有正常的值,程序正常运行;
- 如果传过来的rubyCode 是 null 呢? 那就惨了!!!
因此,在使用rubyCode之前,要做校验:
if(rubyCode != null && rubyCode.equals("EXfsGSEs")){
......
}
上述,只是罗列了最简单的情形,也是我犯过的一些错误,这些错误让我意识到:
不要相信任何东西,除了经过自己检验,正确的,才可以相信。