Лучше выделять жирным, представьте, если у кого-то красный фон. А в сочетании некоторый цветов вообще может "резать" глаза!
Хак нужно дорабатывать, в таком виде, как есть, не годится.
Если выделяемое слово попадает на ссылку. появляются ошибки XHTML
DrDespero (01.07.2010/21:41)
Лучше выделять жирным, представьте, если у кого-то красный фон. А в сочетании некоторый цветов вообще может "резать" глаза!
Поэтому и сделали отдельный класс - ксли тема крассная то просто в классе ред меняем крассный на какой то другой
AlkatraZ (01.07.2010/21:48)
Хак нужно дорабатывать, в таком виде, как есть, не годится.Если выделяемое слово попадает на ссылку. появляются ошибки XHTML
Угу заметил, но это не сложно preg_match_all() рулит, получаем ссылку вырезаем, после подсветки вставляем. надо еще подумать
Krite (02.07.2010/09:33)
Исправил xml ошибку
Молодец, зачод!
Щас то, что надо.
Сегодня прогоню через профилировщик, если нагрузка на сервер в разумных пределах, то включим в оф. версию
Можно в принципе еще каптчу прикрутить, если частые запросы то показывать ее, как в поисковиках
.
Щас глянул в профилировщике, все отлично, сильной нагрузки нет, включим в оф. версиюю.
Только еще немного переделай вот что:
---
Убери этот быдлокод:
function strtolower_ru($text){}
На кой хрен городить эту функцию, если в РНР есть встроенная mb_strtolower()
Переделай на нее и в остальном, пока замечаний нет.
Как переделаешь, кидай сюды и включу в репозиторий.
Ок. Как раз еще одну ошибочку нашел
Вот подробные результаты профилировки.
Как видит, общее время выполнения находится в допустимых нормах, а функция подсветки практически ничего от себя не добавляет
З.Ы.
Для наглядности, профилировку делал на огромной базе форума Газенвагена.