Conversation
| @@ -0,0 +1,29 @@ | |||
| import java.util.ArrayList; | |||
|
|
|||
| public class Formatter { | |||
There was a problem hiding this comment.
Здорово, что форматтер вынесен в отдельный класс!
src/main/java/Main.java
Outdated
| private final static String GREET = "На сколько человек требуется разделить счёт?"; | ||
| private final static String ERRORINCORRECTNUMBER = "Введено некорректное число. \nВведите корректное число."; | ||
| private final static String ADDNEWGOOD = "Добавление товара. Ведите наименование товара или 'Завершить' для перехода к расчёту чека"; |
There was a problem hiding this comment.
Ты вынес текст в константы -- это хорошо. Но кодстайл говорит, что названия констант должны быть написаны в UPPER_SNAKE_CASE
|
|
||
|
|
||
| System.out.println(ADDNEWGOOD); | ||
| String name = scanner.nextLine(); |
There was a problem hiding this comment.
Здесь есть небольшое повторение. Мы выводим строку перед циклом и в конце цикла. Можно использовать цикл do-while, чтобы этого избежать(Но это совсем не обязательно!).
| return true; | ||
| } | ||
|
|
||
| static int inputNumber(Scanner scanner) { |
There was a problem hiding this comment.
Мне нравится, что логика вынесена в отдельные функции, но кажется, что можно было сделать проще используюя стандартные методы сканера.
| if (!Character.isDigit(string.charAt(i))) { | ||
| if (!Character.isDigit(string.charAt(i))) { |
There was a problem hiding this comment.
Зачем тут два раза одинаковая проверка?
| System.out.println(GREET); | ||
| int number = 1; | ||
| Scanner scanner = new Scanner(System.in); | ||
| //int number = inputNumber(scanner); |
There was a problem hiding this comment.
Не очень хорошо коммитить закоментированный код
No description provided.