1

正如您在下面的代码中看到的那样，对于结构相似的三个变量，我使用相同的代码（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));

谢谢。如果您有任何问题，请告诉我。

4

2 回答 2

1

您可以只使用关联数组，为每个股票键存储一个数据数组，并foreach在该嵌套数组上使用嵌套的 's。

像这样的东西：

$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));
}
于 2013-03-11T02:55:06.950 回答
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);
于 2013-03-11T03:05:47.650 回答