Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
<?php
// ...
////////////////
$_engines = unserialize(SENGINES);
if (0 == count($loading)) return -3;
$select=-1;
$delta=0;
$me=count($_engines);
$ms=count($args);
if (count($loading))
foreach ($loading as $k=>$v)
{
if (!isset($v['all'])) continue;
if (0==$v['all']) continue;
$tmp=0;
switch(1)
{
case $ms > 0:
foreach ($args as $engine)
{
if (isset($v[$engine]))
$tmp += (float) $v[$engine]/(float) $v['all'];
}
$tmp=$tmp/$ms;
break;
case $me > 0:
foreach ($_engines as $engine)
{
if (isset($v[$engine]))
$tmp += (float) $v[$engine]/(float) $v['all'];
}
$tmp=$tmp/$me;
break;
}
if ($tmp > $delta) {$delta=$tmp;$select=$k;}
}
if (-1 == $select) return false;
////////////////
// ...
Автор этого убеждает, что тут где-то производится сортировка.
Кстати, это, #4071 и #4215 кодил один и тот же автор. Поэтому тут можно наблюдать десериализацию массива из константы, возвращение непонятных отрицательных статусов, очень информативное именование переменных, копипаст и самое классное — "if для лохов, я использую switch(1)"
тут дело не в if switch(1) это вариант switch(true)
использование одной из особенностей пыхи. Здесь НАХ не нужная. Любовь к копипасту и наплевательство на принятые нормы очевидна
if (-1 == $select) просто убило.
> if (-1 == $select) просто убило.
Что не так? Я так все ифы пишу, вначале чему равно, а потом что - уберегает от случайного $select = -1 присваивания вместо сравнения, ну и сразу видно чему равно, а то иногда строки длинные
switch(1) здесь канеша не в тему =)
LOL
использование одной из особенностей пыхи. Здесь НАХ не нужная. Любовь к копипасту и наплевательство на принятые нормы очевидна
if (-1 == $select) просто убило.
Что не так? Я так все ифы пишу, вначале чему равно, а потом что - уберегает от случайного $select = -1 присваивания вместо сравнения, ну и сразу видно чему равно, а то иногда строки длинные
switch(1) здесь канеша не в тему =)
if (!isset($v['all'])) continue;
if (0==$v['all']) continue;