Показать сообщение отдельно
  #5 (permalink)  
Старый 02.05.2009, 11:51
Аватар для Илья Кантор
Администратор
Отправить личное сообщение для Илья Кантор Посмотреть профиль Найти все сообщения от Илья Кантор
 
Регистрация: 25.05.2007
Сообщений: 1,224

Сообщение от Riim Посмотреть сообщение
1. в commonHandle переменная handlers попадает в window. Нужно var добавить.
Ага, почистил.

Цитата:
2. непонятно зачем в elem.handle проверка (typeof Event !== "undefined")
В статье про это сказано - это для фильтрации редкой ошибки с unload страницы.


Цитата:
3. содержимое commonHandle вынесено в отдельную функцию. Вероятно для увеличения производительности Event.add, но это делается в ущерб производительности elem.handle, которая имеет заметно больший приоритет.
Это чтобы не создавать кучу лишних функций commonHandle и не расходовать тем самым память. Конечно, можно ее заинлайнить.

Цитата:
4. конструкции типа:
...
Там еще множество разных оптимизаций можно применить, но все они в ущерб читабельности. А эта вроде нет.
Да, приятное исправление, сделал его. Особо серьезных оптимизаций там даже в ущерб читабельности не сделаешь имхо.
Ответить с цитированием