Домашнее задание (Проектная работа №1) by GabovAleksander · Pull Request #1 · GabovAleksander/Java-Module-Project-YP

GabovAleksander · 2023-09-17T14:10:17Z

Проектная работа №1

kgornostaeva · 2023-09-18T16:18:41Z

 public class Main {
+
+
+    private static String getCurrencyText(BigDecimal price){


🍏 Лучше вынести этот метод в отдельный класс Formatter, чтобы его можно было использовать в разных частях приложения.

kgornostaeva · 2023-09-18T16:20:02Z

+        String roundPrice=price.toBigInteger().toString();
+        String currency;
+        int lastDigit=Integer.parseInt(roundPrice.substring(roundPrice.length()-1));
+        if (lastDigit==1){


⚠️ Этот алгоритм не предусматривает случаев от 11 по 14. В этих случаях тоже должно быть "рублей"

kgornostaeva · 2023-09-18T16:21:10Z

+        Check check;
+        Integer countPerson;
+
+        NumberFormat nf = NumberFormat.getCurrencyInstance();


🍏 Рекомендую не экономить на количестве букв и писать более развернутые понятные наименовая переменных

kgornostaeva · 2023-09-18T16:23:07Z

+            person= EnterData.enterGuest(check.getCountPerson());
+
+            //Приводим к валюте и убираем обозначение валюты
+            Position position=new Position(nameDish,price,person);


🍏 Немного поехало форматирование. Для форматирования кода и удаления неиспользуемых импортов применяй следующие сочетания клавиш:
Windows: Control+Alt+L
MacOS: Command+Option+L
Больше горячих клавиш можно найти здесь: https://developer.android.com/studio/intro/keyboard-shortcuts

kgornostaeva · 2023-09-18T16:26:33Z

+            System.out.println(
+                    String.format("Добавленные товары: %s. %-50s  %20s %s  Гость №%s",


🍏 Можно упростить вот так:

Suggested change

System.out.println(

String.format("Добавленные товары: %s. %-50s %20s %s Гость №%s",

System.out.printf("Добавленные товары: %s. %-50s %20s %s Гость №%s%n",

kgornostaeva · 2023-09-18T16:35:22Z

+        for(int i=1;i<=check.getCountPerson();i++){
+            int count=0;
+            BigDecimal itog=BigDecimal.ZERO;
+            for (Position position:check.getPositionList()
+                 ) {
+                if (position.getGuest().equals(i)){
+                    count++;
+                    itog=itog.add(position.getPrice());
+                    System.out.println(
+                    String.format("%s. %-50.45s  %20s %s",
+                            count, position.getName(), nf.format(position.getPrice()), getCurrencyText(position.getPrice())));
+                }
+            }


🍏 Здесь получился сложный алгоритм с вложенным циклом. Если у тебя будет 20 гостей и 40 блюд (каждый гость заказал по 2 блюда), то ты пройдешь по массиву блюд 20 раз. Можно создать map или массив чисел по количеству гостей, пройти по массиву блюд один раз и записать для каждого гостя итоговую сумму.

kgornostaeva · 2023-09-18T16:38:52Z

+        BigDecimal price;
+        String readLine;
+        while (true) {
+            System.out.println("Введите цену блюда: ");


🍏 Круто было бы подсказать пользователю, в каком формате от него ожидаются данные. Например, [руб,коп]

kgornostaeva · 2023-09-18T16:41:35Z

+            try {
+                Scanner scanner = new Scanner(System.in);
+                readLine = scanner.nextLine();
+                if(readLine.trim().length()-readLine.trim().indexOf(".")<=3||readLine.trim().indexOf(".")==-1) {


🍏

Suggested change

if(readLine.trim().length()-readLine.trim().indexOf(".")<=3||readLine.trim().indexOf(".")==-1) {

if(readLine.trim().length()-readLine.trim().indexOf(".")<=3|| !readLine.trim().contains(".")) {

kgornostaeva · 2023-09-18T16:44:48Z

+            System.out.println("Введите цену блюда: ");
+            try {
+                Scanner scanner = new Scanner(System.in);
+                readLine = scanner.nextLine();


🍏 Для получения сразу числа с дробной частью можно использовать методы scanner.nextFloat, scanner.nextBigDecimal. Перед ними лучше сделать проверку scanner.hasNextFloat/scanner.hasNextBigDecimal или обработать исключение.

kgornostaeva · 2023-09-18T16:49:23Z

+
+public class EnterData {
+    public static Integer enterGuest(Integer countInCheck){
+        Integer person;


🍏 В большинстве случаев для простых операций достаточно использовать примитивный тип - int. О разнице можно почитать, например, вот здесь: https://habr.com/ru/articles/104231/

kgornostaeva · 2023-09-18T16:57:50Z

Привет, Александр! Извини за задержку с проверкой твоей работы!

GabovAleksander · 2023-09-18T19:26:47Z

Привет, Александр! Извини за задержку с проверкой твоей работы!

Привет, Екатерина! Ничего страшного в задержке нет :). В целом с замечаниями, кроме одного согласен, но т.к. задание учебное не хотелось на него тратить лишнее время. Не согласен с нулевой стоимостью блюда, в задании нет ничего про нулевую стоимость, есть только про отрицательную. В целом нулевая вполне может быть, подарок клиенту на день рождения, просто какая-то акция или замена блюда. Это было предусмотрено изначально и соответствует ТЗ вроде как.

GabovAleksander

Согласен с 11-19 при получении падежа, не согласен с 0 ценой блюда.

kgornostaeva · 2023-09-19T14:24:03Z

                if(readLine.trim().length()-readLine.trim().indexOf(".")<=3||readLine.trim().indexOf(".")==-1) {
                    price = new BigDecimal(readLine);
-                    if(price.compareTo(BigDecimal.ZERO)>=0){
+                    //Предыдущее условие >= соответствовало критериям задачи, стоимость не могла быть отрицательной, в данном случае она не может быть и нулевой,


👌 👌 👌
Комментарий лучше оставить средствами git, а не в коде, чтобы они не оставались в проекте)

kgornostaeva · 2023-09-19T14:26:24Z

+        String currency;
+        if (roundPrice.length()>1){
+            int lastDigits = Integer.parseInt(roundPrice.substring(roundPrice.length() - 2));
+            if(lastDigits>10&&lastDigits<20){


🍏 Для повышения читабельности:

Suggested change

if(lastDigits>10&&lastDigits<20){

if (lastDigits > 10 && lastDigits < 20) {

kgornostaeva · 2023-09-19T14:27:49Z

-        BigDecimal devidedSum=check.getTotalSum().divide(new BigDecimal(check.getCountPerson()),2,RoundingMode.HALF_UP);
-        System.out.println("При делении на всех, каждый должен заплатить: "+nf.format(devidedSum)+getCurrencyText(devidedSum));
+        BigDecimal devidedSum = check.getTotalSum().divide(new BigDecimal(check.getCountPerson()), 2, RoundingMode.HALF_UP);
+        System.out.println("При делении на всех, каждый должен заплатить: " + nf.format(devidedSum) + Formatter.getCurrencyText(devidedSum));


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

Домашнее задание (Проектная работа №1)

583aa37

kgornostaeva reviewed Sep 18, 2023

View reviewed changes

Домашнее задание (Проектная работа №1) v2

8e0f26e

GabovAleksander commented Sep 18, 2023

View reviewed changes

kgornostaeva reviewed Sep 19, 2023

View reviewed changes

		public class Main {


		private static String getCurrencyText(BigDecimal price){

		System.out.println(
		String.format("Добавленные товары: %s. %-50s %20s %s Гость №%s",

	System.out.println(
	String.format("Добавленные товары: %s. %-50s %20s %s Гость №%s",
	System.out.printf("Добавленные товары: %s. %-50s %20s %s Гость №%s%n",

	if(readLine.trim().length()-readLine.trim().indexOf(".")<=3\|\|readLine.trim().indexOf(".")==-1) {
	if(readLine.trim().length()-readLine.trim().indexOf(".")<=3\|\| !readLine.trim().contains(".")) {

	if(lastDigits>10&&lastDigits<20){
	if (lastDigits > 10 && lastDigits < 20) {

Conversation

GabovAleksander commented Sep 17, 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 Sep 18, 2023

Uh oh!

GabovAleksander commented Sep 18, 2023

Uh oh!

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

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants