Как оптимизировать мой код на PHP и сократить повторение функций для немного различимых переменных из массива?

Question

[Speakea1y1289]

Добрый день. Имеется следующий код:

// перебор массива Market и изменение кол-ва и цены. 
foreach ($updMarket as $i => $product) {

	$arAmount = array($product["PROPERTY_56_VALUE"],$product["PROPERTY_57_VALUE"],$product["PROPERTY_58_VALUE"]);
	$amount = array_sum($arAmount);
	
	settype($product["PROPERTY_53_VALUE"],"float");
	settype($product["PROPERTY_54_VALUE"],"float");
	settype($product["PROPERTY_55_VALUE"],"float");
	
	$arPrice = array_filter(array($product["PROPERTY_53_VALUE"],$product["PROPERTY_54_VALUE"],$product["PROPERTY_55_VALUE"]));
	$price = min($arPrice);
	
	$arLoadUpdProduct = Array(
		"amount" => $amount,
		"price" => $price,
	);

}

В данном коде мне не нравится, то что в переменных $arAmount и $arPrice создаётся массив с элементами из переменных из одного и того же массива $product, то есть меняется просто цифры с 53-58, а часть $product["PROPERTY_**_VALUE"] никак не меняется из-за чего получается много текста.
Также не нравится чрезмерное употребление settype также для не много изменяемых переменных:
settype($product["PROPERTY_**_VALUE"],"float");
В остальном вопросов нет. Как лучше сократить эту "вермишель"?

Comments

[DanKud]

А зачем вы преобразуете типы? PHP по дефолту вполне корректно преобразовывает типы в нужный момент. Этот код можно написать например так:

foreach ($updMarket as $i => $product) {
    $numProps = [56, 57, 58];
    $amount = array_sum(array_map(function($n) use ($product) {
        return $product['PROPERTY_' . $n . '_VALUE'];
    }, $numProps));

    $numProps = [53, 54, 55];
    $price = min(array_filter(array_map(function($n) use ($product) {
        return $product['PROPERTY_' . $n . '_VALUE'];
    }, $numProps)));

    $arLoadUpdProduct = [
        'amount' => $amount,
        'price' => $price
    ];
}

[Speakea1y1289]

DanKud, потому что изначально типы данных у данных переменных строковые

[Евгений]

Speakea1y1289, в php нестрогая типизация) т.е. "123" + 456 = 579 строка приводится к числу (если это возможно) и складывается... интерпритатор пытается сам определить (и в 98% случаев определяет верно) что от него хотят, если математические действия - то приводит данные к числу, если конкатенацию - то к строке, если логическое выражение - то к булеву...

[Speakea1y1289]

Евгений, вопрос уже решился, проверил, действительно операция с $price проходит успешно, а до этого не получалось

[Евгений]

Speakea1y1289, но всегда в голове нужно держать возможные проблемы) особенно с типом данных bool это часто проявляется... когда любое число или не пустая строка дадут true, а 0 или пустая строка могут вернуть false при нестрогом сравнении... а так же null и пустой массив тоже дадут false...

[Евгений Ромашкан]

И это все что вам не нравится в этом коде? Ужас!

Answer 1

[mr-troll]

// перебор массива Market и изменение кол-ва и цены. 
foreach ($updMarket as $p) {
  $arLoadUpdProduct = [
    "amount" => $p["PROPERTY_56_VALUE"]+$p["PROPERTY_57_VALUE"]+$p["PROPERTY_58_VALUE"], 
    "price" => min([$p["PROPERTY_53_VALUE"], $p["PROPERTY_54_VALUE"], $p["PROPERTY_55_VALUE"]])
 ];
}

Comments

[Sp1keazyYT]

В чём прикол? Что изменилось?

[mr-troll]

Sp1keazyYT, в том что тут 5 строчек, а не 10, и легко можно понять что происходит, а у тебя ненужные вызовы функций, лишние переменные, и работает в 100 раз медленнее! Тебе оптимизация нужна или что?
А делать форич по трём статическим свойствам - это паттерн плохого программирования, никакого код-ревью не пройдёт.