检查bitstamp.net汇率，并与以前的汇率进行比较。

Question

我写了低质量的php代码，我不是一个好的程序员;)请帮助我，请，优化它;)

这个简单的代码检查bitstamp.net汇率，并将其与以前的汇率进行比较并打印结果。

0) ? 'up' : ' down';

$ltc = getPrice('https://api.binance.com/api/v1/ticker/24hr?symbol=LTCUSDT');
$ltcusdlast = round($ltc["lastPrice"], 1);
$rltc = ($ltc["priceChange"]>0) ? 'up' : ' down';

?>

    BTC 
    $ 

    LTC 
    $ 

但是它的质量很低，而且very的工作速度很慢&。

Answer 1

Id使您的代码更加实用(如果您不打算使用OOP)，这段代码速度慢的事实可能是您的互联网连接(如注释中提到的)。

首先，当api返回公用键时，我只需要将舍入&三元操作符作为一个函数，您可以传递响应，

另外，由于您没有使用全名，只使用简短的代码，整个代码在一个循环中工作，这样您就可以在$coinsToFetch数组中添加更多的硬币短代码。

$data["priceChange"] > 0 ? "up" : "down",
    "lastPrice"=>round($data["lastPrice"], 1)
  ];
}

$coinsToFetch = ["BTC", "LTC"];

foreach ($coinsToFetch as $coin) {
  $coinData = fetchAndDecodeData('https://api.binance.com/api/v1/ticker/24hr?symbol=' . $coin . 'USDT');
  $data = getValuesFromBinanceData($coinData);
  echo " $coin " . $data['lastPrice'] . $data['priceChange'] . "";
}

为了解决脚本相对于internet的速度问题，您可以缓存结果(redis)或将其存储在数据库(mysql)中，这将允许您执行类似的代码(psudeo代码)。

If its less than 5 minutes sinse the cache
  Load the data from the cache
Else 
  Load from binance api

如果要使用缓存/存储，我将将其转换为两个脚本；

1. 它以cron作业的形式运行，以每x分钟填充缓存/存储。
2. 显示数据