Привет. Как это можно упростить? function isOptionDisabled(option) { if

Question

Привет. Как это можно упростить? function isOptionDisabled(option) { if

(option?.label) return false;
if (option?.disabled) return true;
if (option?.status !== 'confirmed' || option?.status === 'pending' || option?.status === 'pending_verification') return true;
return false;
}

#javascript #programming #russian

0

30.07.2021

5 ответов

19 просмотров

Ruslan Ilmerainen

у тебя проблема если option имеет свойство disabled, то эта функция не имеет смысла если disabled вычисляется исходя из каких-то других свойств, то свойство disabled лишнее, так как приводит к неконсистентности данных. в таком случае нужно disabled сделать геттером и запретить прямую запись этого свойства

0

30.07.2021

Даниил Зеликов

свитч кейсами. Читабельнее хоть будет. И что за экономия на фигурных скобках?)

0

30.07.2021

El Anónimo Автор вопроса

Ruslan Ilmerainen
у тебя проблема если option имеет свойство disable...

disabled это проперть на option, который в массиве options с другими подобными.

0

30.07.2021

Ruslan Ilmerainen

Даниил Зеликов
свитч кейсами. Читабельнее хоть будет. И что за эк...

да тут нужно указывать не на форматирование, которое поправить это последнее дело, а на то, что у него проблемы со смыслом функции и c самим объектом option

0

30.07.2021

Vlad Rassokhin · Accepted Answer

Vlad Rassokhin

а зачем все эти ифы если есть флаг disabled ?

0

30.07.2021

169 похожих чатов

Привет. Как это можно упростить? function isOptionDisabled(option) { if

5 ответов

Похожие вопросы