dimakrugly / onix

1 stars 0 forks source link

Чисті функції #48

Closed DmitriiMelnikOnix closed 1 year ago

DmitriiMelnikOnix commented 1 year ago
[...prev.cartData]
  .sort((el, item) => el.productData.price - item.productData.price)
  .reduce((acc, item) => {
    item = { ...item, order: i += 1 };
    acc = [...acc, item];
    return acc;
  }, []),

Краще не змінювати параметри що призодять в функцію (це стосується reduce). Це вважається поганим тоном

А також reduce можна записати простіше

.reduce((acc, item, index) => ([...acc, {...item, order: index + 1 }]), []),

А ще краще з map

.map((item, index) => ({...item, order: index + 1 })),