Пулл реквес первой проектной работы #1

PanBuz · 2023-02-08T14:16:27Z

Вроде все работает...

SeveNChaK

В некоторых файлах код прыгает, отступы не одинаковые - рекомендую использовать встроенные форматтер. Если ты программируешь в IDE от JetBrains, то можешь пройтись по всем файлам и нажать комбинацию (Windows: Shift + Ctrl + Alt + L, MacOS: Shift + Option + Command + L), затем выбрать, какую часть кода хочешь изменить, и выполнить команду. Среда разработки автоматически сделает нужные отступы и переносы (как она это делает можно изменить в настройках, но дефолтные значения вполне нормальные).

SeveNChaK · 2023-02-09T16:05:20Z

src/main/java/Calkulater.java

+    double summ = 0;
+    double money = 0;
+    int ost;
+    double ostl;


⏫ Все поля можно сделать приватными

Сделал

SeveNChaK · 2023-02-09T16:14:30Z

src/main/java/Main.java

+        int person;
+
+
+        while (!sc.hasNextInt()) {


⚠️ Проверки хватает на один раз. Если один раз ввести некорректный int, то првоерка пройдет, но отфильтруется ниже, а если потом ввести буквы, то приложение упадает ниже на sc.nextInt()

Изменил, теперь должно быть все норм)

SeveNChaK · 2023-02-09T16:16:07Z

src/main/java/Main.java

+        }
+
+
+        while (true) {


⏫ Думаю весь этот цикл можно объединить в один предыдущий и тогда можно решить проблему с вводом букв

Разобрался подробнее, теперь все понял как надо, исправил)

SeveNChaK · 2023-02-09T16:17:08Z

src/main/java/Calkulater.java

+                    cost = sc.nextDouble();
+                    if (cost <= 0 ) {
+                        System.out.println("Не корректное цифра");}
+                       // else if (!sc.hasNextDouble()) {System.out.println("Буквы нельзя в стоимсть вводить");}


⚠️ Как раз не хватает этой проверки)

Теперь все есть, должно работать)

SeveNChaK · 2023-02-09T16:17:44Z

src/main/java/Calkulater.java

+                }
+
+                summ = summ + cost;
+                nameAll = nameAll + "\n" + name;


⏫ Так как String является неизменяемым объектом, то при каждом сложении строк создаются новые объекты, что расходует память. Для таких случаев лучше использовать StringBuilder

Так и не смог запустить программу с использованием этого класса, скинул пока без него, но буду изучать эту тему.

SeveNChaK · 2023-02-09T16:18:56Z

src/main/java/Calkulater.java

+                nameAll = nameAll + "\n" + name;
+
+                String messageTemplate1 = "Вы успешно добавили товар на общую сумму %.2f руб., добавьте товар еще, или наберите слово Завершить";
+                System.out.println(String.format(messageTemplate1, summ));


⏫ Можно заменить на System.out.printf(messageTemplate1, summ);

Заменил)

SeveNChaK · 2023-02-09T16:19:58Z

src/main/java/Calkulater.java

+
+        money = summ / person;
+
+        ostl = money % 10f;


⚠️ Этого не достаточно, иногда нужно првоерять 2 последние цифры. Например, для чисел оканчивающихся на 11-14

Заменил значение на 20, и добавил в свич вариантов до 19ти

SeveNChaK · 2023-02-09T16:20:52Z

src/main/java/Ostatok.java

+            case 9: rubli ="Рублей"; break;
+        }
+         String messageTemplate = "Вы заказали %s, скидываемся по %.2f %s.";
+         System.out.println(String.format(messageTemplate, nameAll, money, rubli));


⏫ Тут тоже можно подсократить, как писал уже выше

сократил, пришлось немного помудрить с переносом строки

SeveNChaK · 2023-02-09T16:21:16Z

src/main/java/Ostatok.java

+
+        switch (ost){
+            case 0: rubli ="Рублей";  break;
+            case 1: rubli ="Рубль"; break;


⚠️ Для 11 это сработает не верно. Так же как и для 12-14

добавил до 19ти

SeveNChaK · 2023-02-11T15:00:45Z

src/main/java/Calkulater.java

-    double money = 0;
-    int ost;
-    double ostl;
+    private double summ = 0;


⏫ Поля выше тоже можно сделать приватными

SeveNChaK · 2023-02-11T15:05:49Z

src/main/java/Calkulater.java

-
-        ostl = money % 10f;
-        ost = (int)ostl;
+        padegD = moneyPerPerson % 20f;


⚠️ Чтобы проверить 2 последние цифры нужно делить на 100, а не на 20. Ошибка со склонением все еще остается. Например, при общей сумме = 302, получаем 151 рублей.

И чтобы не делать большой свитч и все работало корретно, стоит просто получить 2 последние цифры числа (остаток от деления на 100), затем проверить, что это >=11 и <=14 (тогда пишем рублей), и если не попадет в это условие, то проверить уже остаток от деления на 10.

Исправил, проверил выводом всех значений.

SeveNChaK · 2023-02-16T22:33:35Z

src/main/java/Ostatok.java

 public class Ostatok {
-     void ostat (int padeg, double moneyPerPerson, String nameOfDishesAll){
+     void ostat (double moneyPerPerson, String nameOfDishesAll){
        String rubli = new String();


String rubli = "";

SeveNChaK · 2023-02-16T22:35:34Z

src/main/java/Ostatok.java

+
+         padegD = moneyPerPerson % 100f;
+         if (padegD > 14) {padegD = padegD % 10f;}
+         padeg = (int) padegD;


Можно убрать лишние шаги и преобразования, и переменную

int padeg = ((int) moneyPerPerson) % 100; if (padeg > 14) { padeg = padeg % 10; }

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

Add dev branch

e6fb7da

Первая попытка сдать проектную работу номер 1

78a61ab

Добавил еще класс, убрал баги

e9f7088

SeveNChaK reviewed Feb 9, 2023

View reviewed changes

Исправил баги, теперь прога не падает при введение не тех символов

3b5f280

SeveNChaK reviewed Feb 11, 2023

View reviewed changes

Исправил правильное склонение "рублей"

fde71bb

SeveNChaK reviewed Feb 16, 2023

View reviewed changes

Пулл реквес первой проектной работы #1

Are you sure you want to change the base?

Пулл реквес первой проектной работы #1

Uh oh!

Conversation

PanBuz commented Feb 8, 2023

Uh oh!

SeveNChaK 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!

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!

3 participants