Топ-10 ошибок из 1000+ JavaScript-проектов и рекомендации по их устранению

[welcomerooot]

За Access-Control-Allow-Origin * надо руки отрывать.

[mayorovp]

За рекомендацию всегда ставить там * — да, надо. А за наличие публичного или защищенного другими механизмами API руки отрывать не обязательно…

[Zoberg]

Дополню, что можно ставить

<script crossorigin='anonymous'

чтобы видеть тексты ошибок.

[justboris]

Разработчикам Github, Яндекс карт и многим другим тоже отрывать будете?

[vintage]

Нужно возвращать содержимое Origin, а не *, чтобы авторизация работала.

[justboris]

Вот это уже конструктивное предложение, интересно.

А то если всем руки поотрывать, кто работать-то будет?

[animhotep]

Сейчас мы проанализируем каждую из них, расскажем о том, что их вызывает, и о том, как с ними бороться.

и как же бороться с ошибкой 9?

[KodyWiremane]

Вовремя инициализировать свои объекты и проверять перед использованием чужие.

[animhotep]

я в таких случаях делаю что-то типа

var test = undefined;
if (!test) {
    test = {
        value: 0
    }
}
test.value = 0;

думал может есть способы по чище

[KodyWiremane]

Ну тут от контекста зависит: если вместо отсутствующего объекта предполагается использовать некий дефолтный объект, то да; а если жизненно важный объект отсутствует там, где ожидается его наличие — то надо как-то сообщать об ошибке, дебажить.

[serf]

Можно для начала попробовать просто включить strictNullChecks опцию компиляции в TypeScript.

[Juma]

А почему нельзя просто использовать проерку содержимого?

if (value === typeof 'string') { // 'function', 'number', Array.isArray(value)
  // code
}

или просто

if (!value) return;

[animhotep]

вобще не приятно, когда падает на Cannot set property '...' of undefined
круто было бы, еслиб котнструкция типа if (!value) return; была в прототипах объектов

[KodyWiremane]

Если объект такой, что undefined — и хрен с ним, то конечно. А вот собрался человек на свадьбу ехать,

var taxi = getTaxi();
taxi.addPassenger(self);

— а такси не приехало. Undefined. Вот это вот if (!value) return, его же люди не поймут. Прежде всего — [его] невеста.

[ThisMan]

Пример с React и state в первом пункте не совсем правильный. Ошибкой будет

Cannot read property 'items' of null

так как явно не указан state, а по умолчанию он равен null

[extempl]

В более современных браузерах можно использовать метод bind() для передачи необходимой ссылки:

В более современных браузерах — это во всех начиная с 13 года. То есть, традиционный подход как раз с bind/call/apply. А подход с кешированием this от незнания того что bind существует.

сарказм on
Это примерно как говорить, что традиционный способ вёрстки — табличный. А последние браузеры ещё и блочный поддерживают.

[fidgethard]

var test = {};

if (typeof test !== "undefined") {
	test.value = 1;
	console.log(test.value);
} else {
	console.log("wtf is 'test'");
} 


test = undefined;

if (typeof test !== "undefined") {
	test.value = 1;
	console.log(test.value);
} else {
	console.log("wtf is 'test'");
}