Завершенное задание by black-wolf17 · Pull Request #1 · black-wolf17/Java-Module-Project-YP

black-wolf17 · 2023-02-21T21:07:41Z

Созадал калькулятор счёта

kirillNay

Комментарии с иконкой ⚠️ пометил как критические, их исправление важно для прохождения на следующий этап.
Остальные комментарии носят рекомендательный характер.

kirillNay · 2023-02-22T13:59:14Z

src/main/java/Formater.java

@@ -0,0 +1,16 @@
+public class Formater {


Хорошая идея вынести в отдельный класс логику форматирования суммы!
Я бы посоветовал вынести из функции format логику вывода результата в консоль и просто отдавать result в качестве результата работы метода.

При таком подходе мы повышаем гибкость нашей программы. Например, если в следующий спринтах при разработке уже Android-приложения нам потребуется функционал форматирования, то мы сможем легко переиспользовать наш класс без внесения изменений формата вывода данных.

Это один из ярких примеров принципа единственной ответственности из принципов SOLID

kirillNay · 2023-02-22T14:31:08Z

src/main/java/Formater.java

@@ -0,0 +1,16 @@
+public class Formater {
+    public static void format(double sum, String rub){


Стоит сделать аргумент rub локальной переменной функции, т.к. мы нигде не используем значение, которое функции передали извне.

Также можно обратить внимание на подсказки Android Studio

kirillNay · 2023-02-22T14:46:18Z

src/main/java/Formater.java

+        if(sum >= 1 && sum < 2){
+            rub = "рубль";
+        } else{
+            if (sum >= 2 && sum < 5){


Я бы посоветовал избегать лишних вложенных конструкций, чтобы код казался более читаемым. Данную проверку можно заменить на цепочку if else:

if (...) { ... } else if (...) { ... }

kirillNay · 2023-02-22T14:52:36Z

src/main/java/GroceryList.java

+import java.util.Scanner;
+
+public class GroceryList {
+    public static void product(int people) {


Функцию product не обязательно делать статичной. Обычно static используется для функций, которые не требуют создания экземпляра класса.

Например, для функции main есть требование, чтобы она была статично и система могла ее вызвать без создать объекта класса Main. Но для GroceryList мы явно создаем объект этого класса

kirillNay · 2023-02-22T14:53:53Z

src/main/java/GroceryList.java

+    public static void product(int people) {
+        double sum = 0.0;
+        String Products = ("Добавленные товары:");
+        String rub = null;


Переменная rub так и остается неизменной и равной null. Вполне возможно, ее инициализация тут осталась по ошибке)

kirillNay · 2023-02-22T15:07:31Z

src/main/java/GroceryList.java

+            System.out.println("Введите название товара");
+            String name = NameProduct.nextLine();
+
+            Scanner PriceProduct = new Scanner(System.in);


Объект класса Scanner - достаточно тяжелый объект и занимает много места в памяти. Мы вполне можем оптимизировать его использование и применять лишь один ранее созданный Scanner при каждом чтении значения из консоли.

Однако тут главное не попасться в ловушку завершения строки. Дело в том, что методы nextInt или nextDouble не считывают символ завершения строки, который мы вводим нажимая "Enter". Из-за этого последующий методы nextLine будет считывать завершение строки, и нам будет казаться, что он игнорируется.
Описание подобной проблемы и ее решение есть на StackOverflow

kirillNay · 2023-02-22T15:08:27Z

src/main/java/Main.java

+                System.out.println("Зачем тебе делить свою еду?\nВведите другое количество");
+            }
+            if (people < 1){
+                System.out.println("А положительных друзей у тебя нет?\nВведите положительное число");


А положительных друзей у тебя нет?

😄

kirillNay · 2023-02-22T15:10:45Z

src/main/java/Main.java

+        System.out.println("На скольких человек поделить счёт?");
+
+        while (true) {
+            int people = scanner.nextInt();


⚠️
Пользователь может ввести на вопрос про количество человек и стоимость товара вместо чисел строки, и приложение упадет. Лучше такого не допускать, а обрабатывать. Для проверки этих ситуаций можно применять методы scanner.hasNextInt() / hasNextFloat() или же оборачивать чтение в try-catch.

kirillNay · 2023-02-22T15:11:25Z

src/main/java/GroceryList.java

+
+
+            while(true) {
+                double price = PriceProduct.nextDouble();


⚠️
Нет обработки бесплатных товаров. Если ввести в качестве цены товара 0, то программа не даст продвинуться дальше и будет попрежнему ожидать ввода положительного значения.

kirillNay · 2023-02-22T15:12:18Z

src/main/java/Formater.java

+            if (sum >= 2 && sum < 5){
+                rub = "рубля";
+            } else {
+                rub = "рублей";


⚠️
Для определения правильного падежа слова “рубль” стоит обратить внимание на цифру, на которую оканчивается целочисленное значение итоговой сумма. Например, если итоговый счет будет равен 21, то при выводе мы будет ожидать “21 рубль”. Кроме того, стоит не забывать, что для чисел “11-19” правильный вариант будет “рублей” вне зависимости от того, на что число оканчивается.

kirillNay · 2023-02-24T22:13:56Z

src/main/java/GroceryList.java


        while (true) {
-            Scanner NameProduct = new Scanner(System.in);
+            Scanner priceProduct = new Scanner(System.in);


Название priceProduct предполагает, что данная переменная хранит в себе цену продукта. Но на самом деле это лишь сканнер, который может использоваться для считывания разного рода информации и не отнесящейся к цену за продукт

Поэтому предлагаю для данной переменной использовать название просто scanner

kirillNay · 2023-02-24T22:21:02Z

src/main/java/Formater.java

+            rub = "рублей";
        }
+        Formater formater = new Formater();
+        formater.result(sum, rub);


Для вызове функции result нет необходимости создавать еще один объект класса Formater, ее можно вызвать просто через result(sum, rub);

В комментарии c67a4c4#r1114363706 я имел в виду возможность избавить класс Formater от логики вывода данных. Все, что возвращал бы его метод format - отформатированную строку

А в месте вызова данного метода уже бы обрабатывалась логика вывода данных

kirillNay · 2023-02-24T22:22:39Z

src/main/java/Formater.java

-            rub = "рубль";
-        } else{
-            if (sum >= 2 && sum < 5){
+    public static void format(double sum){


Как и с классом GroceryList, методы format и result можно оставить без static, т.к. эти методы предполагают создание объекта этих классов

s-buvaka and others added 2 commits November 30, 2022 22:20

Add dev branch

e6fb7da

Коммит с созданным калькулятором

fc30b0e

kirillNay suggested changes Feb 22, 2023

View reviewed changes

Второй коммит с созданным калькулятором

c67a4c4

kirillNay approved these changes Feb 24, 2023

View reviewed changes

		@@ -0,0 +1,16 @@
		public class Formater {
		public static void format(double sum, String rub){

Conversation

black-wolf17 commented Feb 21, 2023

Uh oh!

kirillNay left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants