Андрей Трофимов консольное приложение №1 #1

A2T-user · 2023-10-19T07:44:26Z

Написал этот код, четко следуя заданию, хотя считаю что алгоритм можно улучшить

kgornostaeva · 2023-10-19T11:50:41Z

src/main/java/Main.java

+        int numberOfPersons = dataInput.numberOfPeople(scanner);
+        // Ввод товара
+        ArrayList<LineOfBill> listOfProducts = dataInput.productEntry(scanner);
+        scanner.close();        // Закрыть сканер


👏 Круто, что не забываешь завершить работу со сканнером

Спасибо

kgornostaeva · 2023-10-19T11:52:38Z

src/main/java/LineOfBill.java

+// Класс для хранения данных строки счета
+public class LineOfBill {
+    private String name;
+    private float prise;


🍏 Закралась опечатка)

Suggested change

private float prise;

private float price;

Исправил

kgornostaeva · 2023-10-19T11:54:22Z

src/main/java/LineOfBill.java

@@ -0,0 +1,15 @@
+
+// Класс для хранения данных строки счета
+public class LineOfBill {


🍏 Рекомендую назвать класс, например, Product. Это пригодится при масштабировании, если нам понадобится использовать данные не только для счета, но и для учета проданных товаров/блюд, к примеру

Согласен

kgornostaeva · 2023-10-19T11:57:27Z

src/main/java/DataInput.java

+    // Ввод кол-ва человек
+    public int numberOfPeople (Scanner scanner) {
+        int col;
+        String s = "\nВведите целое число больше 1.", s1 = "На сколько человек необходимо разделить счёт:";


🍏 Рекомендую давать переменным более развернутые названия, чтобы в любой строке кода легко было понять, с чем мы работаем.
Также в java принято каждую переменную писать в отдельной строке, так их проще читать:
https://www.oracle.com/java/technologies/javase/codeconventions-declarations.html

Неизменяемые поля можно сразу объявить константами, тогда они будут созданы один раз во время компиляции:
https://www.baeldung.com/java-constants-good-practices

Согласен

kgornostaeva · 2023-10-19T12:04:33Z

src/main/java/DataInput.java

+            listOfProducts.add(lb);                                               // и добавление его в массив
+            System.out.println("Товар успешно добавлен.");
+            System.out.println("Хотите добавить еще один товар?\nДля завершения процесса добавления товаров введите \"Завершить\":");
+        } while (!scanner.next().equalsIgnoreCase("Завершить"));


🍏 Можно вынести слово "Завершить" в константу. Тогда если в будущем мы захотим заменить завершающую команду на другое слово, то это нужно будет сделать в одном месте

Разумно. Внесу изменения

kgornostaeva · 2023-10-19T12:08:58Z

src/main/java/DataInput.java

+            if (col < 0) {
+                System.out.println("Это отрицательное число!" + s);
+            } else if (col == 0) {
+                System.out.println("На 0 делить нельзя!" + s + "\n" + s1);


🍏 При печати в памяти будут создаваться промежуточные строки из-за множественного использования оператора "+". Можно использовать StringBuilder, который сформирует только результирующую строку, System.out.printf или String.format().
https://habr.com/ru/articles/260767/

Согласен. Внесу изменения

kgornostaeva · 2023-10-19T12:15:41Z

src/main/java/DataInput.java

+            }
+            // Проверка - у цены должно быть две цифры после десятичной точки
+            String[] strPrice = String.valueOf(price).split("\\.", 2);    // Разбиваем строку на массив из 2-х строк: целая часть и дробная
+            if (strPrice[1].length() > 2) {                                     // Если в дробной части больше 2 знаков


🍏 число 2 можно вынести в константу и дать ему имя, чтобы всегда было понятно, почему у него именно такая величина

Не совсем понял о какой 2 идет речь.
В split 2 означает, что строка будет разбита на два массива. Там всегда будет двойка.
А в if без проблем поменяю

kgornostaeva · 2023-10-19T12:48:55Z

src/main/java/DataOutput.java

+        }
+        System.out.println(String.format("---------------------" + "Итого по счету:   %.2f\n", sumBill));         // Подчеркивание и вывод итоговой суммы по счету
+        double fromEveryPerson = sumBill / numberOfPersons;
+        System.out.println(String.format("\nГоспода, с Вас по:   %.2f " + rubles(fromEveryPerson) + "%n", fromEveryPerson));    // Вывод доли счета


Еще можно вот так:

Suggested change

System.out.println(String.format("\nГоспода, с Вас по: %.2f " + rubles(fromEveryPerson) + "%n", fromEveryPerson)); // Вывод доли счета

System.out.printf("\nГоспода, с Вас по: %.2f %s%n", fromEveryPerson, rubles(fromEveryPerson));

Я знаю, просто так нас учили. Исправлю

A2T-user · 2023-10-19T15:57:59Z

Спасибо за замечания. Внес все изменения. Отправляю на проверку

Андрей Трофимов консольное приложение №1

1923475

kgornostaeva reviewed Oct 19, 2023

View reviewed changes

Андрей Трофимов Консольное приложение №1. Версия 2.0

27d5a1a

A2T-user added 2 commits October 20, 2023 14:52

Андрей Трофимов Консольное приложение №1. Версия 3.0

afa1b3a

Андрей Трофимов Консольное приложение №1. Версия 4.0

76321e3

		@@ -0,0 +1,15 @@

		// Класс для хранения данных строки счета
		public class LineOfBill {

	System.out.println(String.format("\nГоспода, с Вас по: %.2f " + rubles(fromEveryPerson) + "%n", fromEveryPerson)); // Вывод доли счета
	System.out.printf("\nГоспода, с Вас по: %.2f %s%n", fromEveryPerson, rubles(fromEveryPerson));

Андрей Трофимов консольное приложение №1 #1

Are you sure you want to change the base?

Андрей Трофимов консольное приложение №1 #1

Uh oh!

Conversation

A2T-user commented Oct 19, 2023

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!

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!

A2T-user commented Oct 19, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants