HelpeR,
все что вы говорите понятно, но зачем использовать, я даже считать не буду просто предположу, 20 переменных вместо одной(1)? ps: размер скрипта слишком большой |
Цитата:
|
var config = {
a1: 1,
a2: 2,
a3: 3
};
|
это то ясно. Я подумал, может какой нибудь особый вариант. Не вижу различий между моим подходом и созданием объекта для удобства пользователя?
|
HelpeR,
В данном случае пользователь я и мне неудобно отслеживать весь мусор, которым ваш подход загаживает моё пространство имён. Не нравиться критика юзайте сами, а не предлагайте мне "гамно" |
Gozar, я всегда положительно отношусь к критике и вам спасибо за подсказку на счет засорения пространства имен. Теперь понял разницу. Буду менять!
|
HelpeR, да и объект тут не при чем. Фигово конечно что столько глобалок, но вопрос не о том даже, а о том что в каждой строке объявляется отдельная переменная. Можно прописать команду var один раз а не 20:
var a=1, b=2, c=3, d=... И насчет сайта - не открывайте страницу улучшений в новом окне - это бесит. |
Надо как-то так:
EffectName.options = {op1: 5, op2: 10};
или так: EffectName.init('elemId', {op1: 5, op2: 10});
еще: EffectName.prototype.defaultOptions = {op1: 5, op2: 10};
EffectName.init('elemId', {op2: 500});// options перезаписывает ключи в defaultOptions
|
Добавил сжатые версии js файлов для скачки. Кстати кто не знает http://shrinksafe.dojotoolkit.org/ думаю самый лучший онлайн сервис по сжатию js файлов
|
HelpeR, http://compressorrater.thruhere.net/ , ага.
|
| Часовой пояс GMT +3, время: 21:32. |