не
не кажется что лучше сделать так - obj = {} ?
⩔wein
ixplo
так оно и так будет undefined
или в цикле если ты прогоняешь объект, то его не будет изначально
ixplo
а тут будет со значением undefined
⩔wein
смысл честно говоря ускользает
Nikolay
нормальный человек напишет obj.param = undefined/null
ixplo
блин, чот вы тупите
Nikolay
:D
ixplo
а если там есть значение
ixplo
его нужно оставить
ixplo
если нет, создать
ixplo
вариант в продакшене в итоге был такой: if (!(param in obj)) obj.param = undefined;
⩔wein
а если там есть значение
сделай проверку.
obj.param = (typeof obj.param === 'undefined' ? undefined : obj.param);
ixplo
ixplo
безх лишнего присваивания самого себя
ixplo
при наличии
⩔wein
разницы на практике никакой, ты ж понимаешь что компилятор это сьест все
Smooth Operator
ixplo
разницы на практике никакой, ты ж понимаешь что компилятор это сьест все
ну поэтому и начальный варик obj.param = obj.param; ок
Mikhail
я бы такое не задавал на собеседовании
ixplo
только к нему нужен камент D:
⩔wein
каменты зло
ixplo
каменты - добрейшее добро
ixplo
спасибо тем, кто их оставляет )
⩔wein
да нифига. комментить функцию с кратким описанием чо она делает - это замечательно и правильно
ixplo
угу
ixplo
и сложные строки тоже
⩔wein
комментить строку кода потому что там какая то магия - зло. перепиши магию
Oleksandr
иногда некогда
ixplo
ну тут где грань?
Oleksandr
переписывать
Oleksandr
в страшном интерпрайзе
Mikhail
угу, если приходится комментировать код, чтобы объяснить что он делает, лучше переписать этот кусок
Oleksandr
так тем более
Oleksandr
все такие умные ))))
ixplo
они просто не умеют в js
⩔wein
ну тут где грань?
ну я для себя провожу грань так, что комментировать можно только функции.
если какой то блок кода непонятен - надо его или переписать или вынести отдельно.
если какая то строка не понятна - надо ее переписать
ixplo
"понятен" - вот вопрос
Oleksandr
то есть можно вынести не понятный блок кода в функию и комментировать уже его ?
Oleksandr
с твоей точки зрения - да
Mikhail
это все на ревью решается
⩔wein
они просто не умеют в js
js довольно хреновый язык, когда начинаешь уходить в дебри с его приведением типов.
в идеале все эти моменты лучше просто не использовать, тем более что это очень просто.
ixplo
я легко читаю такое: f = g && h; f();
Smooth Operator
obj.param = obj.param; // энтерпрайз, не трогать
Mikhail
если у вас вызывать сетер через obj.param = obj.param принято (и вобще пользоваться сетерами с сайдэффектам) — все ок
ixplo
иногда это важно
Nikolay
производительно иметь две конструкции на одной строчке?
ixplo
раздели их на строки )
Nikolay
важно чтобы ты через полгода смог понять что ты там написал )
ixplo
просто многие подумают, что я хочу потом true() сделать
ixplo
хотя я могу там undefined() вызвать случайно :D
ixplo
😅
⩔wein
то есть можно вынести не понятный блок кода в функию и комментировать уже его ?
Зависит. Если непонятный потому что большой и все в комплексе смешивается - да, вынести и комментировать.
Если непонятный потому что там магия - переписать.
ixplo
хотя я могу там undefined() вызвать случайно :D
лучше так, наверно:
let x = () => {};
let f = g && f || x;
f();
⩔wein
просто многие подумают, что я хочу потом true() сделать
разделить на строки и переменные назвать нормально и вполне адекватная констуркция
⩔wein
единственное что не оч понятно как ты f как функцию вызываешь, когда это булеан.
ixplo
лол
ixplo
ну вот и ты не знаешь js
Mikhail
это не булеан :)
ixplo
бгг
⩔wein
ну да. намбер.
Oleksandr
???
ixplo
не
Oleksandr
последнее трушное
Oleksandr
значение
Mikhail
ну нет же
Oleksandr
трушное , не налл, не андефайнд
Oleksandr
и тд
ixplo
расскажу на словах:
Oleksandr
там может быть функция
Nikolay
лучше так, наверно:
let x = () => {};
let f = g && f || x;
f();
лучше
if (g && f) {
f();
} else {
x();
}
Mikhail
да, явное лучше неявного
Nikolay
логика запутывания?