Repository navigation
Проектная Работа № 1 - #1
Senjougaharo wants to merge 4 commits into
Conversation
|
|
||
| String cart = "Добавленные товары:"; | ||
| double totalPrice = 0; | ||
| public void calculator() { |
There was a problem hiding this comment.
Методы лучше стараться называть глаголами, а вот классы можно существительными
| public class Products { | ||
|
|
||
| static double Price; | ||
| static double Price() |
There was a problem hiding this comment.
Названия методов и переменных пишутся с маленькой буквы, только классы - с большой. Лучше методы называть глаголами, отвечая на вопрос, что он делает
| System.out.println("Товар добавлен в список!"); | ||
| System.out.println("Если хотите добавить еще товар, введите \"Да\".\nЕсли хотите завершить покупку, введите \"Завершить\"."); | ||
| String add = scanner.next(); | ||
| if (add.equalsIgnoreCase("да")) { } |
There was a problem hiding this comment.
Поскольку тело if пустое, можно этот if вообще убрать
| System.out.println("Добавленные товары:\n" + names); | ||
| break; | ||
| } | ||
| else; |
There was a problem hiding this comment.
В пустом else тоже нет необходимости
There was a problem hiding this comment.
Извините не могу изменить рекомендательные микромоменты потому что я на ужасных стрессах от того факта что появились 2 непонятных для меня класса которых у меня никогда в жизни не было Formatter и Item. А у меня закрыт доступ ко 2рой проектной работе которую надо сделать за 2 дня и сдать
| Products list = new Products(); | ||
| double price = list.Price(); | ||
| Guests Guests = new Guests(); | ||
| int peopleAmount = Guests.peopleAmount(); |
There was a problem hiding this comment.
Поскольку peopleAmount и Price - статические методы, их можно вызывать так - Guests.peopleAmount() и Products.Price(), тогда переменные list и Guests не нужны
| return totalPrice / friendsCount; | ||
| public String endWord(double a) { | ||
| int cur = (int) Math.floor(a); | ||
| if (cur % 100 > 5 && cur <= 20) { |
There was a problem hiding this comment.
Во второй части сравнивать нужно тоже cur % 100
| return "рублей"; | ||
| } else if (cur % 10 == 1) { | ||
| return "рубль"; | ||
| } else if (cur % 10 > 1 && cur % 10 < 5) { |
There was a problem hiding this comment.
Выражение cur % 10 считается несколько раз, можно посчитать его один раз, записать в переменную и использовать ее
| double price; | ||
| while (true) { | ||
| System.out.println("Введите стоимость товара в формате \"рубли,копейки\":"); | ||
| price = scanner.nextDouble(); |
There was a problem hiding this comment.
Пользователь может ввести на вопрос про стоимость товаров вместо числа строку. Для количества человек ты обрабатываешь такую ситуацию через hasNextInt, стоит добавить обработку и для стоимости
2 participants
No description provided.