+163
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
- 35
- 36
- 37
- 38
- 39
- 40
- 41
- 42
- 43
- 44
<?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)"
Запостил: user654321,
20 Октября 2010
Oleg_quadro 20.10.2010 14:42 # +2
LOL
Vasiliy 20.10.2010 15:34 # −1
использование одной из особенностей пыхи. Здесь НАХ не нужная. Любовь к копипасту и наплевательство на принятые нормы очевидна
if (-1 == $select) просто убило.
Dreyk 20.10.2010 21:24 # +2
Что не так? Я так все ифы пишу, вначале чему равно, а потом что - уберегает от случайного $select = -1 присваивания вместо сравнения, ну и сразу видно чему равно, а то иногда строки длинные
switch(1) здесь канеша не в тему =)
Lure Of Chaos 20.10.2010 16:45 # +1
if (!isset($v['all'])) continue;
if (0==$v['all']) continue;
Uchkuma 25.10.2010 19:22 # +1
Lure Of Chaos 25.10.2010 22:48 # 0