V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
beryl
V2EX  ›  程序员

请教个问题,程序参数判断是否应该多层校验呢?

  •  
  •   beryl · 2018-09-08 20:58:36 +08:00 · 1699 次点击
    这是一个创建于 2266 天前的主题,其中的信息可能已经有所发展或是发生改变。

    今天看到一段 Java 代码,在一个类里面,调用当前类的一个私有方法。 即方法 A 调用私有方法 B. 有个参数对象 cls, 在 A 里面去调用该方法的时候,想去 new 了 cls,然后当做参数传递进去, 在 B 里面又做了一次校验对 cls, 如果非空则重新 new.

    还是直接贴代码吧:

    public ClassAbc {
    
    	public void fooA() {
        
        	// 此处 new 出该对象作为参数传递
        	Cls cls = new Cls();
            FooB(cls);
        }
        
        private boolean fooB(Cls cls) {
        	// ...
            
            // 因为上传传递之前已经 new 了,所以此处是否还需要判断呢
            if (cls == null} {
            	cls = new Cls();
            }
            
            // ...
        }
    }
    

    Thx

    13 条回复    2018-09-08 23:09:15 +08:00
    sunsulei
        1
    sunsulei  
       2018-09-08 21:10:01 +08:00 via iPhone
    能够复用的 就是可以给别人调的 判断
    其实大部分还是判断一下吧 代码丑 但是可以省很多调试时间。
    billlee
        2
    billlee  
       2018-09-08 21:12:16 +08:00
    内部一般用 assert
    beryl
        3
    beryl  
    OP
       2018-09-08 21:14:04 +08:00
    @billlee 怎么讲?
    lscho
        4
    lscho  
       2018-09-08 21:17:07 +08:00
    如果代码中确实遇到相关问题的话,感觉需要工厂模式了啊。。。统一用一个方法来获得对象
    lihongjie0209
        5
    lihongjie0209  
       2018-09-08 21:26:26 +08:00
    precondition postcondistion 了解一下
    xiangyuecn
        6
    xiangyuecn  
       2018-09-08 21:26:33 +08:00
    看到 private 我就笑了笑 不说话
    lihongjie0209
        7
    lihongjie0209  
       2018-09-08 21:27:03 +08:00
    @lscho 那你要不要校验工厂==null?
    beryl
        8
    beryl  
    OP
       2018-09-08 21:31:33 +08:00
    @xiangyuecn ?不太懂
    NickCarter
        9
    NickCarter  
       2018-09-08 21:36:15 +08:00 via iPhone
    private 表示内部用的吧 既然是自己内部用的
    beryl
        10
    beryl  
    OP
       2018-09-08 21:40:46 +08:00
    @NickCarter 所以在疑惑这种内部使用,是不是没有必要这么做了,但是又在想要不要考虑之后可能 private 会变成 public
    xiangyuecn
        11
    xiangyuecn  
       2018-09-08 22:16:52 +08:00
    是时候展现真正的技术了 代码改一改绝对管用:fooB( Cls cls_If_传_Null_弹小 JJ )
    zjp
        12
    zjp  
       2018-09-08 22:34:37 +08:00   ❤️ 1
    一般 public 方法才需要校验参数。private 方法的调用者就是自己,自己清楚传的参数是不是有效的,而且还有 @ Nullable 这些注解辅助。后面方法改成 public 的时候再加校验也没什么问题...
    https://droidyue.com/blog/2018/05/15/who-should-be-responsible-for-checking-parameter/
    victorywangzhcn
        13
    victorywangzhcn  
       2018-09-08 23:09:15 +08:00
    防御式编程了解一下,check 一把无所谓的,理论上 JIT 能干掉~
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2799 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 32ms · UTC 11:59 · PVG 19:59 · LAX 03:59 · JFK 06:59
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.