我們如何優化這些代碼行？

我想知道有什麼方法可以優化代碼行。我知道我們可以在第一個try塊中寫入其他$row[] = $this->getFields($inputHeaders, $result, $fields['row2']);。但我還是想知道的是有減少的代碼行任何其他方式或做我們如何優化這些代碼行？

try { 
     $row[] = $this->getFields($inputHeaders, $result, $fields['row1']); 
    } catch (BAS_Shared_InvalidArgumentException $e) { 
     $logger->err($e->getMessage()); 

     return false; 
    } 

    // Get data for first row. 
    try { 
     $row[] = $this->getFields($inputHeaders, $result, $fields['row2']); 
    } catch (BAS_Shared_InvalidArgumentException $e) { 
     $logger->err($e->getMessage()); 

     return false; 
    }

來源

2016-01-13 yo black

你真的嘗試優化什麼？執行時間處理時間？內存消耗？過早優化是所有邪惡的根源 –

優化並不是一個合適的SO主題。 Code Review更適合這一點，但即使在那裏，您至少還需要包含getFields方法的定義，並且澄清'optimize'的含義 - 執行更快？更容易閱讀？簡單的線條/字符？ – Steve

假設沒有更好的辦法，這在被問及重構，而不是優化我給下面的回答：

$rowNames = array('row1', 'row2'); 

foreach ($rowNames as $rowName) { 
    try { 
     $row[] = $this->getFields($inputHeaders, $result, $fields[$rowName]); 
    } catch (BAS_Shared_InvalidArgumentException $e) { 
     $logger->err($e->getMessage()); 

     return false; 
    } 
}

來源

2016-01-13 14:11:44

我們如何優化這些代碼行？

回答

相關問題