[英]How optimize function of counting fractions?
我編寫了一個從3個元素(整數,分子和分母)中獲取一個數組的函數。 它計算出錯誤的分數到正常值(分子>分母)。
如何優化此功能? 我知道我進行了很多驗證,但是功能看起來很丑。
function calculate_measurement(array $m): array
{
if (isset($m[NUMERATOR]) && isset($m[DENOMINATOR])) {
while (intval($m[NUMERATOR]) >= intval($m[DENOMINATOR])) {
$m[NUMERATOR] -= $m[DENOMINATOR];
if (!isset($m[INTEGER])) {
$m[INTEGER] = 0;
}
++$m[INTEGER];
}
if (0 === $m[NUMERATOR]) {
$m[NUMERATOR] = null;
$m[DENOMINATOR] = null;
}
}
return $m;
}
至少,您可以這樣重寫函數:
function calculate_measurement(array $m): array
{
if (!isset($m[INTEGER])) {
$m[INTEGER] = 0;
}
if (!isset($m[NUMERATOR]) || !isset($m[DENOMINATOR])) {
return $m;
}
while (intval($m[NUMERATOR]) >= intval($m[DENOMINATOR])) {
$m[NUMERATOR] -= $m[DENOMINATOR];
++$m[INTEGER];
}
if (0 === $m[NUMERATOR]) {
$m[NUMERATOR] = null;
$m[DENOMINATOR] = null;
}
return $m;
}
我不了解intval()的用法。 分子/分母可以不是整數嗎? 我寧願看到這樣的功能:
function calculate_measurement(int $i, int $numerator, int $denominator) : array
{
while($numerator >= $denominator) {
$numerator -= $denominator;
++$i;
}
if(0 === $numerator) {
$numerator = $denominator = null;
}
return [
INTEGER => $i,
NUMERATOR => $numerator,
DENOMINATOR => $denominator
];
}
function calculate_measuremenet_arr(array $m): array
{
if (!isset($m[NUMERATOR]) || !isset($m[DENOMINATOR])) {
return $m;
}
return calculate_measurement(
$m[INTEGER] ?? 0,
intval($m[NUMERATOR]),
intval($m[DENOMINATOR])
);
}
不需要循環,只需計算整數和新的分子即可。 這樣的事情應該做到:
function calculate_measurement(array $m): array {
if (!isset($m[NUMERATOR]) || !isset($m[DENOMINATOR])) {
return $m;
}
$m[INTEGER] = intval($m[NUMERATOR] / $m[DENOMINATOR]);
$m[NUMERATOR] -= ($m[INTEGER] * $m[DENOMINATOR]);
if ($m[NUMERATOR] === 0) {
$m[NUMERATOR] = null;
$m[DENOMINATOR] = null;
}
return $m;
}
聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.