函数调用中的多个参数与单个数组

我有一个函数，它接受一组参数，然后将它们作为条件应用于SQL查询。但是，尽管我赞成包含条件本身的单个参数数组：

function searchQuery($params = array()) {
    foreach($params as $param => $value) {
        switch ($param) {
            case 'name':
                $query->where('name', $value);
                break;
            case 'phone':
                $query->join('phone');
                $query->where('phone', $value);
                break;
        }
    }
}

我的同事更喜欢显式列出所有参数：

function searchQuery($name = '', $phone = '') {
    if ($name) {
        $query->where('name', $value);
    }

    if ($phone) {
        $query->join('phone');
        $query->where('phone', $value);
    }
}

他的论点是，通过显式列出参数，该函数的行为将变得更加明显-而不是必须深入代码以找出神秘的论点$param。

我的问题是，在处理很多参数（例如10+）时，这变得非常冗长。有什么首选的做法吗？我最坏的情况是看到以下内容：

searchQuery('', '', '', '', '', '', '', '', '', '', '', '', 'search_query')

恕我直言，您的同事对于上述示例是正确的。您的偏好可能很简洁，但其可读性也较差，因此难以维护。问一个问题，为什么要一开始就编写函数，您的函数“带到表”是什么—我必须详细地了解它的作用以及如何执行它，而仅仅是使用它。通过他的示例，即使我不是PHP程序员，我也可以在函数声明中看到足够多的细节，而不必担心其实现。

就大量参数而言，通常将其视为代码气味。通常，该函数尝试执行的操作太多吗？如果确实发现确实需要大量参数，则它们可能以某种方式关联并且一起属于一个或几个结构或类（甚至可能是相关项的数组，例如地址中的行）。但是，传递非结构化数组并不能解决代码异味。

我的回答或多或少与语言无关。

如果在复杂数据结构（表，记录，字典，对象...）中对参数进行分组的唯一目的是将它们作为一个整体传递给一个函数，则最好避免使用它。这增加了一层毫无用处的复杂性，并使您的意图变得晦涩。

如果分组的参数本身具有含义，则该复杂性层有助于理解整个设计：将其命名为抽象层。

您可能会发现，最好的设计是使用两个或三个参数，而不是将多个参数或一个大型数组组合在一起，以对相关数据进行分组。

就您而言，我更喜欢您同事的方法。如果您正在编写模型，而我正在使用您的模型来开发它们。我看到了您同事方法的签名，可以立即使用。

同时，我将不得不执行您的searchQuery函数，以查看您的函数需要哪些参数。

我只在searchQuery只限于在单个表中搜索的情况下才使用您的方法，因此不会有任何联接。在这种情况下，我的函数将如下所示：

function searchQuery($params = array()) {
    foreach($params as $param => $value) {
        $query->where($param, $value);
    }
} 

因此，我立即知道array的元素实际上是具有该方法的类在您的代码中表示的特定表的列名。

两者都做。 array_merge就像您的同事喜欢的那样，允许在函数顶部显示一个显式列表，同时根据您的喜好避免参数变得笨拙。

我也强烈建议在问题注释中使用@chiborg的建议-您的意图要清楚得多。

function searchQuery($params = array()) {
    $defaults = array(
        'name' => '',
        'phone' => '',
        ....
    );
    $params = array_merge($defaults, $params);

    if(!empty($params['name'])) {
        $query->where('name', $params['name']);
    }
    if (!empty($params['phone'])) {
        $query->join('phone');
        $query->where('phone', $params['phone']);
    }
    ....
}

另外，您可以传递类似于查询字符串的字符串，并使用parse_str（因为您似乎在使用PHP，但是其他解决方案可能在其他语言中可用）将其处理为方法内部的数组：

/**
 * Executes a search in the DB with the constraints specified in the $queryString
 * @var $queryString string The search parameters in a query string format (ie
 *      "foo=abc&bar=hello"
 * @return ResultSet the result set of performing the query
 */
function searchQuery($queryString) {
  $params = parse_str($queryString);
  if (isset($params['name'])) {
    $query->where('name', $params['name']);
  }
  if (isset($params['phone'])) {
    $query->join('phone');
    $query->where('phone', $params['phone']);
  }
  ...

  return ...;
}

并称它为

$result = searchQuery('name=foo&phone=555-123-456');

您可以http_build_query用来将关联数组转换为字符串（相反parse_str）。