Repository navigation
Conversation
|
|
||
|
|
||
| public class Calculator { | ||
| public static final Scanner scanner = new Scanner(System.in); |
There was a problem hiding this comment.
🍏 scanner не используется за пределами класса, значит можно убрать static и поменять модификатор на private
| } | ||
|
|
||
| //Проверка ввода корректной цены | ||
| public static double checkInputPrice(){ |
There was a problem hiding this comment.
🍏static лучше убрать, советую почитать про static дополнительно
|
|
||
|
|
There was a problem hiding this comment.
🍏 Внутри методов лучше не оставлять пустые строки
|
|
||
|
|
||
| System.out.println("\nТовар успешно добавлен! \n"); | ||
| Product prod = new Product(nameOfProduct, priceOfProduct); |
There was a problem hiding this comment.
🍏Это поле, как и класс Product больше нигде не используется, значит они не нужны в коде
| if (isDouble(userInput)) { | ||
| userInputDouble = Double.parseDouble(userInput); |
There was a problem hiding this comment.
🍏Ты 2 раза парсишь Double, лучше так не делать, а перестроить код так, чтобы это пришлось делать только 1 раз, например вытащить код из isDouble сюда и при ошибке выводить сообщение System.out.println("Введите корректную цену товара!");
| int temp = OGS % 100; | ||
| if (temp % 10 == 1 && temp != 11) | ||
| return "ь"; | ||
| else if ((temp % 10 == 2 || temp % 10 == 3 || temp % 10 == 4) && (temp != 12 && temp != 13 && temp!= 14)) |
There was a problem hiding this comment.
🍏(temp % 10 == 2 || temp % 10 == 3 || temp % 10 == 4) эквивалентно (temp % 10 > 0 && temp % 10 < 5), (temp != 12 && temp != 13 && temp!= 14 эквивалентно (temp<12 || temp > 14)
| //Поиск правильного окончания слова рубль | ||
| public static String checkCorrectLetter(int OGS){ | ||
| int temp = OGS % 100; | ||
| if (temp % 10 == 1 && temp != 11) |
There was a problem hiding this comment.
🍏Когда много if/else лучше использовать не ifы, а switch
| while (true) { | ||
|
|
||
| String userInput = scanner.next(); | ||
| if (isInt(userInput)) { |
There was a problem hiding this comment.
🍏Тоже самое замечание, что и с isDouble
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
2 participants
Калькулятор консольное приложение #1