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

Не до комментариев, с форматированием кода сначала разберись

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

Есть некоторые расхождения в стиле от ф-и к функции
Код:
} else {
Код:
}
	else {

тут тоже перевод строки лишний
Код:
for (var property in css) {
		element.style[property] = css[property];
	}

	return element;

Тут я бы выровнял по знаку =
Код:
Notifier.HEADER_TEXT = 'Вообще-то, :';
Notifier.DETAILS_TEXT = 'Что именно не поддерживает мой браузер?';
Notifier.CLOSE_TEXT = 'закрыть';

Код:
if (key in target)return;
где пробел?

Код:
var fix = new Fix;
var notifier = new Notifier;
сделай один var и выровняй по =. Не красиво из-за разной длины переменных? подбери имена получше!

Код:
for (var i = 0; i < elements.length; i++) {
	if (elements[i] === this) {
		return true;
	}
}
Код:
for (var i = 0; i < elements.length; i++) if (elements[i] === this) {
	return true;
}
так вроде лучше. нет?


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

Последний раз редактировалось Tim, 02.05.2013 в 21:02.
Ответить с цитированием