重構PHP代碼 - 減少foreach語句的數量

Question

如您在下面的代碼中看到的，我對三個結構類似的變量使用相同的代碼（foreach循環，數組賦值和尾隨json_decode）三次。我想知道如何優化我的代碼，以便它不會不必要地重複執行功能。變量變量在這種情況下會有幫助嗎？我可以將重複的代碼行移動到第一個foreach聲明中嗎？

這裏是我的代碼看起來像現在：

date_default_timezone_set('America/Los_Angeles'); 

$stocks = array('MSFT' => 'http://ichart.finance.yahoo.com/table.csv?s=MSFT', 'AAPL' => 'http://ichart.finance.yahoo.com/table.csv?s=AAPL', 'FB' => 'http://ichart.finance.yahoo.com/table.csv?s=FB', 'ZNGA' => 'http://ichart.finance.yahoo.com/table.csv?s=ZNGA'); 

foreach ($stocks as $key=>$stock) { 
    $fh = fopen($stock, 'r'); 
    $header = fgetcsv($fh); 

    $varname = $key . '_data'; 

    $$varname = array(); 
    while ($line = fgetcsv($fh)) { 
     ${$varname}[count($$varname)] = array_combine($header, $line); 
    } 

    fclose($fh); 
} 

foreach($MSFT_data as $val){ 
    $MSFT[] = array((strtotime($val['Date']) * 1000), ((float)$val['Close'])); 
} 
$MSFT = json_encode(array_reverse($MSFT)); 

foreach($AAPL_data as $val){ 
    $AAPL[] = array((strtotime($val['Date']) * 1000), ((float)$val['Close'])); 
} 
$AAPL = json_encode(array_reverse($AAPL)); 

foreach($FB_data as $val){ 
    $FB[] = array((strtotime($val['Date']) * 1000), ((float)$val['Close'])); 
} 
$FB = json_encode(array_reverse($FB)); 

感謝。如果您有任何問題，請告訴我。

Answer 1

對於三個環路，請嘗試：

function dateCloseLoop($data) { 
    foreach($data as $val){ 
     $tmp[] = array((strtotime($val['Date']) * 1000), ((float)$val['Close'])); 
    } 
    return json_encode(array_reverse($tmp)); 
} 

所以，你的代碼將是這樣的：

$MSFT = dateCloseLoop($MSFT_data); 
$AAPL = dateCloseLoop($AAPL_data); 
$FB = dateCloseLoop($FB_data); 

Answer 2

您可以使用關聯數組，爲每個存儲鍵存儲數據數組，並在該嵌套數組上使用嵌套的foreach。

事情是這樣的：

$res = array(); 
$stocks = array('MSFT' => '...', 'AAPL' => '...', 'FB' => '...', 'ZNGA' => '...'); 

foreach ($stocks as $stock => $url) { 
    $fh = fopen($url, 'r'); 
    $header = fgetcsv($fh); 

    $res[$stock] = array(); 

    while ($line = fgetcsv($fh)) { 
     $res[$stock][] = array_combine($header, $line); 
    } 

    fclose($fh); 
} 

$json = array(); 

foreach ($res as $stock => $data) { 
    $out = array(); 
    foreach($data as $val){ 
     $out[] = array((strtotime($val['Date']) * 1000), ((float)$val['Close'])); 
    } 
    $json[$stock] = json_encode(array_reverse($out)); 
}