Требуется помощь по кассовому аппарату Free Code Camp

Question

Мне нужна помощь с моим кодом для участия в Cash Register Challenge на FreeCodeCamp.Я прохожу другие проверки, кроме тех, которые для "ОТКРЫТЫ"

Для второй проверки я получаю правильную разбивку ["КВАРТАЛ": 0.25], ["КВАРТАЛ": 0.25] т.е. два квартала, но я нене знаю, как сложить это вместе и вернуть как ["QUARTER": 0.5].

В третьих, разбивка почти налицо, но она не отнимает последнюю копейку, поэтому мне не хватает одной ["PENNY«: 0.01] от окончательной разбивки.Когда я проверяю оставшиеся изменения, это одна копейка.

Так что мне действительно нужна помощь, чтобы вернуть изменение как объединенное значение для каждого типа единиц и посмотреть, почему оно не возвращает общую сумму в третьем случае.

function checkCashRegister(price, cash, cid) {
  var cashAvailable = cid;
  var units = [
    ["PENNY", 0.01],
    ["NICKEL", 0.05],
    ["DIME", 0.1],
    ["QUARTER", 0.25],
    ["ONE", 1],
    ["FIVE", 5],
    ["TEN", 10],
    ["TWENTY", 20],
    ["ONE HUNDRED", 100]
  ].reverse()
  var cashAvailable = cid.slice().reverse()
  var stat = {
    status: '',
    change: []
  };
  var changeRequired = cash - price;
  var totalCash = cashAvailable.flat().filter(x => {
    return isNaN(x) == false
  }).reduce((a, b) => {
    return a + b
  }).toFixed(2)
  var unitsNeeded = []
  if (totalCash == changeRequired) {
    stat.status = 'CLOSED'
    stat.change = cid
  } else if (totalCash < changeRequired) {
    stat.status = 'INSUFFICIENT_FUNDS'
    stat.change = []
  } else if (totalCash > changeRequired) {
    for (var i = 0; i < units.length; i++) {
      while (changeRequired >= units[i][1] && cashAvailable[i][1] > 0) {
        unitsNeeded.push(units[i])
        cashAvailable[i][1] -= units[i][1]
        console.log((changeRequired -= units[i][1]).toFixed(2))
      }
      if (changeRequired > units[8][1]) {
        stat.status = 'INSUFFICIENT_FUNDS'
        stat.change = []
      } else {
        stat.status = 'OPEN';
        stat.change = unitsNeeded
      }
    }
  }

  return stat
}

checkCashRegister(3.26, 100, [
  ["PENNY", 1.01],
  ["NICKEL", 2.05],
  ["DIME", 3.1],
  ["QUARTER", 4.25],
  ["ONE", 90],
  ["FIVE", 55],
  ["TEN", 20],
  ["TWENTY", 60],
  ["ONE HUNDRED", 100]
])

checkCashRegister(19.5, 20, [
  ["PENNY", 1.01],
  ["NICKEL", 2.05],
  ["DIME", 3.1],
  ["QUARTER", 4.25],
  ["ONE", 90],
  ["FIVE", 55],
  ["TEN", 20],
  ["TWENTY", 60],
  ["ONE HUNDRED", 100]
])

это две проверки.

Answer 1

В следующей демонстрации используется класс и карта ES6. Подробности прокомментированы в демоверсии.

/* This Map represents what the till has initially. Each value is 
the count of each type -- not the monetary value.*/
let funds = new Map([
  ['One Hundred', 2],
  ['Twenty', 10],
  ['Ten', 20],
  ['Five', 20],
  ['One', 50],
  ['Quarter', 20],
  ['Dime', 25],
  ['Nickel', 30],
  ['Penny', 100]
]);

/** Register(funds)
@Params: funds[Map]: key = name / value = count not monetary
A class that has the following methods:
  - set/get change 
      Allows direct access to this._change constructor property.
  - transact(cost, payment)
      Simply subtracts cost from payment to determine amount of
      change.
  - distribute(title = 'Change') 
      Takes a given amount and distributes it as currency in the
      smallest amount of denominations. If a title isn't passed, 
      the default is used.
  - remaining(map, title = 'Remaining in Till')
      Pass the Map of change and return a Map of remaining money
      from the fund Map. If a title isn't passed, the default is
      used.
  - displayMap(map)
      A utility method to aid in displaying Maps.
*/
class Register {
  constructor(funds) {
    this.denomination = new Map([
      ["One Hundred", 100],
      ["Twenty", 20],
      ["Ten", 10],
      ["Five", 5],
      ["One", 1],
      ["Quarter", 0.25],
      ["Dime", 0.1],
      ["Nickel", 0.05],
      ["Penny", 0.01]
    ]);
    this.distribution = funds;
    this._change = 0.00;
  }

  set change(money) {
    this._change = money;
  }

  get change() {
    return this._change;
  }

  transact(cost, payment) {
    this.change = payment - cost;
    return this.change;
  }

  distribute(title = 'Change') {
    let dist = new Map();
    dist.set('||', `~~~~~~~~~~[${title}]~~~~~~~~~~`);
    let total = Number(this.change);
    let denom = Array.from(this.denomination.entries());

    for (let [key, value] of denom) {
      if (total >= value) {
        let money = Math.floor(total / value);
        dist.set(key, money);
        total = total % value;
      }
    }
    return dist;
  }

  remaining(map, title = 'Remaining in Till') {
    for (let [key, value] of map.entries()) {
      if (this.distribution.has(key)) {
        let fund = this.distribution.get(key);
        this.distribution.set(key, (fund - value));
      }
    }
    map.set('|', `~~~~~~~~~~[${title}]~~~~~~~~~~`);
    return this.distribution;
  }

  displayMap(map) {
    for (let [key, value] of map.entries()) {
      console.log(key + ': ' + value);
    }
  }
}

// Instantiate class instance as `r` pass the funds Map
const r = new Register(funds);

// Arbitrary values
let cost = 252.63;
let payment = 300.00;

// set/get this.change from constructor
r.transact(cost, payment);

// Returns the change in the least amount of denominations
const dist = r.distribute();

// Returns the remaining count of each denomination after change
const till = r.remaining(dist);

// Display Maps
r.displayMap(dist);
r.displayMap(till);

Answer 2

Не решение, но подсказки, которые могут помочь найти решение:

Здесь вы присваиваете текст (а не число) для var totalCash.toFixed() - это функция форматирования чисел, возвращающая строку по уважительным причинам: числа типа 0.01 не существуют из-за двоичной реализации чисел с плавающей запятой.Не используйте его для округления чисел, это не сработает!

  var totalCash = cashAvailable.flat().filter(x => {
    return isNaN(x) == false
  }).reduce((a, b) => {
    return a + b
  }).toFixed(2)

А затем вы проведете численное сравнение с текстом:

  if (totalCash == changeRequired) {
      ...
  } else if (totalCash < changeRequired) {
      ...
  } else if (totalCash > changeRequired) {
      ...
  }

Решение: Удалите toFixed(2) сверху и обычно используйте toFixed() только для визуализации!

Еще один совет: Здесь вы изменяете значение changeRequired как часть оператора console.log().Если впоследствии вы удалите журнал, семантика кода изменится:

console.log((changeRequired -= units[i][1]).toFixed(2))

Лучше:

changeRequired -= units[i][1];
console.log(changeRequired.toFixed(2));