Comments

И пусть теперь все получиться!#1

Open

maxshef wants to merge 5 commits intomain from

dev

Open

И пусть теперь все получиться! #1
maxshef wants to merge 5 commits intomain from
dev

Conversation

@maxshef

Copy link

Owner

@maxshef maxshef commented May 13, 2023

Это мой первый Pull Request, надеюсь теперь получиться!

maxshef added 2 commits

May 13, 2023 10:26

@maxshef


 И пусть теперь все получиться!

4ba7466

@maxshef


 И пусть теперь все получиться!

a82a0f7

ilshat-abdulin

ilshat-abdulin reviewed

May 13, 2023

View reviewed changes

src/main/java/Main.java Outdated

System.out.println("Сколько счетов у Вас будет?");

int howPeople = 0;

while (true){

howPeople = scanner.nextInt();

Copy link

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

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

⚠ Нужно еще учесть, что пользователь может ввести строку и тогда приложение упадёт. Лучше такого не допускать, а обрабатывать. Можно использовать методы scanner.hasNextInt()/hasNextFloat() или try-catch

src/main/java/Main.java Outdated

Comment on lines 21 to 22

String RUB;

String RUB2;

Copy link

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

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

🍏 Переменные лучше именовать используя lowerCamelCase-стиль

https://habr.com/ru/articles/513176/#:~:text=5.2.5%20%D0%98%D0%BC%D0%B5%D0%BD%D0%B0%20%D0%BD%D0%B5%20%D0%BA%D0%BE%D0%BD%D1%81%D1%82%D0%B0%D0%BD%D1%82%D0%BD%D1%8B%D1%85%20%D0%BF%D0%BE%D0%BB%D0%B5%D0%B9

src/main/java/Main.java Outdated

Comment on lines 25 to 57

if (endingRubles % 100 >= 11 && endingRubles % 100 <= 14) {

RUB = "рублей";

} else

switch (endingRubles %10){

case 1:

RUB = "рубль";

break;

case 2:

case 3:

case 4:

RUB = "рубля";

break;

default:

RUB = "рублей";

break;

}

if (endingRubles2 % 100 >= 11 && endingRubles2 % 100 <= 14) {

RUB2 = "рублей";

} else

switch (endingRubles2 % 10){

case 1:

RUB2 = "рублю";

break;

case 2:

case 3:

case 4:

RUB2 = "рубля";

break;

default:

RUB2 = "рублей";

break;

}

Copy link

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

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

⚠ Здесь дублируется код из-за двух переменных. Лучше не допускать повторений кода. Для избежания этого, можно вычисление окончания слова "рубль", вынести в метод, который на вход может принимать числовое значение и возвращать вычисленную строку. Следовательно, можно этот метод переиспользовать там, где это необходимо.

String formatRub(double price) {
 ////
}

src/main/java/ProductFeatures.java Outdated

String product = scanner.next();

listProducts += product + "\n";

System.out.println("Введите цену товара в формате ХХ,ХХ");

double productPrice = scanner.nextDouble();

Copy link

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

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

⚠ Здесь тоже есть необходимость добавить обработку некорректного ввода в виде строки

src/main/java/ProductFeatures.java Outdated

Comment on lines 14 to 18

if (productPrice <= 0) {

System.out.println("Введена неверная цена, цена должна быть больше 0.00");

}

if (productPrice > 0) {

}

Copy link

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

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

⚠ Вне зависимости от того, что производится проверка ввода отрицательного числа, значение все равно будет сохранено в finalProductPrice

image

У тебя уже есть проверка, в которой можно сохранять положительные значения

if (productPrice > 0) {
}

README.md

@@ -1 +1,2 @@

# Пустой репозиторий для работы с Java кодом в Android Studio

И пусть все получиться! No newline at end of file

Copy link

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

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

👍👍

@maxshef

Copy link

Owner Author

maxshef commented May 15, 2023

Добрый день, что-то в почте только смотрел, только увидел, что проверено еще 13 числа. Не очень понятно, все замечания надо исправить, или это для дальнейшего. Короче я не очень понимаю как понять принята работа или нет, где это можно увидеть?

@ilshat-abdulin

Copy link

ilshat-abdulin commented May 16, 2023 •

edited

Loading

Добрый день, что-то в почте только смотрел, только увидел, что проверено еще 13 числа. Не очень понятно, все замечания надо исправить, или это для дальнейшего. Короче я не очень понимаю как понять принята работа или нет, где это можно увидеть?

Посмотреть статус проверки, наверное, можно на платформе, через которую ты отправлял ссылку. Если что, на этот счет, ты можешь уточнить у наставника или наставницы. Там же я оставил замечания, которые нужно исправить. А здесь я их продублировал и отметил ⚠. Замечания с 🍏 носят рекомендательный характер

@maxshef

Copy link

Owner Author

maxshef commented May 16, 2023

Спасибо, увидел где!

maxshef added 3 commits

May 17, 2023 23:32

@maxshef


 И пусть теперь все получиться!_v1.1 промежуточная

@maxshef


 И пусть теперь все получиться!_v1.2 промежуточная

5acdece

@maxshef


 И пусть теперь все получиться!_v1.2 промежуточная

8a45e10

Labels

None yet

2 participants

@maxshef @ilshat-abdulin

Comments

Conversation

@maxshef maxshef commented May 13, 2023

Uh oh!

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

Uh oh!

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

Uh oh!

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

Uh oh!

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

Uh oh!

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

Uh oh!

@ilshat-abdulin ilshat-abdulin May 13, 2023

Choose a reason for hiding this comment

Uh oh!

maxshef commented May 15, 2023

Uh oh!

ilshat-abdulin commented May 16, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

maxshef commented May 16, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ilshat-abdulin commented May 16, 2023 •

edited

Loading