最佳实践-何时评估函数执行的条件

根据Dry的说法,我会和第一个一起去。

public function Foo() {
    DoThing();
}

private function DoThing() {
    if (!condition) return;
    // do stuff
}

一旦你习惯了这种模式,看到一个孤独的人并不是那么令人不安 DoThing() 在您的代码中。你会开始像读 EnsureThingDone() .

选项四,将谓词和实际调用包装在第三个函数中。

function DoThing() {
    // do stuff
}

function DoThingOnlyIfCondition() {
    if (!condition) return;
    DoThing();
}

function Foo() {
    DoThingOnlyIfCondition();
}

// Foo version 2
function FooBar() {
    DoThing();
}

现在,foo或任何函数都可以使用最合适的doxxx()版本。

我喜欢检查函数内部的前提条件,

public function DoThing()
{
    ValidatePreconditions();
    DoWork();
}

private function DoWork()
{
    //Do the actual work;
}

这样,我可以确定在执行函数之前满足了所有适当的先决条件,并且不需要使用者每次调用函数时都添加不必要的代码。

您可以使用类型系统。使参数为doting对象,该对象只有在传递前提条件时才能实例化。

要做到这一点,一个简单的方法是在该对象上创建一个实例方法。