Я нахожу, что не делать, это трудный урок для изучения, чем то, что нужно делать.
По моему опыту, то, что отделяет эксперта от промежуточного, - это способность выбирать из различных, казалось бы, эквивалентных способов сделать одно и то же.
Итак, когда дело доходит до JavaScript , какие вещи вы не должны делать и почему?
Я могу найти много их для Java, но так как типичный контекст JavaScript (в браузере) сильно отличается от Java, мне любопытно посмотреть, что выйдет.
Язык:
Пространство имен, загрязняющее, создавая большой след переменных в глобальном контексте.
Обработчики событий привязки в форме "foo.onclick = myFunc" (нерационально, должен использоваться attachEvent/addEventListener).
Использование eval практически в любом контексте, отличном от JSON
Почти все использование document.write(используйте методы DOM, такие как document.createElement)
Прототипирование объекта Object (BOOM!)
Небольшой это, но выполнение больших количеств строковых конкат с "+" (создание массива и объединение его гораздо более эффективно)
Обращаясь к несуществующей константе undefined
Дизайн/Развертывание:
(обычно), не поддерживающий поддержку noscript.
Не упаковывать ваш код в один ресурс
Ввод встроенных (то есть тело) скриптов в верхней части тела (они блокируют загрузку)
Спецификация Ajax:
не указывая начало, конец или ошибку запроса пользователю
опрос
передача и разбор XML вместо JSON или HTML (при необходимости)
edit: Я продолжаю думать больше!
Помимо уже упомянутых...
Использование конструкции for..in для перебора массивов
(итерации по методам массивов и индексам)
Использование встроенного Javascript как <body onload="doThis();">
(негибкий и предотвращает несколько прослушивателей событий)
Использование конструктора 'Function()'
(плохо по тем же причинам eval() плохо)
Передача строк вместо функций setTimeout или setInterval
(также использует eval() внутренне)
Опираясь на неявные утверждения, не используя точки с запятой
(плохая привычка подбирать и может привести к неожиданному поведению)
Использование/*.. */для блокировки строк кода
(может мешать литералам регулярных выражений, например: /* /.*/ */)
< евангелизация > И, конечно, не используя Prototype;) </евангелизация >
Самым большим для меня не понимание самого языка программирования JavaScript.
Затем есть те, которые используются во время выполнения браузера:
И, наконец, проблемы с развертыванием.
<plug> У меня есть советы по оптимизации на стороне клиента , которые охватывают некоторые из вещей, упомянутых выше, и многое другое, в моем блоге. </plug>
Несколько вещей прямо на моей голове. Я отредактирую этот список, когда думаю больше.
см. также Crockford "Javascript: The Good Parts" для других вещей, которых следует избегать. (edit:), он немного строг в некоторых своих предложениях, таких как использование "===" над "==", поэтому возьмите их с чем-то вроде соли соли для вас)
любое использование 'with'
с (document.forms [ "mainForm" ]. элементы) { input1.value = "junk";
input2.value = "junk"; }
любая ссылка на
document.all
в вашем коде, если только он не находится в специальном коде, только для IE, чтобы преодолеть ошибку IE. (cough document.getElementById() кашель)
Не использовать среду на основе сообщества для выполнения повторяющихся задач, таких как манипуляция DOM, обработка событий и т.д.
Вы всегда должны положить скобу после инструкции из-за автоматической вставки точки с запятой.
Например:
function()
{
return
{
price: 10
}
}
сильно отличается от этого:
function(){
return{
price: 10
}
}
Becuase в первом примере, javascript вставляет точку с запятой для вас, фактически оставляя вас с этим:
function()
{
return; // oh dear!
{
price: 10
}
}
Вы должны использовать setTimeout вместо setInterval для случаев, когда вам нужно что-то делать повторно.
Если вы используете setInterval, но функция, которая выполняется в таймере, не завершена к моменту окончания следующего таймера, это плохо. Вместо этого используйте следующий шаблон, используя setTimeout
function doThisManyTimes(){
alert("It happening again!");
}
(function repeat(){
doThisManyTimes();
setTimeout(repeat, 500);
})();
Это очень хорошо объясняется Paul Irish на его 10 вещей, которые я узнал из источника jQuery видео
Эффективное кэширование редко выполняется: