MV
const addShape = (attr, type) => attr || Yup[type]();
Size: a a a
2020 June 14
MV
Тебе там просто тернарный не нужен судя по твоей функции
MV
Достаточно || или ??, если нужно только на нуллейбл отпнуть
DB
Достаточно || или ??, если нужно только на нуллейбл отпнуть
да, тоже верно
DB
палок хватает
i
зачем вообще функция на одну строчку
MV
зачем вообще функция на одну строчку
Может много где дёргает, чтобы не засорять код
DB
Может много где дёргает, чтобы не засорять код
угу, хватает
i
Может много где дёргает, чтобы не засорять код
он будет засорять через addShape
MV
он будет засорять через addShape
Ну так хотя бы будет семантически понятно что это за вызов
MV
Ты же код пишешь не только ради кода, но и чтобы кто-то потом это поддерживал
MV
Мне было удобнее читать вызов addShape, чем эльфийский
DB
он будет засорять через addShape
умесно ли 10 раз писать эти условия?
MV
умесно ли 10 раз писать эти условия?
Нет)
MV
Лучше не надо)
MV
Но возможно ты саму логику пишешь не совсем верно
MV
Это нужно понимать контекст задачи и текущую реализацию
i
умесно ли 10 раз писать эти условия?
тебе это нужно и всё attr || Yup[type]();
i
что за Yup тут не понятно
MV
Ну можно как раз это пояснить в комментарии к функции addShape