向每一个错误致敬——当心你的参数

每一次犯错,都是一次蜕变;每一个bug,都是一次成长。

  1. 入参处,对参数进行合法性校验;
  2. 使用不可靠的数据时,对该数据进行校验;

在入参处,一定要对参数进行合法性校验。且看如下代码,看看有什么问题!

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")){
        ......
    }

两种情况:

  1. 如果rubyCode有正常的值,程序正常运行;
  2. 如果传过来的rubyCode 是 null 呢? 那就惨了!!!

因此,在使用rubyCode之前,要做校验:

    if(rubyCode != null  && rubyCode.equals("EXfsGSEs")){
        ......
    }

上述,只是罗列了最简单的情形,也是我犯过的一些错误,这些错误让我意识到:

不要相信任何东西,除了经过自己检验,正确的,才可以相信。

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 161,758评论 24 692
  • 点击查看原文 Web SDK 开发手册 SDK 概述 网易云信 SDK 为 Web 应用提供一个完善的 IM 系统...
    layjoy阅读 9,745评论 0 14
  • 国家电网公司企业标准(Q/GDW)- 面向对象的用电信息数据交换协议 - 报批稿:20170802 前言: 排版 ...
    庭说阅读 8,034评论 6 13
  • 我变得越来越理智了 我是一直都没得到我想要的 但是我知道我一直在成长 果然是有得必有失 只是 心病难医
    Healer_e8b3阅读 62评论 0 0
  • 自我效能就是相信我能行,不断去强化自己的这个意识成为潜意识,需要通过完成一些小小的挑战去强化。像现在每天坚持五商学...
    刘霞阅读 71评论 0 0
  • 每天的晨圈故事里,我们骑着小马,摘了苹果,做了苹果酱,幹了面皮,做了月饼。 每个孩子都特别喜欢那一段:拿一个面皮放...
    honest_lee阅读 159评论 0 0