if(strpos($name, 'fetchRowBy') === 0) {
array_unshift($arguments, substr($name, 10));
return call_user_func_array(array($this, 'fetchRowBy'), $arguments);
}21.07 10:14
|
| | | | | |
|
←
21.07.2008, Nº 1790
Habr, WTF?Хабр не устаёт меня удивлять всё снижающимся качеством статей и кода. Ну вот что это:
21.07 10:14
← 21.07.2008, Nº 1790 |
||||||||||||||||||||||||||||||||||||||||
substr($name, 0, 10)?
сервак не отвечает, хм
strpos($name, 'fetchRowBy') === 0
Хабр?
а что не так с strpos? это самая эффективная проверка substr() менее эффективен из-за дополнительной аллокации памяти под строку
substr_compare
Код не мой, я просто разместил его. :) На самом деле - этот кусок был взят <a href="http://svn.symfony-project.com…ropelFinder.php">отсюда</a> (ctrl-c/ctrl+v, бездумно практически :)), и это, по-моему единственное, что делал не я, а Женя тут придирается. :) Да и вообще: <pre><code class="php">strpos($name, 'fetchAllBy') === 0) substr($name, 0, 10) === 'fetchAllBy'</code></pre> На небольших строках разницы не вижу, а методов длинной в 10k символов я не видел. :)
Жень, добавил бы сюда поддержку html, что ли :)
Нафига тут HTML? :)
Не видишь разницу? :) Представь, что у тебя 10k раз это вызвалось, теперь разница заметна? :) Всякую фиговину в вебе надо представлять внутри ещё одного внешнего цикла, который создают пользователи.
Представь, что у тебя 10k раз это вызвалось, теперь разница заметна? :)Надо проверить ;)
substr_compare Люблю, блин, php. Можно на нём десять лет писать, и всё равно новая функция где-нибудь вылезет.
Так, специально для любителей экономии на спичках. Миллион (!) оборотов цикла: $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 сек
Ну вот, уже проверили :)
Неправильный метод тестирования. Нужно проверять два случая, а не один.
Неправильно проверили.
Из-за «экономии на спичках» AdMe.ru хорошо работает на довольно среднем железе (у меня на предыдущем месте работы десктоп был мощнее того сервера).
в тесте не учтен случай с длинными строками, не содержащими префикса ; )
Так мануалы надо читать :)
http://ez.no/fr/content/downlo…romance_php.pdf Кстати.
Кроме всего прочего, надо всё это запустить несколько раз и подсчитать среднее.
$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 Ещё что-нибудь?
Версия PHP? Видимо, Ilia Alshanetsky должен обновить свой документ.
Да, хабр видимо в дауне был. Или с ДНСом у меня что-то не так. Я бы использовал substr, и логичней и как показывает последний тест - побыстрей. Но скорость в таких случаях очень мизерная, ими можно принебречь на мелких проектах.. Пойду добавлять в http://kurapov.name/technology/web/php/php_speed/
Я мелкими проектами уже много лет не занимаюсь, так что со своей колокольни сужу. Ни и во framework'ах надо аккуратнее быть — неизвестно в каких проектах они будут использоваться.
http://ez.no/fr/content/downlo…romance_php.pdf> Objet non disponible Выложи где-нибудь?
$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."; Такого метода быть не может ;)
Может быть оно? http://ilia.ws/files/frankfurt_perf.ppt
Полностью согласен. Это кстати общая тенденция большинства "geek"'ов, которую приходиться отучать, чтобы другие члены команды не страдали. Понапишут развёртавыемые циклы и функции по 1000 строк с названием "mtzv" и гадай потом что да как. Одна из главных целей в любом продукте - это лёгкость в поддержке, если код понятен, легко расширяем, значит направление верно. Из моего опыта в 90% случаях ботлнеками являются совсем другие вещи, чем мусоличть доли секунд при обработке миллиона операций. Конечно, это всё относительно и дополнительные поблажки могут быть добавлены для малых проектов (команда 2-3 человека), но когда команды 5 человек и более и разработка идёт по agile, то надо интегрироваться в команду а не являтся её тормозом и переписывать работающие куски из-за того, что он работает на 0.05сек медленее при 1m операций. Любой мэнеджер убил бы за такое. :) По поводу железа... благо нынче он достаточно дешовое и тот же quad-core с несколькими гигабайтами памяти можно купить за приемлимые деньги (а уж тем более для тех кто планирует заниматься рекламой).
Это известный PDF Ilia Alshanetsky об оптимизации PHP, он его каждый год выпускает, в инете сотни копий.
О покупке часто речь не идёт. Выделенные сервера довольно часто арендуются. Это даёт массу плюсов при выходе оборудования из строя (к примеру, на adme винты летят от нагрузки раз в полгода). Аренда хорошего сервера — это совсем уже другие деньги. Иногда, существенные, особенно, если арендуется не один сервер. У меня тоже очень большой опыт устранения bottle necks, причём вполне успешный. Последние несколько лет я непрерывно, иногда в реальном времени занимался нагруженными и быстрорастущими проектами.
Вы кстати, сейчас применили троллевский приём: приписать оппоненту какое-то действие и осудить его. Не нужно так делать.
Виноват, отходил. Версия PHP 5.2.0 (cli) под win32. Вообще, проверье сами, это же несложно: <?php $str = "fetchRowByLorem"; $pref = "fetchRowBy"; $preflen = strlen($pref); $t =- microtime(true); for($i=0;$i<1000000;$i++) { (strpos($str, $pref) === 0); //(substr($str, 0, $preflen) === $pref); //(substr_compare($str, $pref, 0, $preflen) === 0); } $t += microtime(true); echo $t."\n"; ?> Вообще, по моему опыту «спичечной» оптимизации php-кода, тормоза возникают в основном в интерпретируемом коде. То есть, библиотечные функции сами по себе работают очень шустро, но лишнее присваивание или лишняя переменная (в PHP-коде) в цикле может сильно просадить скорость. И второй момент — библиотеки в PHP тоже написаны… довольно фиговенько. Даже если есть специальная функция под какую-то задачу, _обычно_ гораздо шустрее бывает использовать более простую функцию, реализующую какой-нибудь классический алгоритм. Как здесь — поиск в строке. Это старая функция, явно использующая какой-то стандартный сишный код, поэтому я бы априори ожидал, что она будет самая скоростная. Так оно, в общем, и оказалось. Substr сам по себе мог бы быть ещё шустрее, но тут наружу надо доставать результат и сравнивать его уже в PHP. Это замедляет. Так что, по-моему, в исходном коде гораздо больший потенциал «спичечной» оптимизации содержится во второй и третьей строках. Но искать альтернативы и проверять лень:) Что касается последнего теста — вспоминаем специфику задачи. Там _всегда_ будут короткие строки.
Да я уже проверил. Зачем в вашем коде «$t =- microtime(true)»? У PHP достаточно много слабых мест, по этой теме есть масса документов. Например, циклы достаточно медленные. С PHP 5.3.0 многое должно измениться. Кстати, сколько прогонов перед усреднением вы делали?
Не совсем понятно причём тут "троллевский приём", ну да ладно... я просто высказывал свою точку зрения и делился своим опытом.
Я же расписал какой именно приём применён.
Да я уже проверил. Зачем в вашем коде «$t =- microtime(true)»?В смысле, зачем? Чтобы float получить. Кстати, сколько прогонов перед усреднением вы делали?Прогонял раз по несколько. Это на домашней машине, она ничем другим не занималась, так что результат был стабильным ± единички во втором знаке. 5.3 — да, скорей бы уж.
«-=» зачем.
$t =- microtime(true); … $t += microtime(true); Для красоты внешнего вида, исключительно:) |
| Евгений Степанищев (imbolk@gmail.com)
t=0.075
|
