Помощь при оптимизации вызовов функции usort в PHP

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); 
     } 


    } 

Обычно, когда я вижу первые 2 строки в любой моего кода, он кричит мне: реорганизовать! Я думаю, это потому, что они идентичны.

Я знаю, что могу сделать функцию 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 так что часть просто дублирование. Рассчитайте $sortA один раз, где бы вы ни установили $this->sortBy. Вы застряли в линиях Arr::get. return $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)); 
}