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

每一次犯错,都是一次蜕变;每一个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")){
        ......
    }

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

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

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 213,014评论 6 492
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 90,796评论 3 386
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 158,484评论 0 348
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 56,830评论 1 285
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 65,946评论 6 386
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,114评论 1 292
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,182评论 3 412
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,927评论 0 268
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,369评论 1 303
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,678评论 2 327
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,832评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,533评论 4 335
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,166评论 3 317
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,885评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,128评论 1 267
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,659评论 2 362
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,738评论 2 351

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 171,884评论 25 707
  • 点击查看原文 Web SDK 开发手册 SDK 概述 网易云信 SDK 为 Web 应用提供一个完善的 IM 系统...
    layjoy阅读 13,724评论 0 15
  • 国家电网公司企业标准(Q/GDW)- 面向对象的用电信息数据交换协议 - 报批稿:20170802 前言: 排版 ...
    庭说阅读 10,934评论 6 13
  • 我变得越来越理智了 我是一直都没得到我想要的 但是我知道我一直在成长 果然是有得必有失 只是 心病难医
    Healer_e8b3阅读 139评论 0 0
  • 自我效能就是相信我能行,不断去强化自己的这个意识成为潜意识,需要通过完成一些小小的挑战去强化。像现在每天坚持五商学...
    423429d90f38阅读 156评论 0 0