StyleT Feb 8 2011 at 20:58

Пишем модуль рейтинга для Joomla, или AJAX по-русски

7 min

7.6K

Website development*

From sandbox

-1

Comments 12

wishope Feb 8 2011 at 21:09

по-русски
(grammar-nazi)

-1

StyleT Feb 8 2011 at 21:11

спасибо, поправил

TheShock Feb 8 2011 at 21:31

Очень не понравилось:
1. Жуткий быдлокод на стороне сервера. Ну буду холиварить по поводу того, что голоса неплохо было бы хранить в отдельной таблице many-to-many, но, даже то, что есть сейчас можно было записать без этого десятка повторений одного и того же. Копипаст — зло.
2. Не очень согласен с подходом на стороне клиента. Надо было как минимум отказаться от css в js и использовать классы.

Ну а вообще — вполне на уровне всех остальных модов для Джумлы.

StyleT Feb 8 2011 at 21:35

нет пределов для совершенствования, каждый может переделать и добавить что то по своему усмотрению, порой чем красивее или короче код, тем сложнее его понять новичку, а статья именно для них

-2

TheShock Feb 8 2011 at 22:03

Нельзя новичков учить плохому стилю.
"UPDATE `jos_likerating_news` можно схлопнуть запросто.

А вообще — если бы вы разнесли по методам, сделали легкий рефакторинг, то было бы намного понятнее в том числе новичкам.

StyleT Feb 8 2011 at 22:05

вай, и правда упустил, рефакторинг обязательно сделаю когда напишу компонент рейтинга пользователей и кармы

zupernintendo Feb 8 2011 at 22:00

пример того как ненадо писать модули для Joomla.
модули больше подходят только для вывода каких-либо данных. в данном случае лучше было бы написание компонента.
фронтэнд часть компонента позволяет использовать стандартными средствами любые запросы.
решение предлагаемое tushov.ru/post/568 вообще дурацкий хак и ересь.