Repository navigation
Conversation
|
|
||
| String theLastDigit = strIntegerPartOfNumber.substring(strIntegerPartOfNumber.length() - 1); | ||
|
|
||
| if (theLastDigit.equals("1")){ |
There was a problem hiding this comment.
⏫ Тут делается не правильное склонение для числе 11-14. Это специфичный кейс и его стоит обработать отдельно.
|
|
||
| } | ||
|
|
||
| public static String GetCurrency(double costPerPerson){ |
There was a problem hiding this comment.
⏫ Имена методов принято начинать с маленькой буквы
|
|
||
| System.out.println("На скольких человек разделить счет?"); | ||
|
|
||
| while (true) { |
There was a problem hiding this comment.
⏫ Лучше не писать код одним полотном, а разделять на смысловые кусочки, которые можно вынести в функцию. Например, этот цикл можно вынести в функцию и назвать в стиле readNumberOfPeople. Так код будет выглядить аккуратнее, его будет легче читать и другим разработчиком будет легче понимать общую структуру.
| System.out.println("Товар добавлен - " + productName); | ||
| } | ||
|
|
||
| void sum(double sum) { |
There was a problem hiding this comment.
⏫ Можно сделать приватным, так как используется только в рамках данного класса.
| this.totalAmount = this.totalAmount + sum; | ||
| } | ||
|
|
||
| void FillInTheListOfProducts(String productName) { |
There was a problem hiding this comment.
⏫ Тоже можно сделать приватным, а также лучше переименовать метод, чтобы начиналось с маленькой буквы
| double totalAmount = 0; | ||
| String listOfProducts = ""; | ||
|
|
||
| void AddProductToCalculator(String productName, double costOfGood){ |
|
|
||
|
|
||
|
|
||
| if (costOfGoods > 0) { |
There was a problem hiding this comment.
⏫ Так как мы даем пользователю возможность вводить дробное число, возникает ситуация, когда над приложением захотят поиздеваться и ввести какую-нибудь нереальную сумму (например, 0,00000000009). В такой ситуации при округление получится, что все должны заплатить по 0 рублей и кто-то потеряет в деньгих. Можно было бы попробовать добавить валидацию таких ситуаций или сразу ограничить ввод 2 знаками после запятой и уведомить об этом пользователя. А может быть ограничить минимальную цену, которую можно указать.
3 participants
No description provided.