Может отрефакторить?
Как минимум заюзать тернарные операторы
ну оптимизировать, только если внутрянку каждого метода, чтоб еще на пару наносекунд быстрее было, но а так, по код стайлу, можно на es6 классы переписать
У тебя лишние проверки Добавь else и не проверяй дважды items.isApproved
используй ES10
А может и не стоит. Preconditional return тоже неплохой паттерн, тк из него легко читается какие условия должны выполнится, чтобы return был другой
Согласен, не всмотрелся в код
И смотря на твою компоненту - не понятно какие поля содержаться в items. Или типизируй в пропсах объект items, или используй деструктуризацию. Тогда глядя в самое начало компоненты, я буду понимать какие данные были поданы на вход
Тоже самое касается хука useStyles. Я не знаю какие поля содержаться в переменной classes. Используй деструктуризацию для явной инициализации данных из хука {approvedCard, emptyCard, итп} = useStyles()
есть либа clsx, а лучше использовать data-attr
тебе бы линтером работать в ide) чтоб проверял очепятки) меньше дебажить бы пришлось😂
посмотри https://www.npmjs.com/package/classnames
Обсуждают сегодня