Насколько существенна такая оптимизация?

Feb 12, 2009 15:38

reset($aZoneIds);
while (list($key, $zoneId) = each($aZoneIds)) {...}

UPD - фрагмент из кода серьёзного ad-cервера. А люди, успешно занимающиеся рекламой, деньги считать умеют

UPD2 Разница между each и foreach таки нашлась:
Unless the array is referenced, foreach operates on a copy of the specified array and not the array itself.

Leave a comment

zerkms February 12 2009, 13:40:12 UTC
ни на сколько. оптимизировать нужно то, что тормозит

Reply

sb16 February 12 2009, 14:14:48 UTC
Ответ неверный. dasboot ниже дал точный ответ.

Reply

phorror February 12 2009, 14:17:42 UTC
ой, не надо вот этого.

Reply

sb16 February 12 2009, 14:25:59 UTC
Не ну почему же? Может чуваку нравится циклы миллионами гонять? Может он AI пишет какой-то, который целыми сутками циклы гонят. Почему нет? (:

Reply

phorror February 12 2009, 14:28:16 UTC
гонял бы - сам бы увидел, помощь со стороны не потребовалась бы.
об этом и речь. нахрена любая оптимизация, если о её эффективности у посторонних спрашивать?

Reply

ezdakimak February 12 2009, 15:40:19 UTC
Ну и пусть гоняет. Может админы на камне суп греют, а с оптимизацией кода придется ещё за микроволновку платить! И где экономия? Где оптимизация!?

Reply

sb16 February 12 2009, 15:45:06 UTC
Ой не подумал! Ой горе мне! Бедные админы...

Reply

zerkms February 13 2009, 05:55:28 UTC
если он гоняет циклы миллионами - то профайлинг покажет это. и тогда вопрос будет звучать иначе: как конкретно в этом случае, конкретно в этой задаче конкретно эти строчки оптимизировать.
вполне вероятно, что если такое произошло - то нужно выбирать более оптимальный алгоритм, а не сравнивать, быстрее echo $a . $b или echo $a , $b;

Reply

zerkms February 13 2009, 05:57:23 UTC
а кто собственно мерило точности? предъявите, пожалуйста, линейку точности ответов.

Reply

nickodimm February 12 2009, 14:34:08 UTC
см. апдейт.. код правильный и отнюдь не wtf..

Reply

pigmeich February 12 2009, 14:39:54 UTC
Про wtf тут никто и не говорит.

Reply


Leave a comment

Up