Хак поиска по форуму — Page 2

DrDespero

1 июл 2010 г., 20:41

Лучше выделять жирным, представьте, если у кого-то красный фон. А в сочетании некоторый цветов вообще может "резать" глаза!

AlkatraZ

1 июл 2010 г., 20:48

╭∩╮ (`-`) ╭∩╮

Хак нужно дорабатывать, в таком виде, как есть, не годится.
Если выделяемое слово попадает на ссылку. появляются ошибки XHTML

VSCher

1 июл 2010 г., 20:56

DrDespero (01.07.2010/21:41)
Лучше выделять жирным, представьте, если у кого-то красный фон. А в сочетании некоторый цветов вообще может "резать" глаза!

Поэтому и сделали отдельный класс - ксли тема крассная то просто в классе ред меняем крассный на какой то другой

Krite

1 июл 2010 г., 21:59

AlkatraZ (01.07.2010/21:48)
Хак нужно дорабатывать, в таком виде, как есть, не годится.Если выделяемое слово попадает на ссылку. появляются ошибки XHTML

Угу заметил, но это не сложно preg_match_all() рулит, получаем ссылку вырезаем, после подсветки вставляем. надо еще подумать

Krite

2 июл 2010 г., 8:33

Исправил xml ошибку

Прикрепленные файлы:

698

(3.10 KB)

forum_search2.zip

AlkatraZ

2 июл 2010 г., 9:39

╭∩╮ (`-`) ╭∩╮

Krite (02.07.2010/09:33)
Исправил xml ошибку

Молодец, зачод!

Щас то, что надо.
Сегодня прогоню через профилировщик, если нагрузка на сервер в разумных пределах, то включим в оф. версию

Krite

2 июл 2010 г., 9:42

Можно в принципе еще каптчу прикрутить, если частые запросы то показывать ее, как в поисковиках .

AlkatraZ

2 июл 2010 г., 9:55

╭∩╮ (`-`) ╭∩╮

Щас глянул в профилировщике, все отлично, сильной нагрузки нет, включим в оф. версиюю.
Только еще немного переделай вот что:
---
Убери этот быдлокод:
function strtolower_ru($text){}

На кой хрен городить эту функцию, если в РНР есть встроенная mb_strtolower()
Переделай на нее и в остальном, пока замечаний нет.
Как переделаешь, кидай сюды и включу в репозиторий.

Krite

2 июл 2010 г., 10:02

Ок. Как раз еще одну ошибочку нашел

AlkatraZ

2 июл 2010 г., 10:03

╭∩╮ (`-`) ╭∩╮

Вот подробные результаты профилировки.
Как видит, общее время выполнения находится в допустимых нормах, а функция подсветки практически ничего от себя не добавляет
З.Ы.
Для наглядности, профилировку делал на огромной базе форума Газенвагена.

Прикрепленные файлы:

655

(24.58 KB)

search_profiling.gif