Repository navigation
Conversation
| } | ||
|
|
||
| } | ||
| class Calculator{ |
There was a problem hiding this comment.
🍏 В Java принято каждый класс описывать в отдельном файле
| System.out.println("Введите стоимость"); | ||
| if (scanner.hasNextFloat()){ | ||
| cost = scanner.nextFloat(); | ||
| }else{scanner.next();} |
There was a problem hiding this comment.
🍏 Немного поехало форматирование. Для форматирования кода и удаления неиспользуемых импортов применяй следующие сочетания клавиш:
Windows: Control+Alt+L
MacOS: Command+Option+L
Больше горячих клавиш можно найти здесь: https://developer.android.com/studio/intro/keyboard-shortcuts
| String product = scanner.next(); | ||
| float cost = 0f; | ||
| while (cost <= 0) { | ||
| System.out.println("Введите стоимость"); |
There was a problem hiding this comment.
🍏 Круто было бы подсказать пользователю, в каком формате ему нужно ввести стоимость. Например, [руб,коп]
| Calculator calc = new Calculator(participantsQuantity); | ||
|
|
||
| String userDeсision = ""; | ||
| String breakDeсision = "Завершить"; |
There was a problem hiding this comment.
🍏 Так как эта строка не меняется во время работы программы, можно вынести ее в константы. https://www.baeldung.com/java-constants-good-practices
| } | ||
| calc.addProduct(product, cost); | ||
|
|
||
| System.out.println("Добавить ещё один товар? (Если это всё, введите \"" + breakDeсision + "\")"); |
There was a problem hiding this comment.
👏 Круто, что сообщаешь пользователю, как выйти из сценария
| } | ||
|
|
||
| float result = this.cost / this.participantsQuantity; | ||
| System.out.println("Количество участников: " + this.participantsQuantity + " \nДобавленные товары: \n" + this.products + "\n-> Итого с человека: " + CurrencyFormatter.getCostRepresentation(result)); |
There was a problem hiding this comment.
🍏 При печати в памяти будут создаваться промежуточные строки из-за использования оператора "+". Можно использовать StringBuilder, который сформирует только результирующую строку. https://habr.com/ru/articles/260767/
|
|
||
| float result = this.cost / this.participantsQuantity; | ||
| System.out.println( | ||
| //Поправил, сделал правда через формат. |
2 participants
Несколько опоздал со сдачей, прошу прощенье)