ru_vds Feb 6 2018 at 12:52

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

11 min

30K

RUVDS.com corporate blogWebsite development*JavaScript*Debugging*

Translation

+15

Comments 17

welcomerooot Feb 6 2018 at 13:03

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

mayorovp Feb 6 2018 at 13:33

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

Zoberg Feb 6 2018 at 17:52

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

<script crossorigin='anonymous'

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

justboris Feb 6 2018 at 23:21

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

vintage Feb 7 2018 at 11:50

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

justboris Feb 7 2018 at 13:03

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

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

animhotep Feb 6 2018 at 13:54

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

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

KodyWiremane Feb 6 2018 at 14:29

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

animhotep Feb 6 2018 at 16:08

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

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

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

-2

KodyWiremane Feb 6 2018 at 20:45

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

serf Feb 6 2018 at 14:03

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

Juma Feb 6 2018 at 16:53

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

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

или просто

if (!value) return;

-2

animhotep Feb 6 2018 at 17:11

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

-2

KodyWiremane Feb 6 2018 at 21:07

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

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

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

-1

ThisMan Feb 7 2018 at 08:21

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

Cannot read property 'items' of null

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

extempl Feb 7 2018 at 09:28

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

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

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

fidgethard Feb 7 2018 at 11:38

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'");
}

-1