Comments

Первый ПР к консольному приложению#1

Open

Mereinalamone wants to merge 2 commits intomain from

dev

Open

Первый ПР к консольному приложению #1
Mereinalamone wants to merge 2 commits intomain from
dev

Conversation

@Mereinalamone

Copy link

Owner

@Mereinalamone Mereinalamone commented Jul 30, 2023 •

edited

Loading

Первый ПР к консольному приложению калькулятор счёта

@Mereinalamone


 Первый комит к калькулятору счёта

27edc21

gusar-off

gusar-off reviewed

Jul 31, 2023

View reviewed changes

src/main/java/Main.java

while (true) {

try {

Scanner scanner = new Scanner(System.in);

numberPeople = scanner.nextInt();

Copy link

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

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

⚠️ Если передать в этот момент произвольную строку программа завершит работу с исключением. Перед тем, как прочитать инт из сканнера, нужно сначала проверить возможно ли это при помощи соответствующего метода hasNextInt

src/main/java/DivisionCheck.java Outdated

while(true) {

try{

System.out.printf("Введите название товара No%d: ", i);

Scanner scanner = new Scanner(System.in);

Copy link

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

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

Лучшее переиспользовать инстанс Scanner, а не создавать каждый раз новый.

src/main/java/DivisionCheck.java

try{

System.out.printf("Введите цену товара No%d: ", i);

Scanner scanner = new Scanner(System.in);

price = scanner.nextFloat();

Copy link

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

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

Тоже самое, что и с hasNextInt только тут надо hasNextFloat

src/main/java/DivisionCheck.java

}

System.out.println("Список добавленных товаров");

System.out.println("|Наименование товара |Стоимость |");

for (i = 0; i < productList.size(); i = i + 1) {

Copy link

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

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

Можно воспользоваться циклом foreach, было бы удобнее, подробнее можно почитать тут
https://javarush.com/groups/posts/for-each-java

@Mereinalamone


 Исправил замечания:

585974c

1) использовал методы hasNext() для обработки введённой пользователем инфармации
2) теперь переиспользуется инстанс Scanner, без пересоздания в каждом цикле как было

Labels

None yet

2 participants

@Mereinalamone @gusar-off

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

Первый ПР к консольному приложению#1

Первый ПР к консольному приложению #1
Mereinalamone wants to merge 2 commits intomain from
dev

Conversation

@Mereinalamone Mereinalamone commented Jul 30, 2023 •

edited

Loading

Uh oh!

Uh oh!

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

Uh oh!

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

Uh oh!

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

Uh oh!

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments

Conversation

@Mereinalamone Mereinalamone commented Jul 30, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

Uh oh!

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

Uh oh!

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

Uh oh!

@gusar-off gusar-off Jul 31, 2023

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

@Mereinalamone Mereinalamone commented Jul 30, 2023 •

edited

Loading