Замечания по lw6

[ivklgn]

1. Нет смысла передавать в конструктор аргументы
2. В методе setCash - в условии рекомендуется использовать строгое сравнение
3. Т.к мы изучили es6 - поменять работу со строками в формате es6:

   console.log('Ваш баланс: ' + this.moneyCash + ' руб.')

4. Метод checkNumber, chooseCoffee - не хватает скобки.
5. В методе checkNumber - сделать проверку номера напитка не зависимости от id и количества напитков в меню

6. Метод getRemainCash - объявить переменную перед использованием.

   рекомендация для lw7 - подумать о возвращаемых данных от методов

kQ=0,5 kT=0,8

[ivklgn]

5 замечание: сделать проверку номера выбранного напитка без зависимости от длинны массива:

  checkNumber(num) {
    if ((num < 1) && (num >= this.coffeeMenu.length))
      return false;
    else
      return true;
  }

6 замечание: в этом коде есть что то лишнее:

  getRemainCash(remain) {
    return remain = this.moneyCash - this.coffeeMenu[this.currentCoffeeNumber - 1].price;
  }

kT=0.8 kQ=0,8

[ivklgn]

метод checkNumber(num)

 checkNumber(num) {
    if ((num < 1) && (num >= this.coffeeMenu.id))
      return false;
    else
      return true;
  }

в условии (num >= this.coffeeMenu.id) - будет всегда false, условие сделано неверно. имеет смысл взять id из меню и положить в обычный массив и проверить есть ли номер напитка в этом массиве