幫助與PHP

Question

優化調用的usort功能這是我的回調我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

有沒有辦法將這四行 變成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行。 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)); 
}