Repository navigation
HW - #66
HW#66yaAppolitta wants to merge 12 commits into
Conversation
Финал. Попытка 1
Финал. Попытка 1
Финал. Попытка 1
Финал. Попытка 1
Финал. Попытка 1
Финал. Попытка 1
Финал. Попытка 2
| visitor = scanner.nextInt(); | ||
| if (visitor <=1) { | ||
| System.out.println("Некорректное кол-во гостей"); | ||
| System.out.println("Введите колличество гостей"); |
There was a problem hiding this comment.
Можно перенести эту строчку наверх, сразу после while, тогда отсюда и перед while её можно удалить
|
|
||
| Product list = new Product(); | ||
| Calculate menu_list = new Calculate(); | ||
| while (enough) { |
There was a problem hiding this comment.
Можно просто прописать while(true)
| boolean enough = true; | ||
|
|
||
| Product list = new Product(); | ||
| Calculate menu_list = new Calculate(); |
There was a problem hiding this comment.
Для именования переменных лучше использовать верблюжий регистр, то есть menuList
| menu_list.getListPrice(list.price); | ||
| System.out.println("Хотите ввести ещё товар?"); | ||
| System.out.println("Eсли вы хотите ввести ещё товар, введите ДА"); | ||
| System.out.println("Eсли вы хотите ввести ещё товар, введите Завершить "); |
There was a problem hiding this comment.
Наверное, тут должно быть "Если вы не хотите больше вводить товары, введите Завершить"
| } | ||
| } | ||
|
|
||
| if (!enough){ |
There was a problem hiding this comment.
Этот if можно убрать, и переменную enough тоже. Из цикла while может быть выход, только если пользователь введет Завершить, и тогда должен будет выполниться код здесь
|
|
||
| if ((rub == 0 ) || (rub == 5) || (rub == 6) || (rub == 7 ) || (rub == 8 ) || (rub == 9)){ | ||
| rubString = "рублей"; | ||
| } else if ((rub == 2) || (rub == 4) || (rub == 3) ){ |
There was a problem hiding this comment.
Можно упростить - rub >= 2 && rub <= 4
| String receipt = ""; | ||
| double pay = 0; | ||
|
|
||
| public void getAddList (String name, double price) { |
There was a problem hiding this comment.
Этот и getListPrice методы называются "get", то есть "получить", но ничего не возвращают. Лучше назвать "addProduct" и "addPrice". Ещё можно передавать в параметре этого метода сразу объект класса Product, и из него брать имя и цену
| } | ||
| } | ||
|
|
||
| public static class Product { |
There was a problem hiding this comment.
Классы лучше в отдельный файл располагать, и тогда не надо будет делать их статическими
| String rubString = ""; | ||
| int rub = (int) (pay % 10); | ||
| int rubTeen = (int) (pay % 100); | ||
| if ((rub == 0 ) || (rub == 5) || (rub == 6) || (rub == 7 ) || (rub == 8 ) || (rub == 9 ) || ((rubTeen >= 11) && (rubTeen <= 14))){ |
There was a problem hiding this comment.
Можно упростить условие - rub == 0 || (rub >= 5 && rub <= 9) || (rubTeen >= 11 && rubTeen <= 14)
2 participants
Спринт 2. Проектная работа 1.
попытка 2