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

[beryl]

今天看到一段 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

[sunsulei]

能够复用的 就是可以给别人调的 判断
其实大部分还是判断一下吧 代码丑 但是可以省很多调试时间。

[billlee]

内部一般用 assert

[beryl]

@billlee 怎么讲？

[lscho]

如果代码中确实遇到相关问题的话，感觉需要工厂模式了啊。。。统一用一个方法来获得对象

[lihongjie0209]

precondition postcondistion 了解一下

[xiangyuecn]

看到 private 我就笑了笑 不说话

[lihongjie0209]

@lscho 那你要不要校验工厂==null?

[beryl]

@xiangyuecn ？不太懂

[NickCarter]

private 表示内部用的吧 既然是自己内部用的

[beryl]

@NickCarter 所以在疑惑这种内部使用，是不是没有必要这么做了，但是又在想要不要考虑之后可能 private 会变成 public

[xiangyuecn]

是时候展现真正的技术了 代码改一改绝对管用：fooB( Cls cls_If_传_Null_弹小 JJ )

[zjp]

一般 public 方法才需要校验参数。private 方法的调用者就是自己，自己清楚传的参数是不是有效的，而且还有 @ Nullable 这些注解辅助。后面方法改成 public 的时候再加校验也没什么问题...
https://droidyue.com/blog/2018/05/15/who-should-be-responsible-for-checking-parameter/

[victorywangzhcn]

防御式编程了解一下，check 一把无所谓的，理论上 JIT 能干掉~