Comments

В файлах код прыгает, отступы не одинаковые - рекомендую использовать встроенные форматтер. Если ты программируешь в IDE от JetBrains, то можешь пройтись по всем файлам и нажать комбинацию (Windows: Shift + Ctrl + Alt + L, MacOS: Shift + Option + Command + L), затем выбрать, какую часть кода хочешь изменить, и выполнить команду. Среда разработки автоматически сделает нужные отступы и переносы (как она это делает можно изменить в настройках, но дефолтные значения вполне нормальные).

src/main/java/Main.java Outdated

double totalAmount = 0.0;

while (true) { // не стал делать проверку по введенному значению. Она есть дальше при делении счета.

System.out.println("Введите название и цену товара! Если хотите закончить, то введите 'завершить'!");

nameOfProduct = scanner.next();

Copy link

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

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

⚠️ Лучше использовать nextLine, чтобы считать всю строку, потому что если ввести название из нескольких слов, то приложение упадет, так как после прочтения первого слова пойдет читать цену товара

src/main/java/Main.java

System.out.println(String.format(fullListsCost, totalAmount) + calc.GetRubleAddition(priceOfProduct));

break;

} else {

priceOfProduct = scanner.nextDouble();

Copy link

@SeveNChaK SeveNChaK Mar 15, 2023

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

while (true) {

while (!scanner.hasNextInt()) {

System.out.println("Не является числом, повторите ввод!");

String numberOfPersons1 = scanner.next();

Copy link

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

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

Можно просто scanner.next(); без записи в переменную

src/main/java/Main.java Outdated

System.out.println("Не является числом, повторите ввод!");

String numberOfPersons1 = scanner.next();

}

while (scanner.hasNextInt()) {

Copy link

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

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

Этот цикл не нужен, мы уже сделали проверку в предыдущем цикле, так что тут точно будет int

src/main/java/Main.java Outdated

numberOfPersons = scanner.nextInt();

if (numberOfPersons <= 1) {

System.out.println("Количество персон должно быть больше 1. Повторите ввод!");

} else if (numberOfPersons > 1) {

Copy link

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

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

Можно просто } else {

src/main/java/Calculator.java Outdated

@@ -0,0 +1,28 @@

public class Calculator {

public String GetRubleAddition(double num) {

Copy link

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

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

Методы принято называть с маленькой буквы

@V3hby


 Add dev branch ver2

02fcb7a

SeveNChaK

SeveNChaK reviewed

Mar 15, 2023

View reviewed changes

src/main/java/Main.java Outdated

System.out.println("Введите название и цену товара! Если хотите закончить, то введите 'завершить'!");

nameOfProduct = scanner.next();

Calculator calc = new Calculator();

{

Copy link

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

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

Похоже на лишнюю скобочку. Ты получается просто весь код еще дополнительно в блок обернул, в этом нет необходимости. Кажется, что можно убрать ее и соответствующую ей закрывающую скобку

V3hby added 2 commits

March 16, 2023 08:13

@V3hby


 Add dev branch ver2

6cd547a

@V3hby


 Add changes dev branch ver2

3748fe8

Labels

None yet

2 participants

@V3hby @SeveNChaK

Comments

Conversation

@V3hby V3hby commented Mar 15, 2023

Uh oh!

@SeveNChaK SeveNChaK left a comment

Choose a reason for hiding this comment

Uh oh!

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

Uh oh!

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

Uh oh!

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

Uh oh!

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

Uh oh!

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

Uh oh!

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

Uh oh!

@SeveNChaK SeveNChaK Mar 15, 2023

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants