在 PHP 中测试可选参数

Question

我有一些跨类的“setter”方法，为了方便起见，我添加了一个可选参数 $previous，它通过引用获取参数并用现有值填充它，然后将其替换为新值一。例如：

public function set_value($key, $value, &$previous = null)
{
    $previous = $this->get_value($key);
    $this->_values[$key] = $value;
    return $this;
}

这很好用；然而，在某些情况下，相应的“getter”方法有点进程密集，无条件运行它是一种浪费。我想我可以测试一下：

if(null !== $previous)
{
    $previous = $this->get_value($key);
}

但这不起作用，因为作为 $previous 参数传递的变量通常尚未在其范围内定义，并且默认为 null。我破解的唯一解决方案是：

public function set_value($key, $value, &$previous = null)
{
    $args = func_get_args();
    if(isset($args[2])
    {
        $previous = $this->get_value($key);
    }
    $this->_values[$key] = $value;
    return $this;
}

或者，单行：

if(array_key_exists(2, func_get_args()))
{
    // ...
}

我不喜欢方法体依赖于参数索引（当它看起来应该是不必要的）一种更干净的方式来实现我在这里所追求的目标？

---

我试过了：

if(isset($previous)){}

if(!empty($previous)){}

if(null !== $previous){}

都不起作用。

到目前为止可能的解决方案：

if(func_num_args() == $num_params){}

if(array_key_exists($param_index, func_get_args())){}

// 5.4
if(isset(func_get_args()[$param_index])){}

// 5.4
if(func_num_args() == (new \ReflectionMethod(__CLASS__, __FUNCTION__))
    ->getNumberOfParameters()){}

---

@DaveRandom - 所以，在以下领域：

define('_NOPARAM', '_NOPARAM' . hash('sha4096', microtime()));

function foo($bar = _NOPARAM)
{
    // ...
}

---

@hoppa - 用例：

$obj->set_something('some_key', $some_value, $previous) // set
    ->do_something_that_uses_some_key()
    ->set_something('some_key', $previous) // and reset
    ->do_something_that_uses_some_key()
    -> ...

而不是：

$previous = $obj->get_something('some_key'); // get
$obj->set_something('some_key', $some_value) // set
    ->do_something_that_uses_some_key();
    ->set_something($previous) // and reset
    ->do_something_that_uses_some_key();
    -> ...

Answer 1

可能不是您想要解决问题的方式（以某种方式测试可选参数），但这就是我实现它的方式：

public function set_value($key, $value)
{
    $this->_values[$key] = $value;
    return $this;
}
public function set_get_value($key, $value, &$previous)
{
    $previous = $this->get_value($key);
    $this->_values[$key] = $value;
    return $this;
}

用例示例：

$obj->set_get_something('some_key', $some_value, $previous) // set AND get
    ->do_something_that_uses_some_key()
    ->set_something('some_key', $previous) // and reset
    ->do_something_that_uses_some_key()
    -> ...

为什么使用另一个函数？

这个解决方案有一些优点：

1. 名称更明确，对其他编码人员来说更少混乱
2. 没有隐藏的副作用
3. 解决了已经有值的（未定义）变量的问题
4. 没有调用func_num_args或其他一些“元”的开销“函数

编辑：拼写错误 代码。

编辑2：删除了 &$previous set_get_value() 函数的默认值（感谢 draevor）

Answer 2

摘自上面的评论/讨论：

为了检查参数是否已传递，您有 2 个选项 - 根据值检查参数的值（就像您对 null 所做的那样）或检查参数的数量。

如果您选择第一个选项，则没有任何值不能从函数外部传递，因此总是有可能出现误报（与现在使用 null 时发生的情况相同）。 DaveRandom 的随机字符串示例对于大多数情况来说应该足够了，但我认为它有点矫枉过正。

我认为第二个选项是最干净的（快速、可读等）。作为对 func_get_args 所做的一个小改进，我将使用 func_num_args - 这样您将检查传递参数的数量，而不是参数指数。