Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #11 (permalink)  
Старый 24.06.2011, 23:21
Интересующийся
Отправить личное сообщение для Shock Посмотреть профиль Найти все сообщения от Shock
 
Регистрация: 16.01.2009
Сообщений: 20

Да, тот самый =)
Ответить с цитированием
  #12 (permalink)  
Старый 25.06.2011, 00:07
Интересующийся
Отправить личное сообщение для Shock Посмотреть профиль Найти все сообщения от Shock
 
Регистрация: 16.01.2009
Сообщений: 20

Ваша функция toArray имеет ряд недостатков. Как по скорости, так и по логике. Лучше использовать что-то типа такого:
var toArray = function(o) {
	return Array.prototype.slice.call(o);
}
Ответить с цитированием
  #13 (permalink)  
Старый 25.06.2011, 00:11
Аватар для DNemo
Новичок на форуме
Отправить личное сообщение для DNemo Посмотреть профиль Найти все сообщения от DNemo
 
Регистрация: 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);
}
С этой функцией вышла целая история, надо было просто посмотреть как она реализована в какой-нибудь популярной библиотеке.
Ответить с цитированием
  #14 (permalink)  
Старый 25.06.2011, 00:14
Интересующийся
Отправить личное сообщение для Shock Посмотреть профиль Найти все сообщения от Shock
 
Регистрация: 16.01.2009
Сообщений: 20

Чего охренеть?
Я Вам вот что советую - не стоит распылять силы OS сообщества - лучше присоединяйтесь к LibCanvas
Ответить с цитированием
  #15 (permalink)  
Старый 25.06.2011, 00:17
Аватар для DNemo
Новичок на форуме
Отправить личное сообщение для DNemo Посмотреть профиль Найти все сообщения от DNemo
 
Регистрация: 24.06.2011
Сообщений: 4

Ну как что, Вы, можно сказать, великий Гуру для меня
С удовольствием присоединюсь, все никак руки не дойдут разобраться с AtomJS.
Ответить с цитированием
  #16 (permalink)  
Старый 25.06.2011, 00:19
Интересующийся
Отправить личное сообщение для Shock Посмотреть профиль Найти все сообщения от Shock
 
Регистрация: 16.01.2009
Сообщений: 20

Пишите в Jabber shock@jabber.com.ua , пообщаемся)
Ответить с цитированием
  #17 (permalink)  
Старый 25.06.2011, 21:26
Новичок на форуме
Отправить личное сообщение для Kolyaj Посмотреть профиль Найти все сообщения от Kolyaj
 
Регистрация: 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-коллекциями такой финт не пройдёт.
Ответить с цитированием
  #18 (permalink)  
Старый 25.06.2011, 21:30
Интересующийся
Отправить личное сообщение для Shock Посмотреть профиль Найти все сообщения от Shock
 
Регистрация: 16.01.2009
Сообщений: 20

Зачем в Canvas-фреймворке костыли для IE, если даже с ними там ничего не заработает?
Ответить с цитированием
  #19 (permalink)  
Старый 25.06.2011, 21:39
Новичок на форуме
Отправить личное сообщение для Kolyaj Посмотреть профиль Найти все сообщения от Kolyaj
 
Регистрация: 19.02.2008
Сообщений: 9,177

Сообщение от Shock
Зачем в Canvas-фреймворке костыли для IE
Действительно.
Ответить с цитированием
  #20 (permalink)  
Старый 25.06.2011, 23:42
Аватар для DreamTheater
Профессор
Отправить личное сообщение для DreamTheater Посмотреть профиль Найти все сообщения от DreamTheater
 
Регистрация: 15.02.2011
Сообщений: 471

Ну почему же, в IE9 все замечательно работает, причем даже быстрее чем в других браузерах.
Ответить с цитированием
Ответ



Опции темы Искать в теме
Искать в теме:

Расширенный поиск