Выполненная проектная работа №1 by hasket · Pull Request #1 · hasket/Java-Module-Project-YP

hasket · 2023-10-22T08:48:09Z

No description provided.

…ограмма не очень аохожа на программу программиста, но я обдумывал это решение до пол 2-го ночи, вроде даже реально похоже на подобие норм программы.

Anna-Malinina · 2023-10-22T11:04:06Z

+
+            try {
+                // вызывал конструктор parseDouble типа Double и перевожу строку в double
+                // Сам я не додумался, а за меня додумался интернет


Это норма) Главное уметь искать и понимать, что ты нашел)

Anna-Malinina · 2023-10-22T11:07:17Z

+
+    public static double sumPrice(HashMap <String, Double> productsMap){
+        double sumPriceResult = 0.00f;
+        for (Object i: productsMap.keySet()) {


Так как у мапы тип ключей String, то вместо Object было бы корректнее сразу написать String

Anna-Malinina · 2023-10-22T11:09:00Z

+        if (floor(price) == 1.00 || floor(price) == 1.0) {
+           return "рубль";
+       }
+        return "рубля";


Есть несколько случаев:

Оканчивается на 1- рубль

3-4 - рубля

0, 5-9, 11 - 19 - рублей

Anna-Malinina · 2023-10-22T11:10:19Z

+I'm so sorry
+
+##
+##  Calculator vashego nastorenia version 0.3


Люблю такие комментарии (❁´◡`❁)

Anna-Malinina · 2023-10-22T11:11:32Z

+
+        while (humans <= 1) {
+            System.out.print("Введите количество гостей ");
+            humans = scanner.nextInt();


Если ввести не интовое значение, то упадет. Нужно либо оборачивать в try-catch, либо проверять ввод через метод scanner.hasNextInt()

Anna-Malinina · 2023-10-22T11:12:09Z

+        Formatter formatter = new Formatter();
+        double productsSum = 0.00f;
+        double productPrice = 0.00f;
+        HashMap<String, Double> productsMap = new HashMap<String, Double>();


Отличный вариант использовать мапу)

Anna-Malinina · 2023-10-22T11:13:14Z

+                // Сам я не додумался, а за меня додумался интернет
+                productPrice = Double.parseDouble(scanner.next());
+                if (productPrice > 0) {
+                productsMap.put(productName, productPrice);


Здесь съехало форматирование, должен быть сдвинут на таб блок. Поможет комбинация ctrl + alt + L или cmd + opt + L

Anna-Malinina · 2023-10-22T11:15:31Z

+                    System.out.println("Добавленные товары:\n");
+                    for (String i : productsMap.keySet()) {
+                        System.out.printf("Товар \"%s\" стоимостью \"%s\" %s\n", i,
+                                productsMap.get(i).toString().replace(",", "."),


В русском языке разделитель дробной и целой части запятая, в английском точка. Раз мы общаемся на русском, то можно было оставить запятую)

Anna-Malinina · 2023-10-22T11:15:50Z

+                productsMap.put(productName, productPrice);
+                System.out.println("Товар успешно добавлен\n");
+
+                System.out.print("Желаете добавть ещё товар? \n");


Опечаточка в добавить

Anna-Malinina · 2023-10-22T11:16:32Z

+                        wishQuestion.equalsIgnoreCase("нет")) {
+                    System.out.println("Добавленные товары:\n");
+                    for (String i : productsMap.keySet()) {
+                        System.out.printf("Товар \"%s\" стоимостью \"%s\" %s\n", i,


Чтобы слегка облегчить метод и улучшить читаемость, все эти принты можно было бы вынести в отдельный метод

Anna-Malinina · 2023-10-23T07:19:03Z

-       }
-        return "рубля";
+// Это студия такой красивый формат(котлин стайл)
+// привела, сам не догадался =(


Все приходит с опытом) Потом привыкнешь и уже будет у самого глаза болеть от кривого форматирования))

Anna-Malinina · 2023-10-23T07:24:17Z

-        return "рубля";
+// Это студия такой красивый формат(котлин стайл)
+// привела, сам не догадался =(
+        return switch ((int) floor(price)) {


Чтобы проверить правильно окончание, следует посчитать два значения: 1) Последнюю цифру числа - это остаток от деления на 10, то есть price % 10. 2) Последние две цифры - это остаток деления на 100: price % 100.

Далее первым делом надо проверить, входит ли остаток от деления на 100 в диапазон 11-19. Это тот случай, когда будет рублей.
Потом проверяем остаток деления на 10. Если 1 - это рубль (чтобы учесть случаи, когда число кончается на 1: 21, 31, 101 и тд). Если 2-4 - рубля. И остальное будет рублей

про деление по модулю знаю, использовал, тут не стал по невнимательности.
и не учёл про то, что цена может быть больше 19-ти рублей

Anna-Malinina · 2023-10-23T07:30:32Z

-            if (humans < 1) {
-                System.out.println("Счёт не делится на такое количество человек.\n" +
-                        "Введите корректное число людей");
+            try {


Anna-Malinina · 2023-10-23T07:34:14Z

+                formatter.Formatter(productsSum));
+        System.out.printf("Каждый посетитель должен заплатить %.2f %s \n",
+                Calculator.priceForPerson(productsMap, humans),
+                //Насколько правильно вызвать функцию, в которой вызывается функция?   <------


Да в целом так делают) Если выражение не становится супер сложным и длинным

…у синтаксиса при ошибках, поправил ввод числа, теперь он и с запятой правильно записывает и с точкой

Anna-Malinina · 2023-10-24T13:30:34Z

+        int modPrice = (int) (floor(price));
+        //это хотфикс, потом понял, где неправильно
+        // и в итоге не смог быстро всё пофиксить по нормальному
+        if (modPrice % 100 > 11 && modPrice % 100 < 15)


Ну вообще можно было просто сохранить mod 10, mod 100, а потом в свитч кейсе сначала мод 100 проверить на 11-15, а потом мод 10 на все остальное

Anna-Malinina · 2023-10-24T13:30:54Z

                }
+            } catch (Exception e) {
+                System.err.println("Ёкарный бабай, шнурок, целое число больше одного введи");
+                // Как понял из описаний в интернетах, из-за того, что первый раз мы вводим неправильный


Я тоже так поняла))

Anna-Malinina · 2023-10-24T13:31:34Z

                } else {
-                    System.out.println("Ёкарный бабай, шнурок, когда последний раз за " +
-                            "твой кофе заправка платила?");
+                    //Моя девушка сказала, раз эт неправильно, то почему оно не красится=)


Ничесе, я такого варианта еще не встречала в этой работе)) Девушке респект

Artem Vazerov added 2 commits October 21, 2023 13:04

Сделал первые 2 задания практической работы

9b8a0cf

Anna-Malinina reviewed Oct 22, 2023

View reviewed changes

Сделал правки согласно ревью

ba94dc3

Anna-Malinina reviewed Oct 23, 2023

View reviewed changes

Artem Vazerov added 4 commits October 24, 2023 00:35

хотфикс

2c1c71d

хотфикс

75f566a

коммент

62dfdce

Anna-Malinina reviewed Oct 24, 2023

View reviewed changes

Conversation

hasket commented Oct 22, 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!

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