Habr, WTF?

Хабр не устаёт меня удивлять всё снижающимся качеством статей и кода. Ну вот что это:

if(strpos($name, 'fetchRowBy') === 0) {
      array_unshift($arguments, substr($name, 10));
      return call_user_func_array(array($this, 'fetchRowBy'), $arguments);
    }

Ещё бы регулярное выражение использовал, ей-богу!

Комментарии 39

jimidini (jimidini.ya.ru) № 5
Комментарий для Евгения Степанищева:

а что не так с strpos? это самая эффективная проверка
substr() менее эффективен из-за дополнительной аллокации памяти под строку
FX Poster (blog.fxposter.org) № 7
Код не мой, я просто разместил его. :)

На самом деле — этот кусок был взят
Евгений Степанищев (bolknote.ru) автор № 10
Комментарий для blog.fxposter.org:

Не видишь разницу? :) Представь, что у тебя 10k раз это вызвалось, теперь разница заметна? :) Всякую фиговину в вебе надо представлять внутри ещё одного внешнего цикла, который создают пользователи.
FX Poster (blog.fxposter.org) № 11

Комментарий для Евгения Степанищева:

Представь, что у тебя 10k раз это вызвалось, теперь разница заметна? :)

Надо проверить ;)

david-m.livejournal.com № 12
substr_compare

Люблю, блин, php. Можно на нём десять лет писать, и всё равно новая функция где-нибудь вылезет.

david-m.livejournal.com № 13
Так, специально для любителей экономии на спичках. Миллион (!) оборотов цикла:

$str = «fetchRowByName»;
$pref = «fetchRowBy»;
$preflen = strlen($pref);

(strpos($str, $pref) === 0) — 0.45 сек
(substr($str, 0, $preflen) == $pref) — 0.68 сек
(substr($str, 0, $preflen) === $pref) — 0.55 сек (! — хозяйке на заметку)
(substr_compare($str, $pref, 0, $preflen) == 0) — 0.84 сек
Евгений Степанищев (bolknote.ru) автор № 15
Комментарий для david-m.livejournal.com:

Неправильный метод тестирования. Нужно проверять два случая, а не один.
Евгений Степанищев (bolknote.ru) автор № 17
Комментарий для david-m.livejournal.com:

Из-за «экономии на спичках» AdMe.ru хорошо работает на довольно среднем железе (у меня на предыдущем месте работы десктоп был мощнее того сервера).
arty (arty.name) № 18
Комментарий для david-m.livejournal.com:

в тесте не учтен случай с длинными строками, не содержащими префикса ; )
Евгений Степанищев (bolknote.ru) автор № 19
Комментарий для david-m.livejournal.com:

Так мануалы надо читать :)
Евгений Степанищев (bolknote.ru) автор № 21
Комментарий для david-m.livejournal.com:

Кроме всего прочего, надо всё это запустить несколько раз и подсчитать среднее.
david-m.livejournal.com № 22
$str = «Lorem ipsum dolor sit amet, consectetuer adipiscing elit, sed diam nonummy nibh euismod tincidunt ut laoreet dolore magna aliquam erat volutpat. Ut wisi enim ad minim veniam, quis nostrud exerci tation ullamcorper suscipit lobortis nisl ut aliquip ex ea commodo consequat. Duis autem vel eum iriure dolor in hendrerit in vulputate velit esse molestie consequat, vel illum dolore eu feugiat nulla facilisis at vero eros et accumsan et iusto odio dignissim qui blandit praesent luptatum zzril delenit augue duis dolore te feugait nulla facilisi.»;
$pref = «fetchRowBy»;

(strpos($str, $pref) === 0) — 0.67
(substr($str, 0, $preflen) === $pref) — 0.58
(substr_compare($str, $pref, 0, $preflen) == 0) — 0.82

Ещё что-нибудь?
Евгений Степанищев (bolknote.ru) автор № 23
Комментарий для david-m.livejournal.com:

Версия PHP?

Видимо, Ilia Alshanetsky должен обновить свой документ.
Артём Курапов (kurapov.name) № 24
Да, хабр видимо в дауне был. Или с ДНСом у меня что-то не так.

Я бы использовал substr, и логичней и как показывает последний тест — побыстрей. Но скорость в таких случаях очень мизерная, ими можно принебречь на мелких проектах..
Пойду добавлять в http://kurapov.name/technology/web/php/php_speed/
Евгений Степанищев (bolknote.ru) автор № 25
Комментарий для kurapov.name:

Я мелкими проектами уже много лет не занимаюсь, так что со своей колокольни сужу. Ни и во framework'ах надо аккуратнее быть — неизвестно в каких проектах они будут использоваться.
FX Poster (blog.fxposter.org) № 27

Комментарий для david-m.livejournal.com:

$str = «Lorem ipsum dolor sit amet, consectetuer adipiscing elit, sed diam nonummy nibh euismod tincidunt ut laoreet dolore magna aliquam erat volutpat. Ut wisi enim ad minim veniam, quis nostrud exerci tation ullamcorper suscipit lobortis nisl ut aliquip ex ea commodo consequat. Duis autem vel eum iriure dolor in hendrerit in vulputate velit esse molestie consequat, vel illum dolore eu feugiat nulla facilisis at vero eros et accumsan et iusto odio dignissim qui blandit praesent luptatum zzril delenit augue duis dolore te feugait nulla facilisi.»;

Такого метода быть не может ;)

Owner (companyowner.myopenid.com) № 29
Комментарий для david-m.livejournal.com:

Полностью согласен. Это кстати общая тенденция большинства «geek»'ов, которую приходиться отучать, чтобы другие члены команды не страдали. Понапишут развёртавыемые циклы и функции по 1000 строк с названием «mtzv» и гадай потом что да как. Одна из главных целей в любом продукте — это лёгкость в поддержке, если код понятен, легко расширяем, значит направление верно. Из моего опыта в 90% случаях ботлнеками являются совсем другие вещи, чем мусоличть доли секунд при обработке миллиона операций.

Конечно, это всё относительно и дополнительные поблажки могут быть добавлены для малых проектов (команда 2-3 человека), но когда команды 5 человек и более и разработка идёт по agile, то надо интегрироваться в команду а не являтся её тормозом и переписывать работающие куски из-за того, что он работает на 0.05сек медленее
при 1m операций. Любой мэнеджер убил бы за такое. :)

По поводу железа… благо нынче он достаточно дешовое и тот же quad-core с несколькими гигабайтами памяти можно купить за приемлимые деньги (а уж тем более для тех кто планирует заниматься рекламой).
Евгений Степанищев (bolknote.ru) автор № 31
О покупке часто речь не идёт. Выделенные сервера довольно часто арендуются. Это даёт массу плюсов при выходе оборудования из строя (к примеру, на adme винты летят от нагрузки раз в полгода).

Аренда хорошего сервера — это совсем уже другие деньги. Иногда, существенные, особенно, если арендуется не один сервер.

У меня тоже очень большой опыт устранения bottle necks, причём вполне успешный. Последние несколько лет я непрерывно, иногда в реальном времени занимался нагруженными и быстрорастущими проектами.
Евгений Степанищев (bolknote.ru) автор № 32
Вы кстати, сейчас применили троллевский приём: приписать оппоненту какое-то действие и осудить его. Не нужно так делать.
Евгений Степанищев (bolknote.ru) автор № 30
Комментарий для blog.fxposter.org:

Это известный PDF Ilia Alshanetsky об оптимизации PHP, он его каждый год выпускает, в инете сотни копий.
david-m.livejournal.com № 33
Виноват, отходил. Версия PHP 5.2.0 (cli) под win32. Вообще, проверье сами, это же несложно:

Вообще, по моему опыту «спичечной» оптимизации php-кода, тормоза возникают в основном в интерпретируемом коде. То есть, библиотечные функции сами по себе работают очень шустро, но лишнее присваивание или лишняя переменная (в PHP-коде) в цикле может сильно просадить скорость. И второй момент — библиотеки в PHP тоже написаны… довольно фиговенько. Даже если есть специальная функция под какую-то задачу, _обычно_ гораздо шустрее бывает использовать более простую функцию, реализующую какой-нибудь классический алгоритм. Как здесь — поиск в строке. Это старая функция, явно использующая какой-то стандартный сишный код, поэтому я бы априори ожидал, что она будет самая скоростная. Так оно, в общем, и оказалось. Substr сам по себе мог бы быть ещё шустрее, но тут наружу надо доставать результат и сравнивать его уже в PHP. Это замедляет.

Так что, по-моему, в исходном коде гораздо больший потенциал «спичечной» оптимизации содержится во второй и третьей строках. Но искать альтернативы и проверять лень:)

Что касается последнего теста — вспоминаем специфику задачи. Там _всегда_ будут короткие строки.
Евгений Степанищев (bolknote.ru) автор № 34
Комментарий для david-m.livejournal.com:

Да я уже проверил. Зачем в вашем коде «$t =- microtime(true)»?

У PHP достаточно много слабых мест, по этой теме есть масса документов. Например, циклы достаточно медленные. С PHP 5.3.0 многое должно измениться.

Кстати, сколько прогонов перед усреднением вы делали?
Owner (companyowner.myopenid.com) № 35
Комментарий для Евгения Степанищева:

Не совсем понятно причём тут «троллевский приём», ну да ладно… я просто высказывал свою точку зрения и делился своим опытом.
Евгений Степанищев (bolknote.ru) автор № 36
Комментарий для companyowner.myopenid.com:

Я же расписал какой именно приём применён.
david-m.livejournal.com № 37
Да я уже проверил. Зачем в вашем коде «$t =- microtime(true)»?

В смысле, зачем? Чтобы float получить.

Кстати, сколько прогонов перед усреднением вы делали?

Прогонял раз по несколько. Это на домашней машине, она ничем другим не занималась, так что результат был стабильным ± единички во втором знаке.

5.3 — да, скорей бы уж.

david-m.livejournal.com № 39
Комментарий для Евгения Степанищева:

$t =- microtime(true);
…
$t += microtime(true);

Для красоты внешнего вида, исключительно:)