1) брэйсы содержащие только один стейтмент, отмазка в виде зато код расширяемый не катит, задача изначально такая что функция может вернуть либо да либо нет и ни в какой параллельной вселеной она не может возвращать что-то еще
2) проверка и сравнивание значения булевой переменной (=== true опустить надо)
3) неиспользование тернарника в наиочевиднейшем случае
Нажмите, чтобы раскрыть...
Я хоть не js макака, но могу сказать, что:
1) Докапываться до брейсов... Вот это точно "студент 3 курса".
2) === правильно стоит, потому что проверка не только на true/false, но и на тип аргумента. Нет строгой типизации и потому boolToWord переводит ИМЕННО И ТОЛЬКО ТИП boolean в текст "Yes" или "No".
Для пигпол-адепта специально:
3) Да, более читаемо куча
:
value === true ? "Yes" : "False"
, чем:
boolToWord(value)
ок не прав
но в этом случае просто прикрути проверку на бул
если да то да
если нет то нет
если не бул то "не бул"
Нажмите, чтобы раскрыть...
Ты чуть выше тут до отсутствия тернарного оператора докопался и брейсов, типа много строк, а теперь предлагаешь делать отдельно сначала проверку на тип и присыпать проверкой на значение самой переменной? Предлагаю ещё генериками обкрутить 