Как оптимизировать PHP код в данном примере?

Question

[Newer123]

Начинаю изучать PHP и решил задачу вот таким способом

<?php
$arrX = array(2, 4, 6, 8, 10, 12, 14, 16, 18, 20);
$arrY = array(1, 3, 5, 7, 9, 11, 13, 15, 17, 19);
$iter = 100000;
$start = microtime(true);

$number_p = 3.14;
$distance = 40 / 5;
$total = 0;

for ($a = 0; $a < $iter; $a++) {
    for ($i = 0; $i < count($arrX); $i++) {
        for ($j = 0; $j < count($arrY); $j++) {
            $value = ($arrX[$i] * ($number_p + $a)) * ($arrY[$j] + ($number_p * $distance));

            switch ($i) {
                case 3:
                    $total = $total + $value + 100;
                    break;
                case 6:
                    $total = $total + $value + 500;
                    break;
                case 9:
                    $total = $total + $value + 1000;
                    break;
                default:
                    $total = $total + $value;
                    break;
            }

            if ($value % 2 == 0) {
                $total = $total + 1;
            }

            if ($value % 2 != 0) {
                $total = $total + 2;
            }
        }
    }
}
echo "Total: $total";

$time = microtime(true) - $start;
echo "<br>Script time is $time";

Знаю, что код можно оптимизировать и цель уменьшить время исполнения.

Это тренировчная задача и цель просто как можно лучше оптимизировать.
Подскажите, пожалуйста, как можно оптимизировать данный код?

Comments

[FanatPHP]

https://toster.ru/q/681712?e=8234901#answer_item_1...

[kafkiansky]

Никак не оптимизируете. Выкиньте его. Вы только учитесь, нечего вам думать об оптимизации. Непонятно, какую задачу решает этот код, чтобы дать советы.

[Илья]

Ты написал бы хоть что-ли, какую задачу ты решил) а то доставать эту информацию из твоего кода не очень хочется)

[AUser0]

У вас некорректные данные!
Правильное Script time равно 7.4835469722748.

[Дмитрий]

Ну как минимум я нашёл два места для улучшения, но не факт что быстрее будет, но код короче точно
Ещё три места нашёл

[Newer123]

Дмитрий, покажите, пожалуйста

[Дмитрий]

Newer123, 40 / 5 = 8 так на всякий
Цикл один лишний, главный цикл делает 10 итерации с расчётами., потом тупо в холостую
Свич заменить на тернарник, массивы можно создать через range
С компа вечером могу сократить код в три или четыре раза, но сам код безсмысленный

[FanatPHP]

Ты уж определись, тебе быстрее или короче

[Newer123]

FanatPHP, и то и то. Учусь, а спросить, кроме этого сервиса некого, поэтому к любым правкам проявляю интерес

[FanatPHP]

Изучая, как писать бессмысленный код, ты научишься только писать бессмысленный код.

[Newer123]

FanatPHP, задача его оптимизировать, пускай он даже бессмысленный. Тут идет расчет на понимание синтаксиса и как лучше что сделать. Вот Дмитрий посоветовал и есть уже в каком направлении думать.

[FanatPHP]

Он тебе насоветует. Думать надо в направлении писать читаемый код, а не короткий.

[Дмитрий]

Newer123,
$total += $value % 2 === 0 ? 1 : 2;
Вместо двух иф

[Newer123]

Дмитрий, спасибо большое!:)

[Дмитрий]

Newer123, с switch тоже можно провести магию, создать массив с ключами 0, 3, 6, 9 , значения 0, 100, 500, 1000. Дальше додумать можно

Answer 1

[Павел Чесноков]

Немного ускорить можно отказавшись от подсчёта длины массивов в циклах, посчитав их только по разу:

$arrXcount = count($arrX);
$arrYcount = count($arrY);

for ($a = 0; $a < $iter; $a++) {
    for ($i = 0; $i < $arrXcount; $i++) {
        for ($j = 0; $j < $arrYcount; $j++) {
            ...
        }
    }
}

Также можно отказаться от двойного сравнения, поменяв это:

if ($value % 2 == 0) {
    $total = $total + 1;
}
if ($value % 2 != 0) {
    $total = $total + 2;
}

на это:

if ($value % 2 == 0) {
    $total = $total + 1;
} else {
    $total = $total + 2;
}

#PS: Ещё несколько микро шагов можно убрать отказавшись от двойных кавычек, а следовательно переменных в строках:

echo "Total: $total";
echo "<br>Script time is $time";

Используя одинарные кавычки, говорим интерпретатору, что не нужно в этой строке ничего искать и исправлять, просто выводим результат:

echo 'Total: ' . $total;
echo '<br>Script time is ' . $time;

Comments

[Newer123]

Спасибо большое! :)

[FanatPHP]

И тут начался ад.

[Евгений]

Newer123, И конкатенацию заменяем на запятые

echo 'Total: ' , $total;
echo '<br>Script time is ' , $time;

Answer 2

[neol]

У вас большая часть выражения может быть вынесена из цикла и рассчитана 1 раз, а не 100000. Как-то так:

for ($i = 0; $i < count($arrX); $i++) {
    for ($j = 0; $j < count($arrY); $j++) {
        $tmpValue = $arrX[$i] * ($arrY[$j] + ($number_p * $distance));
        for ($a = 0; $a < $iter; $a++) {
            $value = $tmpValue * ($number_p + $a);

Comments

[Newer123]

Огрмное спасибо! :)

Answer 3

[toxa82]

<?php
$arrX = array(2, 4, 6, 8, 10, 12, 14, 16, 18, 20);
$arrY = array(1, 3, 5, 7, 9, 11, 13, 15, 17, 19);
$iter = 100000;
$start = microtime(true);

$number_p = 3.14;
$distance = 40 / 5;
$total = 0;

$cntX = count($arrX);
$cntY = count($arrY);

$var0 = $number_p * $distance;
for ($a = 0; $a < $iter; $a++) {
    $varA = $number_p + $a;
    foreach ($arrX as $i => $x) {
        $varB = $x * $varA;
        foreach ($arrY as $j => $y) {
            $value = $varB * ($y + $var0);
            $total += $value;
            switch ($i) {
                case 3:
                    $total += 100;
                    break;
                case 6:
                    $total += 500;
                    break;
                case 9:
                    $total += 1000;
                    break;
            }
            if ($value % 2 == 0) {
                $total = $total + 1;
            } else {
                $total = $total + 2;
            }
        }
    }
}
echo "Total: $total";

$time = microtime(true) - $start;
echo "<br>Script time is $time";

Comments

[Newer123]

Ух ты! Спасибо большое! :)

Answer 4

[FanatPHP]

<?php
$start = microtime(true);

$total = '1.9317181388926E+14';
echo "Total: $total\n";

$time = microtime(true) - $start;
echo "<br>Script time is $time\n";

выводит

Total: 1.9317181388926E+14
<br>Script time is 2.7179718017578E-5

Какой вопрос, такой и ответ