Conversation
# Conflicts: # src/main/java/AccountDivider.java
Charmandik
left a comment
There was a problem hiding this comment.
Пожалуйста выноси весь новый код в отдельную ветку. В текущем pull request не было всех изменений из класса Calculator, поэтому пришлось писать просто к классу коментарии, а не к конкретному месту
| import java.util.Scanner; | ||
|
|
||
| public class AccountDivider { | ||
| public void start() { |
There was a problem hiding this comment.
В методе start класса AccountDivider необходимо предусмотреть обработку исключений, которые могут возникнуть при вводе пользователя. Например, если пользователь введет не числовое значение при запросе количества гостей или стоимости товара, программа может завершиться с ошибкой InputMismatchException.
|
|
||
|
|
||
|
|
||
| public class Calculator { |
There was a problem hiding this comment.
Разделение счета: В методе splitBill(int kGuests) класса Calculator есть ошибка при определении окончания для слова "рубль". У тебя есть проверка part % 10 == 1 && part % 100 != 11, которая верна только для чисел, заканчивающихся на 1, кроме чисел, оканчивающихся на 11. Однако, это не учитывает случаи чисел, заканчивающихся на 11, 12 и 13, которые также используют слово "рублей", а не "рубль".
| List<Item> getItems() { | ||
| return items; | ||
| } | ||
|
|
There was a problem hiding this comment.
В методе addItem(String name, double price) класса Calculator нет проверки на пустое имя товара или неположительную цену. Можно добавить проверки на такие случаи и предпринять соответствующие действия (например, вывести сообщение об ошибке и запросить ввод данных еще раз).
|
|
||
|
|
||
|
|
||
| public class Calculator { |
There was a problem hiding this comment.
Метод splitBill(int kGuests) класса Calculator имеет некоторые повторяющиеся участки кода, такие как форматирование вывода суммы и окончания. Ты можешь вынести этот код в отдельный метод для повышения читаемости и поддерживаемости кода
сдача готового кода