Как оптимизировать функцию подсчета фракций? - PullRequest
Новая
       24

Как оптимизировать функцию подсчета фракций?

0 голосов
/ 26 сентября 2018

Я пишу функцию, которая получает один массив из 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;
        }

Ответы [ 2 ]

0 голосов
/ 26 сентября 2018

Цикл не нужен, просто вычислите целое число и новый числитель.Примерно так должно это сделать: 

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;
}
0 голосов
/ 26 сентября 2018

По крайней мере, вы можете переписать функцию следующим образом: 

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])
     );
}
...