Показать сообщение отдельно
  #8 (permalink)  
Старый 12.03.2013, 14:58
Аватар для Tim
Tim Tim вне форума
Профессор
Отправить личное сообщение для Tim Посмотреть профиль Найти все сообщения от Tim
 
Регистрация: 05.06.2009
Сообщений: 1,703

вообще если понятность не очень страдает то я экономлю место по вертикали и не создаю переменные которые используются 1, 2 раза. лучше чтобы код по вертикали был компактнее.

if(!(this instanceof Imgt)) {
    return new Imgt(id);
}
 
this.element = document.getElementById(id);


тут явно строка пустая не нужна например. сверху от условия да, чтобы не сливалось можно пропустить.
из-за таких лишних строк кроме одной ф-й ни чего на экране не видно, не видно как она связана с другим кодом.

p.s.: хз, многим покажется мелочью и мб они будут правы, но мне глаз режет как то это всё последнее время.

p.s.s: документационные комментарии все мои усилия сводят на нет
__________________
Лучше установить FreeBSD, чем потратить 30 лет на Linux'ы и выяснить какой из них хуже.
Самые главные в жизни вещи - не вещи! (было написано на одном гараже =)
Ответить с цитированием