美文网首页AndroidAndroid技术知识技术干货
向每一个错误致敬——当心你的参数

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

作者: 张庚 | 来源:发表于2016-03-24 18:02 被阅读4751次

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

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

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

相关文章

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

    每一次犯错,都是一次蜕变;每一个bug,都是一次成长。 入参处,对参数进行合法性校验;使用不可靠的数据时,对该数据...

  • 司考复习日记2017.9.15

    向熬过的每一个夜晚致敬!

  • 向每一个错误致敬—— AndFix学习记录

    项目以飞快的速度迭代,2周进行一次迭代升级。每次开发完功能跑通,在现有机型上测试没问题的话,就提交市场。 在以极快...

  • 向不跪着致敬

    向您致敬 向勇敢致敬 向不跪着致敬 向扑通扑通前赴后继奔向大海的雪花致敬

  • 致敬

    向诗人致敬 向诚实致敬 向四野八荒的诗致敬 向逃过诗人的语言致敬

  • 万物安稳

    花朵向根须致敬 种子向蜜蜂致敬 雨水向河床致敬 蜘蛛向黑暗致敬 男人向大地致敬 母亲向天空 问安:“跟好人学好人 ...

  • 可怜生肖百魅生  不为此物千重引

    当按响了留言 我已离开了你的城市 相信路上 有你来过的足迹 当心都已厌倦 别再修复碎了的场景 你曾带着错误的爱 向...

  • 向诗篇致敬

    向诗篇致敬向诗篇里的每一颗汉字致敬向留着诗篇沉睡的纸张致敬向穿过窗户,无私给诗篇温暖的阳光致敬向庇护诗篇,防止它被...

  • 今天,致敬国之脊梁,致敬每一个为抗疫努力的你

    今天,致敬国之脊梁,致敬每一个为抗疫努力的你

  • 再向诗人致敬

    再向诗人致敬 向作曲家致敬 向演唱者致敬 每听《七子之歌》 都让我热泪盈盈… 你可知“妈港”不是我的真名姓? 我离...

网友评论

    本文标题:向每一个错误致敬——当心你的参数

    本文链接:https://www.haomeiwen.com/subject/hxmrlttx.html