Почему в корзине считается только последний discount?

Question

[Boris]

Помогите пожалуйста найти баг или исправить ошибку, при подсчете скидки из второго массива берет только последний объект. Возможно есть лучший способ для решения такой задачи.

function totalCost(goods, discounts) {
  let count = 0;
  let sum = 0;
  for (let i in discounts) {
    let str = discounts[i].discount.replace("%", "");
    str = str / 100;
    discounts[i].discount = str;
  }
  for (let i in goods) {
    if (discounts.length === 0) {
      count = goods[i].value * goods[i].amount;
    }
    for (let j in discounts) {
      if (goods[i].name === discounts[j].name) {
        count =
          (goods[i].value - goods[i].value * discounts[j].discount) *
          goods[i].amount;
      } else {
        count = goods[i].value * goods[i].amount;
      }
    }
    sum += count;
    console.log(count, goods[i].name);
  }
  console.log("sum = " + sum);
}
totalCost(
  [
    { name: "Milk", value: 10, amount: 2 },
    { name: "Vegetables", value: 10, amount: 1 },
    { name: "Meat", value: 50, amount: 1 },
    { name: "Salt", value: 10, amount: 3 },
  ],
  [
    { name: "Milk", discount: "50%" },
    { name: "Vegetables", discount: "50%" },
    { name: "Salt", discount: "50%" },
  ]
);

Answer 1

[0xD34F]

for (let i in goods) {
  if (discounts.length === 0) {
    count = goods[i].value * goods[i].amount;
  }
  for (let j in discounts) {
    if (goods[i].name === discounts[j].name) {
      count =
        (goods[i].value - goods[i].value * discounts[j].discount) *
        goods[i].amount;
    } else {
      count = goods[i].value * goods[i].amount;
    }
  }
  sum += count;
  console.log(count, goods[i].name);
}

Зачем на каждой итерации цикла, ищущего скидку, пересчитывать цену? Предположим, нашли скидку для текущего товара, что дальше? Дальше, на следующей итерации, цена будет рассчитана заново, но уже без скидки - ведь объект скидки будет другим, будет соответствовать другому товару. Исключение - если скидка найдена на последней итерации.

Правильно будет вычислять до цикла сумму без скидки, искать в цикле скидку, если нашли - пересчитывать со скидкой, прерывать цикл:

for (const n of goods) {
  let s = n.value * n.amount;
  for (const m of discounts) {
    if (n.name === m.name) {
      s *= 1 - m.discount;
      break;
    }
  }
  sum += s;
}

Возможно есть лучший способ для решения такой задачи.

function totalCost(goods, discounts) {
  discounts = Object.fromEntries(discounts.map(n => [ n.name, 1 - parseFloat(n.discount) / 100 ]));
  return goods.reduce((acc, n) => acc + n.value * n.amount * (discounts[n.name] ?? 1), 0);
}

Comments

[Чипекве]

[ n.name, parseFloat(n.discount) / 100 ]

[ n.name, 1 - parseFloat(n.discount) / 100 ]

[0xD34F]

Чипекве, ну да. Есть такой косяк. Поправил.

[Антон Шаманов]

проще ввести полноценные классы

class CartItem
{
       constructor(name, price, qty = 1, discount = 0) {
            this._name = name;
            this._price = parseFloat(price);
            this.qty = qty;
            this.discount = discount;
       }
       get name() {
            return this._name;
       }
       set gty(value) {
            value = parseInt(value);
           if (value < 1) {
                 throw Error('...');
           }
            this._qty = value ;
       }
       get gty() {
            return this._qty;
       }
       set discount(value) {
           value= parseFloat(value);
           if (value< 0) {
                 throw Error('...');
           }
            this._discount = value;
       }
       get discount() {
            return this._discount;
       }
       get price() {
            return this._price;
       }
       get totalPrice() {
            let total = this.qty * this.price;
            return total - (total / 100) * this.discount;
       }
}