ПР по домашнему заданию по результатам 2 спринта#122

Closed

AshZoom wants to merge 19 commits intoYandex-Practicum:master from

AshZoom:master

Closed

ПР по домашнему заданию по результатам 2 спринта #122
AshZoom wants to merge 19 commits intoYandex-Practicum:master from
AshZoom:master

Conversation

@AshZoom

Copy link

@AshZoom AshZoom commented Nov 8, 2022

Направляю на проверку домашнее задание по результатам 2 спринта

s-buvaka and others added 8 commits

July 27, 2022 22:57


 Init commit

83a7216

@AshZoom


 Change project name

@AshZoom


 Merge remote-tracking branch 'origin/master'

6c18393

@AshZoom


 Merge remote-tracking branch 'origin/master'

ab6a7a3

@AshZoom


 Merge remote-tracking branch 'origin/master'

e22643c

@AshZoom


 Merge remote-tracking branch 'origin/master'

b852f71

@AshZoom


 Merge remote-tracking branch 'origin/master'

fa20498

@AshZoom


 Домашнее задание по реузльтатам 2 спринта А.Шмаков

d0d3d01

faritowich

faritowich reviewed

Nov 9, 2022

View reviewed changes

src/main/java/Main.java Outdated

Double result=0.00;

//вводим количество участников и сохраняем в переменной sTart

People sTart=new People();

Copy link

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Название переменной sTart не совсем понятно. Лучше делать наименования более содержательными. Тема названий, да и не только хорошо раскрывается во 2 главе книги Роберта Мартина "Чистый код". Также на хабре есть краткий конспект книги: https://habr.com/ru/post/485118/
Также все поля и методы класса помечены static, это означает, что для работы с ними не нужно создавать экземпляр класса, а можно вызвать его напрямую из класса, например People.countPeople()

Copy link

Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено
"Также на хабре есть краткий конспект книги: https://habr.com/ru/post/485118/" @azat Nabiullin: Спасибо за рекомендацию!

src/main/java/Main.java Outdated

System.out.println(list);

System.out.println(String.format("Итого: "+"%.2f"+" руб.",Products.total));

System.out.println("Делим на: "+sTart.numberOfPeople+" участников");

result=Products.total/sTart.numberOfPeople;

Copy link

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно People.numberOfPeople вместо sTart.numberOfPeople

Copy link

Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено

src/main/java/Main.java Outdated

//вывод данных

System.out.print("Всего добавлено в список: "+(Products.count-1)+" поз."+"\n");

System.out.println(list);

System.out.println(String.format("Итого: "+"%.2f"+" руб.",Products.total));

Copy link

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно использовать метод printf:

System.out.printf("Итого: "+"%.2f"+ "руб.%n",Products.total);

Copy link

Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено.
@azat Nabiullin:
Я правильно понимаю разницу между методом println и printf (см/ ниже)?
Используем метод System.out.printf,т.к. хотим просто вывести строку в консоль. Если бы мы хотели получить отформатированную строку для дальнейшего использования в коде, используем System.out.println.

src/main/java/People.java Outdated

}

if (numberOfPeople <= 1) {

System.out.println("Неправильный ввод. Попробуйте ввести число участников ещё раз.");

Copy link

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно весь этот блок с if (numberOfPeople <= 1) просто перенести в блок else предыдущего if

Copy link

Author

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено.
Спасибо! Т.к. это моя первая программа сразу не увидел такую возможность

src/main/java/Products.java Outdated

String Name;//объявляем статическую переменную списка позиций для передачи в класс Main

Double Price;

Copy link

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В Java принята конвенция, по которой все имена переменных называются camelCase'ом, то есть с маленькой буквы. Почитать про конвенции названий можно здесь:
https://www.oracle.com/java/technologies/javase/codeconventions-namingconventions.html

Copy link

Author

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено.
@azat Nabiullin: Спасибо за рекомендацию!

src/main/java/Products.java Outdated

Scanner scanner = new Scanner(System.in);

System.out.println("Укажите наименование Товара:");

String name1 = scanner.next();

Copy link

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше избегать названий типа name1, price2 и т.д. Здесь можно назвать просто name.

Copy link

Author

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено

src/main/java/Products.java Outdated

name1=String.format( "Поз."+count+": "+name1+" "+ "%.2f"+" руб.",j);

System.out.println("Товар добавлен в список "+ name1);

count++;//количество позиций в списке

total=total+j;//вычисляем общую стоимость всех введенных товаров

Copy link

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно использовать короткую конструкцию total += j вместо total=total+j

Copy link

Author

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Спасибо! Исправлено.

src/main/java/Products.java Outdated

}

//метод проверки выхода из процесса составления списка товаров

public static Boolean exitOfList() {

Copy link

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Метод возвращает boolean, но при этом возвращаемый тип нигде не используется, а просто присваивается значение переменной finishOfList и в цикле while проверяется.
Так что в этом методе можно ничего не возвращать и сделать void, а возвращаемое значение убрать

Copy link

Author

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено.

AshZoom and others added 10 commits

November 14, 2022 13:41

@AshZoom


 Исправление замечаний от 09.11.22 (Домашнее задание по результатам 2 ...

d318a18

...спринта А.Шмаков)

@AshZoom


 Исправление замечаний от 09.11.22 (Домашнее задание по результатам 2 ...

612491c

...спринта А.Шмаков)

@AshZoom


 Merge remote-tracking branch 'origin/master'

3c1fd83

@AshZoom


 Merge remote-tracking branch 'origin/master'

20a83e0

@AshZoom


 Merge remote-tracking branch 'origin/master'

4678bed

@AshZoom


 Merge branch 'master' into main

@AshZoom


 Merge pull request #1 from AshZoom/main

Init commit

@AshZoom


 Merge remote-tracking branch 'origin/master'

05a9371

@AshZoom


 Merge remote-tracking branch 'origin/master'

cc2f795

@AshZoom


 Merge remote-tracking branch 'origin/master'

b977276

faritowich

faritowich reviewed

Nov 27, 2022

View reviewed changes

src/main/java/Products.java Outdated

} else{

finishOfList=true;

}

return;

Copy link

@faritowich faritowich Nov 27, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В методах, не возвращающих результат (void) return в конце метода не нужен. return нужен для возвращения результата, а смысл в void в том, что он результат не возвращает.

Copy link

Author

@AshZoom AshZoom Nov 30, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено. Спасибо!

src/main/java/Products.java

} else{

finishOfList=true;

}

Copy link

@faritowich faritowich Nov 27, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно влю эту if-else конструкцию упростить до такой:
finishOfList= !add.equalsIgnoreCase("завершить");

Copy link

Author

@AshZoom AshZoom Nov 30, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено. Не знал о такой конструкции :(
Спасибо! Буду использовать.

src/main/java/Products.java Outdated

String priceOfgood = scanner.next();

j = Double.parseDouble(priceOfgood);

if(j<0){

o=10/0;

Copy link

@faritowich faritowich Nov 27, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно обойтись и без o=10/0, а просто выводить сообщение об ошибочном вводе. break в if нет так что и так будет повторный запрос ввода.
Кстати, когда надо искусственно вызвать ошибку, можно использовать конструкцию throw new Exception("Текст ошибки");

Copy link

Author

@AshZoom AshZoom Nov 30, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено. Я торопился сдать работу, чтобы успеть приступить к проектной работе No2, т.к. запутался как правильно оформлять Pull Request. Опыта в программировании нет и поэтому я решил задачу"в лоб" через костыли (деление на ноль).
Спасибо! Информацию по вызову ошибку записал для дальнейшего использования/

@AshZoom


 Исправление замечаний по проеткной работе 1

461b194

@thathorizon thathorizon closed this

Aug 15, 2023

Labels

None yet

3 participants

@AshZoom @faritowich @thathorizon

Conversation

@AshZoom AshZoom commented Nov 8, 2022

Uh oh!

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 9, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 14, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 27, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 30, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 27, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 30, 2022

Choose a reason for hiding this comment

Uh oh!

@faritowich faritowich Nov 27, 2022

Choose a reason for hiding this comment

Uh oh!

@AshZoom AshZoom Nov 30, 2022

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Comments