Здравствуйте! by dariaban · Pull Request #1 · dariaban/Java-Module-Project-YP

dariaban · 2023-08-27T21:43:04Z

Понимаю, что это ужасно, но учусь программировать с нуля и это все, что я пока могу. Извините:(

kgornostaeva · 2023-08-28T16:10:09Z

+
+class Dish {
+    Scanner scanner = new Scanner(System.in);
+    Scanner scanner1 = new Scanner(System.in);


🍏 Достаточно использовать один Scanner. Если при этом программа начинает работать некорректно, проверь, что циклы написаны правильно.

kgornostaeva · 2023-08-28T16:12:13Z

+import java.util.Scanner;
+
+
+class Dish {


🍏 Рекомендую класс Dish, в котором будут поля для сохранения названия и цены блюда. А операции со списком блюд вынести в отдельный класс.
Также можно вынести в отдельный класс форматирование вывода (слова "рубль")

kgornostaeva · 2023-08-28T16:13:15Z

+    Scanner scanner1 = new Scanner(System.in);
+    public String finish = "Завершить";
+    public double sumOfDishes;
+    public String name, check = "проверка";


🍏 Для лучшей читаемости лучше разнести объявление этих двух переменных в разные строки

kgornostaeva · 2023-08-28T16:15:10Z

+    Scanner scanner = new Scanner(System.in);
+    Scanner scanner1 = new Scanner(System.in);
+    public String finish = "Завершить";
+    public double sumOfDishes;


🍏 Сейчас любое из этих полей можно изменить снаружи класса. Это не очень хорошо, лучше для внутренних переменных использовать модификатор private.
https://javarush.com/groups/posts/1988-modifikatorih-dostupa-private-protected-default-public

kgornostaeva · 2023-08-28T16:16:25Z

+class Dish {
+    Scanner scanner = new Scanner(System.in);
+    Scanner scanner1 = new Scanner(System.in);
+    public String finish = "Завершить";


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

kgornostaeva · 2023-08-28T16:24:16Z

+        return numberOfPeople;
+    }
+    public void peoplescalculator2(){
+        while (!scanner.hasNextInt()){


🍏 Круто, что обрабатываешь неверный ввод!

kgornostaeva · 2023-08-28T16:25:55Z

+
+        return numberOfPeople;
+    }
+    public void peoplescalculator2(){


🍏 Рекомендую давать методам более понятные названия. Со стороны не ясно, в чем может быть разница между peoplescalculator1 и peoplescalculator2.

kgornostaeva · 2023-08-28T16:39:18Z

+    }
+
+    public void result() {
+        sumPerPerson = sumOfDishes / numberOfPeople;


🍏 Здесь происходит неявное приведение суммы конечного результата к integer, так как производится операция между разными типами чисел (double и integer). Из-за этого в итоге мы получаем результат без копеек

kgornostaeva · 2023-08-28T16:40:11Z

+
+    public void result() {
+        sumPerPerson = sumOfDishes / numberOfPeople;
+        System.out.println("Вы заказали " + "\n" + listOfNAmes + "\n" + ". Общая стоимость стоимость заказа " + sumOfDishes + " " + "Каждый заплатит " + String.format("%.2f", Math.floor(sumPerPerson)) +" "+ "рубля");


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

kgornostaeva · 2023-08-28T16:40:46Z

@@ -1,6 +1,26 @@
+import java.util.ArrayList;
+import java.util.Scanner;


🍏 Остались неиспользуемые импорты, лучше их удалить

kgornostaeva · 2023-08-28T16:43:23Z

Понимаю, что это ужасно, но учусь программировать с нуля и это все, что я пока могу. Извините:(

Рекомендую никогда не извиняться заранее за свою работу) Все когда-то учились) У тебя хорошая первая попытка!

kgornostaeva · 2023-08-29T08:48:48Z

-

+}
+    class Calculator {


🍏 В Java принято каждый класс выносить в отдельный файл. В Kotlin это не будет требоваться

kgornostaeva · 2023-08-29T08:49:45Z

+}
+    class Calculator {
+        Scanner scanner = new Scanner(System.in);
+        StringBuilder builder = new StringBuilder();


🍏 Не используется

kgornostaeva · 2023-08-29T08:53:45Z

+                } else {
+                    System.out.println("Делим на " + numberOfPeople + " человек.");
+                }
+            } while (numberOfPeople < 1);


⚠️ < 2 или <= 1

kgornostaeva · 2023-08-29T08:55:55Z

            System.out.println("Введите название блюда");
-            scanner.next();
+            while (!scanner.hasNext()) {
+                System.out.println("Что-то пошло не так!");


🍏 Строку "Что-то пошло не так!" можно вынести в константу, так как она повторяется несколько раз в коде приложения. В этом случае если тебе нужно будет ее немного изменить, например, написать "Ой, что-то пошло не так!", то ты внесешь изменения в одном месте кода, а не вовсех местах использования.

kgornostaeva · 2023-08-29T09:01:09Z

+            double number = (int) numberOfPeople;

+            sumPerPerson = sumOfDishes / number;
+            StringBuilder builder = new StringBuilder("Вы заказали " + "\n" + listOfNAmes + "\n" + "Общая стоимость стоимость заказа " + String.format("%.2f", Math.floor(sumOfDishes)) + " " + formater.formaterSum(sumOfDishes) + "." + " " + "Каждый заплатит " + String.format("%.2f", Math.floor(sumPerPerson)) + " " + formater.formaterSumPerPerson(sumPerPerson) +".");


🍏 Вся мощь StringBuilder в этой строке сведена к нулю, так как сначала выполнятся все действия операторов "+" и результат уже готовым будет передан в StringBuilder)
Попробуй для построения строки использовать методы самого StringBuilder:

StringBuilder builder = new StringBuilder("Вы заказали \n") builder.append(listOfNAmes) builder.append("\n") ... ... System.out.println(builder);

Больше про методы StringBuilder можно почитать вот здесь с примерами:
https://javarush.com/quests/lectures/questsyntaxpro.level09.lecture06

kgornostaeva · 2023-08-29T09:04:37Z

+    }
+class Formater {
+    public String format;
+    public String formaterSum(double sumOfDishes) {


🍏 formaterSum делает абсолютно то же самое, что и formaterSumPerPerson. Эти методы имеют одинаковую сигнатуру, принимают одинаковые типы аргументов и возвращают одинаковые типы значений. Достаточно использовать один метод, просто передавать в него разные данные.

kgornostaeva · 2023-08-29T09:08:43Z

+    public String formaterSum(double sumOfDishes) {
+        int rub = (int) (sumOfDishes % 10);
+
+        switch (rub) {


🍏 Возможно, для этой ситуации подойдет использование простого if elseIf else, так как здесь всего четыре возможных случая. Нужно проверить, что число входит в один из четырех интервалов:

== 1 рубль > 1 && <= 4 рубля >= 5 && <= 9 рублей ==0 рублей

kgornostaeva · 2023-08-29T09:09:55Z

+class Formater {
+    public String format;
+    public String formaterSum(double sumOfDishes) {
+        int rub = (int) (sumOfDishes % 10);


⚠️ Эта проверка не покрывает случаев 11 рублей, 12 рублей

kgornostaeva · 2023-08-29T09:11:10Z

 public class Main {
    public static void main(String[] args) {
-        int numberOfPeople = 0;
        Dish dish = new Dish();


🍏 Здесь dish уже не используется, можно удалить

…о очень полезно!

kgornostaeva · 2023-08-30T18:31:07Z

+    }
+
+    public String askName(String name) {
+        System.out.println("Введите название блюда");


🍏 Все повторяющиеся строки лучше выносить в константы)

kgornostaeva · 2023-08-30T18:32:03Z

+                scanner.next();
+            } else {
+                sumOfDishes = sumOfDishes + price;
+                System.out.println("Вы заказали: " + listOfNAmes + " " +  price + " на сумму: " + String.format("%.2f",sumOfDishes));


🍏 Здесь тоже можно использовать StringBuilder, чтобы не создавать промежуточных строк

kgornostaeva · 2023-08-30T18:34:00Z

+        double number = (int) numberOfPeople;
+
+        sumPerPerson = sumOfDishes / number;
+        StringBuilder builder = new StringBuilder("Вы заказали ");


🍏 Всю работу билдера по формированию результирующей строки можно вынести в отдельный метод, в который будут переданы необходимые параметры

kgornostaeva · 2023-08-30T18:34:25Z

+public class Formater {
+    public String format;
+
+    public String formater(double test) {


Учу Java с нуля, еле-еле заставила этого уродца работать.

3fa8671

kgornostaeva reviewed Aug 28, 2023

View reviewed changes

Второй вариант

72079e3

kgornostaeva reviewed Aug 29, 2023

View reviewed changes

Последний. привет, Екатерина, спасибо большое за комментарии, это был…

c446854

…о очень полезно!

kgornostaeva reviewed Aug 30, 2023

View reviewed changes

		@@ -1,6 +1,26 @@
		import java.util.ArrayList;
		import java.util.Scanner;

Conversation

dariaban commented Aug 27, 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!

kgornostaeva commented Aug 28, 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants