帮助优化 PHP 中对 usort 函数的调用

Question

这是我对 usort() 的回调

public function sortProperties($a, $b) {

        $sortA = inflector::camelize(str_replace('-', '_', $this->sortBy));
        $sortB = inflector::camelize(str_replace('-', '_', $this->sortBy));

        $a = Arr::get($a, $sortA);
        $b = Arr::get($b, $sortB);


        if (is_numeric($a) AND is_numeric($b)) {
            return  $a < $b; 
        } else {
            return strcasecmp($a, $b); 
        }


    }

通常，当我看到代码中任意的前两行时，它会向我尖叫：重构！ em> 我想这是因为它们是相同的。

我知道我可以创建一个函数 getCamelized()，但我不认为我会在此之外再次使用它。

有没有办法把这4行变成2行？ func_get_args() 或 array_walk() 可以帮助我吗？

另外，这个排序功能有什么问题吗？

Answer 1

有没有办法把这4行转过来
变成2？

    $sortA = $sortB = inflector::camelize(str_replace('-', '_', $this->sortBy));

对于其他两行：

    list($a, $b) = array(Arr::get($a, $sortA), Arr::get($b, $sortB));

至于排序，至少对我来说似乎没问题。

Answer 2

$sortA == $sortB 所以这部分只是重复。无论您在哪里设置 $this->sortBy，都计算一次 $sortA。您所遇到的 Arr::get 行。 返回 $a < $b; 似乎是错误的，你应该返回一个 -ve, 0, +ve 数字。

...
function setSortBy($sortBy) {
    $this->sortBy = $sortBy;
    $this->sortByCam = inflector::camelize(str_replace('-', '_', $sortBy));
}
....

public function sortProperties($a, $b) {

    $a = Arr::get($a, $this->sortByCam);
    $b = Arr::get($b, $this->sortByCam);

    if (is_numeric($a) && is_numeric($b)) {
        return $a - $b;
    } else {
        return strcasecmp($a, $b); 
    }

}

类似的事情。主要思想是让骆驼化部分脱离循环。

Answer 3

请注意，strcasecmp 将返回一个 int（1、0 或 -1），< 将返回一个布尔值。您确实需要使用其中之一。另请注意，strnatcasecmp 可能会为您提供您想要的数字和字符串行为，因此请尝试以下操作：

public function sortProperties($a, $b) {
  $aInflected = Arr::get($a, $sort = inflector::camelize(str_replace('-', '_', $this->sortBy)));
  return strcasecmp($aInflected, Arr::get($b, $sort));
}