KA
Size: a a a
2020 April 28
DT
я не про это.
KA
а про что?
DT
чувствуешь мою точку в конце?
DT
это не задача мап делать еще псевдо if
М
чувствуешь мою точку в конце?
Выглядит угрожающе.
z
а про что?
Много лишнего кода который не нужен
KA
Много лишнего кода который не нужен
конкретнее плез
KA
это не задача мап делать еще псевдо if
вынести типо фильтрацию ( ифки ) в filter и красоту объекта делать в map?
М
конкретнее плез
Проверки проводи в одном месте, построение визуала и вывод - в другом
DT
мап фильтр или редюс, либо уж в форе все творить
z
Начали с первой строки что имя items слишком COMMON для описания коллекции
Строчные функции можно писать короче
map reduce filter можно применить оптимальнее или как коллега сказал forEach
Не надо менять типы коллекции
Если копируешь объект это можно сделать деструктуризацией
Строчные функции можно писать короче
map reduce filter можно применить оптимальнее или как коллега сказал forEach
Не надо менять типы коллекции
Если копируешь объект это можно сделать деструктуризацией
z
Плюс у тебя есть на первой строке Items и ещё внутри el.items что вводит в заблуждение надо переименовать
z
item Тоже переименовать во что то осмвсленное
z
Flat вообще зачем там ? У тебя подмассивы есть ?
KA
Flat вообще зачем там ? У тебя подмассивы есть ?
да
z
Почему они есть ?
М
Предлагаю начать сначала и узнать бизнес-задачу, которую решает данный код 🙂
М
А потом уже о реализации говорить