保护私人/受保护方法的输入？

Question

通常,所有理智的开发人员都在努力确保所有公共方法的输入(转换为适当的类型,验证,消毒等)

我的问题是:您是否在代码中验证了传递给protected/private方法的参数？在我看来,没有必要,如果你正确安装公共方法的参数并从外部返回值(其他类,数据库,用户输入等...).

但我经常面对框架和应用程序(即prestashop命名一个),其中验证通常在方法调用中重复,在方法体中再次用于securize返回值 - 我认为,这是创建性能开销,也是一个标志糟糕的设计.

Answer 1

如果您坚持认为公共 API 应该具有能够防御不良参数的实现，那么您的标准不应该是方法的可见性，而应该是 API 的用户是否会直接调用该方法（或通过另一种推迟验证）。

应该进行验证的方法示例：

class A {
    protected final function myMethodDefaultImplementation(...) {
        /* subclasses can just call this method in their myMethod implementations */
        /* should do validation */
        ...
    }
    protected abstract myMethod(...);

    public function orderByDate() {
        return $this->orderBy(ORDER_BY_DATE)
    }

    private function orderBy($crit) {
        /* should do validation */
        ...
    }
}