使用函数进行编程练习

Question

我有一个返回整数的函数，但是我想扩展它以向其添加新参数。但是，使用此参数，该函数必须返回一个数组。

• 有一个基于参数返回数组或整数的函数是不好的做法吗？
• 如果是这样，我该如何解决这个问题？

我认为仅仅复制粘贴整个函数额外 4-5 行也是不好的做法。

Answer 1

如果可能的话，我会从一个函数中调用另一个函数。

function getSingle($arg)
{
    // Do whatever it is your function should do...
    return 1;
}

function getMultiple($args)
{
    $out = array();
    foreach ($args as $arg) {
        $out[] = getSingle($arg);
    }

    return $out;
}

对于您想要的功能，可能无法做到这一点，但这可能是一个不错的选择。

---

作为额外说明，由于这些函数彼此相关，我会将它们编写为类方法以将它们“分组”在一起。

以Users为例；我可能需要一个函数来获取单个用户，另一个函数来获取多个用户。将这些方法收集在一个类中是有意义的：

class Users
{
    public function getUser($id){}

    public function getUsers(array $id = null){}
}

Answer 2

在 PHP 中，我会说“一个函数一种输出类型”。 （FALSE 值除外，它在 PHP 中具有特殊含义）。如果 PHP 支持重载，情况可能会有所不同，但事实并非如此。但这里有一个问题，为什么不简单地使用一个函数来包装这两个函数呢？

 function my_wrapped_function($param1, array $param2)
 {
     return $param1 * count($param2);
 }

 function get_array_from_wrapped( $param1, array $param2 )
 {
     return array( $param1, my_wrapped_function($param1, $param2));
 }
 function get_int_from_wrapped( $param1, array $param2 )
 {
     return my_wrapped_function($param1, $param2);
 }

Answer 3

在我看来，这是不好的做法，因为它可能会导致使用它时出现问题，因为您并不总是知道您是从函数中获取 int 还是数组。

我的建议

总是返回一个数组（即使它只有一项长），并且有一个更通用的函数，易于处理。

Answer 4

我反对将事物作为数组返回，如果您的函数正在修改多个值，那么您编写的函数可能不是正确的，并且可能您需要有两个相同的函数。这样做的一个主要原因干净且可维护的代码，明天一个新人不需要去想知道他会发生什么，因为在某些情况下你必须修改多个参数，在这种情况下，进行调用看看该方法做了什么并返回正确的值和另一个值通过引用传递并编辑 它。

这是通过引用传递的示例..

<?php
function foo(&$var)
{
    $var++;
}

$a=5;
foo($a);
// $a is 6 here
?>

Answer 5

您在这里列出了一些原则：

1. 使函数参数具有一种含义（并且函数理想情况下为零参数，不惜一切代价避免更多参数）。
2. 让一个函数只做一件事。
3. 不要重复自己。

我认为这三个都是有效的。你的问题听起来像是 3. 和 2. 或 1.. 之间的交易。我认为你不应该将它们相互交易。

您没有分享太多关于您试图通过该功能实现的目标，因此完全抽象，我可以从中看到以下路径：

1. 您面临着设计问题。重新设计和重构您的代码。

嗯，听起来有点像聪明人的说法，但老实说，您可能没有正确使用迭代器（foreach），或者您开始​​创建一个“一个函数完成所有操作”的函数，而您不应该这样做。这最终会重复代码。