将整个函数体放在 if 语句中是否被认为是不好的做法？

Question

我经常发现自己编写的函数看起来像：

public function foo($param): void 
{
    if($param) {
        //do something
    }
}

如果$param eval 为 false，则不执行任何代码。

这看起来是一种不好的做法，是否有任何关于如何避免这种编码的指南/模式？

Answer 1

如果一个函数是一个全有或全无的条件，我首先有 else 条件 - 这样，如果 $param 解析为 false，该函数会立即退出：

public function foo($param): void 
{
    if(!$param) {
        return;
    }
    //do something
}

Answer 2

您可以像这样改进功能：

function foo($param='defaultValue') 
{
    if($param)
    {
        //do something

        return true; // or something else
    }

    return false; // by default return false
}

通过这样做，如果 $param 将被评估为 false，则将使用 return false。

此外，如果您的函数位于类中，最好为该函数指定访问修饰符（public、protected、private）。

Answer 3

我个人会将此代码更改为：

function foo($param) 
{
    $param 
      && (function () {
        // Do code here.
      })();
}

但是如果我想在其他地方调用内部函数，并保持一切干净，我可能会将其重构为：

function foo($param) 
{
  $param
    && bar();
}

function bar() 
{
  // Do code here
}

然后我会调整 foo 的命名，以明确它是 bar 的包装器并重命名 bar 以便听起来它缺少包装器，可能类似于 action_only 或 @987654328 @

然后我还将它写在每个函数的文档中，在 OOP 领域，这将是一个私有方法，但这也不会强制执行真正的隐私，请参阅：What is the point of OOP visibility in PHP when Closures and Reflections are available?，实际上是函数内部的闭包，在第一个例子中，在技术层面上更“私密”

Answer 4

这取决于你想用这个函数做什么。 但我的意见是你的条件应该在函数之外，例如

function foo() {
    // doing something
}

然后

if ($params) {
    foo();
}

它看起来更可定制

Answer 5

将access modifiers 添加到函数中

类属性必须定义为公共、私有或受保护。如果使用 var 声明，该属性将被定义为 public。

public function foo($param=null) 
{
    /* Condition while not empty then proceed */
    if(!empty($param)) {
        //do something
    } else {

        return false;
    }
}