Conversation
src/main/java/Calculator.java
Outdated
| System.out.println("Давайте добавим ваши товары."); | ||
| while(true){ | ||
| System.out.println("Введите название товара или 'Завершить' если вы закончили."); | ||
| String name = scanner.next(); |
There was a problem hiding this comment.
В текущей реализации, при вводе названия товара, программа использует scanner.next(), что ограничивает ввод до первого пробела. Если название товара состоит из нескольких слов, только первое слово будет использовано.
src/main/java/Calculator.java
Outdated
| } | ||
|
|
||
| totalSum += value; | ||
| allPurchasees = allPurchasees.concat(String.format("\n%s %.2f", name, value)); |
There was a problem hiding this comment.
При использовании concat() в цикле добавления товаров, каждый раз создается новая строка, что может привести к неэффективному использованию памяти. Лучше использовать StringBuilder
src/main/java/Calculator.java
Outdated
| double value; | ||
| while (true){ | ||
| System.out.println("Введите стоимость " + name + ":"); | ||
| value = scanner.nextDouble(); |
There was a problem hiding this comment.
После использования сканнера его следует закрыть с помощью scanner.close().
src/main/java/Calculator.java
Outdated
| System.out.println(String.format("Каждый гость должен заплатить %.2f %s", pricePerPerson, getPadezh(pricePerPerson)) ); | ||
| } | ||
|
|
||
| String getPadezh(double d){ |
There was a problem hiding this comment.
лучше всё же не использовать русские слова написанные английскими буквами, это может в превратиться в poluchitPadezh() :D
|
Имя товара теперь может состоять из нескольких слов. Правильный падеж для рублей 11-19.
|
Поправил замечания |
Есть пара вопросов, примечаний: