Conversation
src/main/java/CountFriends.java
Outdated
|
|
||
| static private int countFriends=0; | ||
| double countMoney; | ||
| static Scanner sc=new Scanner(System.in); |
There was a problem hiding this comment.
Некритичное замечание: хорошей практикой является делать вызов scanner.close() после того, как сканнер больше не используется. Это необходимо для того, что бы этот объект не потреблял ресурсы, тогда когда это уже не требуется.
src/main/java/CountFriends.java
Outdated
| public static int inputCountFriends(double countMoney) { | ||
| do { | ||
| System.out.println("На скольких человек необходимо разделить счёт?"); | ||
| countFriends = sc.nextInt(); |
There was a problem hiding this comment.
приложение завершается с ошибкой в строке countFriends = sc.nextInt() при вводе букв. Здесь нужно или обработать исключение, или сделать предварительно проверку if(sc.hasNextInt()){...}
src/main/java/Products.java
Outdated
| while(true) { | ||
| System.out.println("Введите стоимость товара"); | ||
| String stringPrice = sc.nextLine(); | ||
| price = Double.parseDouble(stringPrice); |
There was a problem hiding this comment.
приложение падает с ошибкой в строке price = Double.parseDouble(stringPrice); при вводе букв или символов, которые невозможно распарсить в число double. Здесь нужно или обработать исключение, или сделать предварительно проверку if(sc.hasNextDouble()){...}
| @@ -0,0 +1,42 @@ | |||
| import java.util.Scanner; | |||
There was a problem hiding this comment.
В java принят определенный стиль форматирования кода. Если не вдаваться в подробности, то отформатировать код можно быстрой комбинаций клавиш Ctrl+Alt+L(Windows) или (⌘+⌥+L)(Mac)
src/main/java/CountFriends.java
Outdated
| public static int inputCountFriends(double countMoney) { | ||
| do { | ||
| System.out.println("На скольких человек необходимо разделить счёт?"); | ||
| countFriends = sc.nextInt(); |
There was a problem hiding this comment.
Некритичное замечание: эта переменная не используется, ее можно удалить.
src/main/java/Products.java
Outdated
| price = Double.parseDouble(stringPrice); | ||
| if (price < 0) { | ||
| System.out.println("Неправильная стоимость товара"); | ||
| continue; |
There was a problem hiding this comment.
Некритичное замечание: этот continue можно убрать - все равно дальше программа закончит эту итерацию цикла и перейдет к следующей
src/main/java/Products.java
Outdated
| inputName = sc.nextLine(); | ||
| inputName = inputName.trim(); | ||
| if (inputName.equalsIgnoreCase("Завершить")) break; | ||
| else continue; |
There was a problem hiding this comment.
Некритичное замечание: этот continue можно убрать - все равно дальше программа закончит эту итерацию цикла и перейдет к следующей
src/main/java/Main.java
Outdated
| System.out.println("Hello world!"); | ||
| Scanner sc=new Scanner(System.in); | ||
|
|
||
| try { |
There was a problem hiding this comment.
Некритичное замечание: заворачивать почти весь код метода в блок try - не очень хорошая практика, тк мы точно не узнаем где произошла ошибка. Рекомендуется делать это точечно, то есть мы точно знаем , где большая вероятность возникновения исключений: при считывании и парсинге того, что вводит пользователь ( sc.nextInt(), sc.nextDouble(), Double.parseDouble() ) .Лучше обработать исключения именно в этих строках или же избежать возникновения исключения(если это, конечно, возможно). Здесь можно сделать проверку if(sc.hasNextInt()){}
No description provided.