Создание компонента «Конструктор пиццы» #1
Conversation
keksobot
changed the title
There was a problem hiding this comment.
Большой путь начинается с первого шага, а большой проект начинается с первого пул реквеста))).
Ты хорошо справилась с заданием. Есть только одно замечание. В блоке "Выберите размер" подписи к размерам у всех элементов одинаковые (23 см). Перепроверь цикл v-for. Эта одна очень маленькая ошибочка которую ты быстро исправишь.
Остальные комментарии по коду это просто рекомендации на которые стоит обратить внимание.
Внимательно перепроверяй проект на соответствие техническому заданию перед сдачей. И помни, что если возникают вопросы, то их всегда можно задавать в чате. Жду PR с исправлением.
| return textSize; | ||
| }, | ||
|
|
||
| getSouce(name) { |
There was a problem hiding this comment.
А названии метода закралась опечатка)) В метод лучше передать id соуса. Так может получится, что названия названия могут совпадать или быть написаны с опечаткой. ID подразумевает под собой уникальный идентификатор, который гарантирует, что у каждого элемента списка он свой.
| import misc from "@/static/misc.json"; | ||
| import pizza from "@/static/pizza.json"; | ||
| import user from "@/static/user.json"; | ||
| export default { |
There was a problem hiding this comment.
Секцию импортов лучше отделять пустой строкой. Так легче читать код.
| pizzaSize(size) { | ||
| let textSize = ""; | ||
| switch (Number(size.replace(` см`, ""))) { | ||
| case 23: | ||
| textSize = `small`; | ||
| break; | ||
| case 32: | ||
| textSize = `normal`; | ||
| break; | ||
| case 45: | ||
| textSize = `big`; | ||
| break; | ||
| } |
There was a problem hiding this comment.
Если в метод передать size.id, то можно не делать лишнюю операцию Number(size.replace( см, "")) и в case использовать значения id.
Для значений 'small', 'normal', 'big' желательно завести словарь, ключами которго будут size.id. Тогда можно при помощи него заменить конструцию switch. Подумай как это можно сделать.
| pizzaSizeValue(size) { | ||
| let textSize = ""; | ||
| switch (Number(size.replace(` см`, ""))) { | ||
| case 23: | ||
| textSize = `small`; | ||
| break; | ||
| case 32: | ||
| textSize = `normal`; | ||
| break; | ||
| case 45: | ||
| textSize = `big`; | ||
| break; | ||
| } | ||
|
|
||
| return textSize; | ||
| }, |
There was a problem hiding this comment.
Логика этого метода дублируется в методе pizzaSize. В методе pizzaSize можно было получить значение textSize одной строчкой const textSize = this.pizzaSizeValue(size);.
🎓 Создание компонента «Конструктор пиццы»