Показать сообщение отдельно
  #59 (permalink)  
Старый 16.01.2013, 14:28
Профессор
Отправить личное сообщение для DjDiablo Посмотреть профиль Найти все сообщения от DjDiablo
 
Регистрация: 04.02.2011
Сообщений: 1,815

Пробежался по библиотеке в http://jru-framework.ru/
первое что бросилось в глаза, это использования только одной функции в селекторах querySelectorAll и больше нечего.

в то время как обычно стараются использовать самый подходящую для селектора функцию
пример из моей либы, но подобный код есть почти во всех фреймворках
classSelectorRE : /^\.([\w-]+)$/,
idSelectorRE : /^#([\w-]+)$/,
tagSelectorRE : /^[\w-]+$/,

qsa:function(element,selector){
  if ( this.idSelectorRE.test(selector) ) return [ element.getElementById( selector.replace("#","") ) ];
  if ( this.classSelectorRE.test(selector) ) return element.getElementsByClassName(selector.replace(".","")) ;
  if ( this.tagSelectorRE.test(selector) ) return element.getElementsByTagName(selector);
  return element.querySelectorAll(selector); 
}

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

Последний раз редактировалось DjDiablo, 16.01.2013 в 14:43.
Ответить с цитированием