проект_19.12 - #1

Добрый вечер! Отправила повторно, не знаю правильно или нет, заранее прошу прощения, как смогла) 19.12.2022, 20:23, "Alex Korneev" ***@***.***>***@***.*** commented on this pull request.В некоторых файлах едет код, стоит отрефакторить. Если ты используешь IDE от JetBrains, то можешь пройтись по всем файлам и нажать комбинацию (Windows: Shift + Ctrl + Alt + L, MacOS: Shift + Option + Command + L), затем выбрать какую часть кода хочешь изменить и выполнить команду. Среда разработки автоматически сделает нужные отступы и переносы (как она это делает можно изменить в настройках, но дефолтные значения вполне нормальные).In src/main/java/Calculate.java:> @@ -0,0 +1,22 @@ +public class Calculate { + +    int people; ⏫ Стоит сделать переменную private. Переменные, классы и методы лучше делать того уровня видимости, который необходим в той лии иной ситуации. Данная переменная используется только внутри класса, поэтому ей лучше быть приватной.In src/main/java/Calculate.java:> @@ -0,0 +1,22 @@ +public class Calculate { + +    int people; + +    static String listProduct = "Добавленные товары:"; +    static double sum = 0; Это тоже можно приватным сделать или оставить так и убрать метод getSum(), и сразу обращаться к переменнойIn src/main/java/Calculate.java:> @@ -0,0 +1,22 @@ +public class Calculate { + +    int people; + +    static String listProduct = "Добавленные товары:"; +    static double sum = 0; + +    Calculate(int people){ +        this.people = people; +    } +    void addProduct(Product product) { +        System.out.println(product.name + product.price + "Товар успешно добавлен"); Не помешает немножко пробелов, а то слегка склеивается вывод)In src/main/java/Main.java:>   -        int friendCount; -        while (true) { -            System.out.println("На сколько человек необходимо разделить счет?"); -            friendCount = scanner.nextInt(); - -            if (friendCount > 1) { -                break; -            } else if (friendCount == 1) { -                System.out.println( -                        "Нет смысла делить сумму на одного человека. Давайте попробуем ввести другое значение, которое будет больше единицы."); -            } else { -                System.out.println("Неверное количество друзей. Значение должно быть болье единицы, давайте попробуем еще раз."); +        while (!(scanner.hasNextDouble())) { System.out.println("Ошибка,нужно ввести именно число"); ⚠️ Дальше по коду мы ожидаем количество человек - целое число, но проверку здесь делаем на double, из-за этого при вводе не целого числа программа падает. Необходимо это исправить, стоит использовать hasNextInt(). Остальные некорректные кейсы обрабатываются хорошо.In src/main/java/Main.java:>   -            System.out.println( -                    "Хотите добавить еще один товар? Введите любой символ для продолжения, либо 'Завершить' если больше нет товаров для добавления"); -            String answer = scanner.next(); +                while (true) { +                    if (price > 0) { ⏫ Так как мы даем пользователю возможность вводить дробное число, возникает ситуация, когда над приложением захотят поиздеваться и ввести какую-нибудь нереальную сумму (например, 0,00000000009). В такой ситуации при округление получится, что все должны заплатить по 0 рублей и кто-то потеряет в деньгих. Можно было бы попробовать добавить валидацию таких ситуаций или сразу ограничить ввод 2 знаками после запятой и уведомить об этом пользователя. А может быть ограничить минимальную цену, которую можно указать.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>