- 1
- 2
- 3
for ($i = 0, $cnt = count($arList); $i < $cnt; $i++) {
$arConfig["CONFIG"]["category"]["VALUES"][$arList[$i]] = GetMessage('SALE_DH_RUSSIANPOST_CONFIG_CATEGORY_'.$arList[$i]);
}-
PHP / Говнокод #8396
+151
... вместо человеческого foreach($arList as $k => $v) { ... }
Запостил:
Iceman,
02 Ноября 2011
Follow us!
Блин, не можешь сделать нормально, ну сделай ты хоть через foreach.
Страшнее всего то, что такие конструкции в коде вообще повсюду, а не на отдельно взятом участке.
> на право
1. Лишний вызов count()
2. Счётчик остаётся в памяти после прохождения цикла.
Чем это плохо:
1. При постоянных вызовах лишних функций повсеместно в коде значительно снижается производительность.
2. Переменная не забирается сборщиком мусора. При использовании foreach() переменные внутри цикла ($key, $value) уничтожаются после прохождения цикла, освобождая память.
инициализировать массив $arConfig куда записывает строку (видимо ланг ключ или название константы) -
'SALE_DH_RUSSIANPOST_CONFIG_CATEGORY_'.$ arList[$i]
2. что значит лишний вызов count? (без него здесь никак, я так думаю что в реализации foreach тоже вшит этот метод) Как Вы видите вызов этой функции находится внутри секции инициализации цикла. Так что данная функция вызовится только раз!
3. "При использовании foreach() переменные внутри цикла ($key, $value) уничтожаются после прохождения цикла, освобождая память" -
запустите этот пример и Вы увидите, что не правы! ничего не удаляется (юазайте подход ООП для того, чтобы не захломлять глобальную область видимости переменными):
<?php
$aTest = array(
'first' => array(
'name' => 'Alex'
),
'second' => array(
'Petia',
),
);
foreach($aTest as $sKey => $aValues)
{
echo $sKey , '<br />';
}
echo '<hr />';
echo $sKey;
Насчёт foreach я действительно был неправ, каюсь. Ключ в памяти остаётся. Ман подсказал: Warning! Reference of a $value and the last array element remain even after the foreach loop. It is recommended to destroy it by unset().
Использование цикла со счётчиком в условиях первоначальной задачи не является ошибкой. Просто когда писали этот код, никто не задумывался о том, что ключи могут в перспективе изменяться. И если ввести в конфигурационный массив символьный ключ то всё вообще полетит к чёртовой матери. Такие вот дела. Просто good practice.
слово говно — 糞 (くそ, читается как [кусо] или [ксо])
Говно - друг или враг?
Говно - друг или враг? Умы прогрессивной общественности давно заняты этим вопросом, изучается метафизика говна и пишутся стихи.
Что думают об этом наши форумчане? Совершенно нелишне будет аргументировать свою позицию.
Я, например, считаю, что говно в целом дружественная нам субстанция. По крайней мере в физическом смысле.
1. Это хорошее удобрение.
2. Из говна делают конфетки. Во всяком случае - пытаются. В любом случае - это творчески развивает личность.
3. Говно прекрасно плавает. Соответственно, чем больше в вас говна, тем меньше шансов утонуть.
4. Говно производят абсолютно все люди, что вполне может способствовать их сближению.
Где мои посылки из поднебесной?!