Skip to content

First request on this 2nd sprint's project#1

Open
looee1q wants to merge 2 commits intomainfrom
dev
Open

First request on this 2nd sprint's project#1
looee1q wants to merge 2 commits intomainfrom
dev

Conversation

@looee1q
Copy link
Copy Markdown
Owner

@looee1q looee1q commented Feb 11, 2023

First request on this 2nd sprint's project

// вы не должны ограничиваться только классом Main и можете создавать свои классы по необходимости
System.out.println("Привет Мир");
//Первая часть задачи (ввод количества человек):
System.out.println("На скольких человек необходимо разделить счёт?");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Лучше внести этот принтлн в метод correctedPersonsNumber, потому что это часть той логики

System.out.printf("С каждого %.2f %s!%n", calculator.totalPrice / personsNumber, rubInCorrectCase(calculator.totalPrice / personsNumber));
}

public static int correctedPersonsNumber() {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

В Java принята конвенция, согласно которой методы принято называть глаголами, см. https://www.oracle.com/java/technologies/javase/codeconventions-namingconventions.html

}

public static String rubInCorrectCase(double sumOfEachPerson) {
int a = (int) sumOfEachPerson;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Лучше давать переменным более говорящие названия

listOfGoods += nameOfItem + " " + String.format("%.2f", item.priceOfItem) + "\n";
totalPrice += priceOfItem;
System.out.println("Товар успешно добавлен! Вы хотите добавить еще товар?");
System.out.println("Введите команду \"Завершить\" для того, чтоб завершить процесс добавления товаров.");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Такие подряд идущие println можно объединить в один.

}
return true;
}
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Круто, что сам реализовал логику для проверки ввода, дело полезное. Также можно использовать готовые решения, например, методы Scanner.nextInt(), nextDouble()

//Класс для проверки является ли введенный символ (символы) неотрицательным целым или вещественным числом.
public class IsItANumber {

public boolean naturalNum(String str) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

В этой программе уже было использовано ключевое слово static, можно также сделать эти методы тоже static, так как они служебные. Тогда нам не нужно будет создавать экземпляр класса IsItANumber для вызова его методов

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants