24.06.2011, 23:21
|
Интересующийся
|
|
Регистрация: 16.01.2009
Сообщений: 20
|
|
Да, тот самый =)
|
|
25.06.2011, 00:07
|
Интересующийся
|
|
Регистрация: 16.01.2009
Сообщений: 20
|
|
Ваша функция toArray имеет ряд недостатков. Как по скорости, так и по логике. Лучше использовать что-то типа такого:
var toArray = function(o) {
return Array.prototype.slice.call(o);
}
|
|
25.06.2011, 00:11
|
|
Новичок на форуме
|
|
Регистрация: 24.06.2011
Сообщений: 4
|
|
Сообщение от Shock
|
Код, кстати, на удивление хорош. Конечно, есть некоторые ошибки, кое-где видны не очень хорошие знания JavaScript, но в общем - код на редкость качественный
Например:
this.setListener = function(type, listener) {
switch (type) {
case 'mouseover':
this.mouseover.set(listener);
break;
case 'mouseout':
this.mouseout.set(listener);
break;
case 'mousemove':
this.mousemove.set(listener);
break;
case 'mousedown':
this.mousedown.set(listener);
break;
case 'mouseup':
this.mouseup.set(listener);
break;
case 'mouseclick':
this.mouseclick.set(listener);
break;
}
}
Вполне можно заменить на:
this.setListener = function(type, listener) {
if ( ['mouseover', 'mouseout', 'mousemove', 'mousedown', 'mouseup', 'mouseclick'].indexOf(type) === -1 ) return;
this[type].set(listener);
}
|
Да, профессионально программированием занимаюсь менее года.
Сообщение от Shock
|
Да, тот самый =)
|
Охренеть
Сообщение от Shock
|
Судя по коду вы хорошо знаете другой язык. Например, Java
|
Действительно, изучать программирование я начал именно с Java
Сообщение от Shock
|
Ваша функция toArray имеет ряд недостатков. Как по скорости, так и по логике. Лучше использовать что-то типа такого:
var toArray = function(o) {
return Array.prototype.slice.call(o);
}
|
С этой функцией вышла целая история, надо было просто посмотреть как она реализована в какой-нибудь популярной библиотеке.
|
|
25.06.2011, 00:14
|
Интересующийся
|
|
Регистрация: 16.01.2009
Сообщений: 20
|
|
Чего охренеть?
Я Вам вот что советую - не стоит распылять силы OS сообщества - лучше присоединяйтесь к LibCanvas
|
|
25.06.2011, 00:17
|
|
Новичок на форуме
|
|
Регистрация: 24.06.2011
Сообщений: 4
|
|
Ну как что, Вы, можно сказать, великий Гуру для меня
С удовольствием присоединюсь, все никак руки не дойдут разобраться с AtomJS.
|
|
25.06.2011, 00:19
|
Интересующийся
|
|
Регистрация: 16.01.2009
Сообщений: 20
|
|
Пишите в Jabber shock@jabber.com.ua , пообщаемся)
|
|
25.06.2011, 21:26
|
Новичок на форуме
|
|
Регистрация: 19.02.2008
Сообщений: 9,177
|
|
Shock,
вы под IE не пишите, как я понимаю?
Сообщение от Shock
|
if ( ['mouseover', 'mouseout', 'mousemove', 'mousedown', 'mouseup', 'mouseclick'].indexOf(type) === -1 ) return;
|
indexOf у массивов нет в IE.
Сообщение от Shock
|
return Array.prototype.slice.call(o);
|
В IE с DOM-коллекциями такой финт не пройдёт.
|
|
25.06.2011, 21:30
|
Интересующийся
|
|
Регистрация: 16.01.2009
Сообщений: 20
|
|
Зачем в Canvas-фреймворке костыли для IE, если даже с ними там ничего не заработает?
|
|
25.06.2011, 21:39
|
Новичок на форуме
|
|
Регистрация: 19.02.2008
Сообщений: 9,177
|
|
Сообщение от Shock
|
Зачем в Canvas-фреймворке костыли для IE
|
Действительно.
|
|
25.06.2011, 23:42
|
|
Профессор
|
|
Регистрация: 15.02.2011
Сообщений: 471
|
|
Ну почему же, в IE9 все замечательно работает, причем даже быстрее чем в других браузерах.
|
|
|
|