Kirill Kostin #11
Kirill Kostin #11
Conversation
| } | ||
| console.log(summCal); | ||
| } | ||
| Order.prototype.payForOrder = function(){ |
There was a problem hiding this comment.
Привет! Молодец, попробовал и на прототипах и на классах реализовать, есть некоторые замечания, например, хотелось бы видеть обработку ошибки при попытке добавить позицию в уже оплаченный заказ
| this.drink = []; | ||
| } | ||
|
|
||
| Order.prototype.addHamburger = function(size, toping){ |
There was a problem hiding this comment.
Тут бы порекомендовала написать универсальные методы добавления позиции (гамбургера, салата или напитка) в заказ и ее удаления
| return this.salad.calories*this.weight/100; | ||
| } | ||
| getPrice(){ | ||
| return this.salad.price*this.weight/100; |
There was a problem hiding this comment.
100 в данном случае магическое число, его лучше вынести в константу
| price: 15, | ||
| calories: 10, | ||
| } | ||
| if (stuffing.toLowerCase()=='cheese') |
There was a problem hiding this comment.
Для читабельности можно попробовать переписать if-else на switch-case
| return 0; | ||
| } | ||
| } | ||
| getCalories(){ |
There was a problem hiding this comment.
Функции с названием getPrice, getCalories просто повторяются в классах различных типов еды. По итогу нет общего предка (например, Item или Product), от которого наследовались бы гамбургер, салат и напиток.
Предлагаю создать общего предка и отнаследоваться
No description provided.