Javascript-форум (https://javascript.ru/forum/)
-   Элементы интерфейса (https://javascript.ru/forum/dom-window/)
-   -   критика jq-виджета (https://javascript.ru/forum/dom-window/34377-kritika-jq-vidzheta.html)

zlodiak 01.01.2013 17:01

критика jq-виджета
 
уважаемые эксперты, скажите пожалуйста что в этом виджете написано нехорошо?

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

melky 01.01.2013 19:50

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

есть куча статей по оптимизации, сюда их можно применить.

и ещё, возможности расширения виджета нет - каК, например, уже при использовании, указать эффект при появлении\скрытии виджета, или же другой сss-класс для виджета?

DjDiablo 02.01.2013 12:39

мёртворождённый плагин.

1е. Никто небудет вызывать для каждой ссылки
new Tooltip({
			  elem: $('#link'),
			  html: "подсказка длиннее, чем элементэлементэлемент"
});


Это если у меня сто ссылок на странице мне надо сто раз твою функцию вызвать чтоли ?
а если я удалю одну из ссылок, мне чо искать в скрипте код и удалять его ??? (я ленивый, я нехочу :) )

2е. Если у меня по ajax подгружается часть страницы с контентом и в этом контенте есть подсказки, тот как подключить твой плагин к этим подсказкам, а самое главное где взять текст для подсказок ????

Вывод: если представить что я вебмастер а ты разработал плагин для меня. То ты меня обрёк на написание кода для каждой ссылки на каждой странице, практически похоронил мне возможность использования плагина вместе с ajax. Меня такое решение не устраивает совершенно, я считаю что задание провалено и с работой ты несправился, денег платить тебе небуду !:D

как заказчик я хочу по возможности иметь дело с html вместо непонятного мне javascript, я хочу использовать ajax и как можно меньше думать о том как связать твой плагин с ajax (в идеале я вообще не должен об этом думать)

ничего личного :) просто пытаюсь втолковать что ты сделал нетак (ты же просил критику :) ).

О коде рассуждать пока немогу, так как задачу ты нерешил. Твой код положит проект при расширении так как превратит жизнь верстальщика/вебмастера/контент менеджера/или когото там ещё в настоящий ад. Плагин в итоге придётся выпиливать из проекта и заменять чем нибудь вменяемым.

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

zlodiak 02.01.2013 16:50

спасибо за мнение. я неудачно поставил вопрос, меня все таки больше интересует в правильном ли стиле оформлен ООП-код

danik.js 04.01.2013 16:58

методы должны быть в прототипе


Часовой пояс GMT +3, время: 04:55.