Skip to content

Артём К. - #1

Open
ArtemPsy wants to merge 3 commits into
mainfrom
dev
Open

ArtemPsy wants to merge 3 commits into
mainfrom
dev

Conversation

@ArtemPsy

Copy link
Copy Markdown
Owner

Артём К.
Спринт 2

Comment thread src/main/java/Main.java
while (true) {
strIn = scanner.nextLine();

if (strIn.matches("^[0-9]+$")) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⏫ Интересный способ использовать регулярку для проверки на вводимое число. Такое регульрное выражение можно заменить на более лаконичное ^\\d+$

Comment thread src/main/java/Main.java

if (strIn.matches("^[0-9]+$")) {
numb = Integer.parseInt(strIn.trim());
if (numb <= 1)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⏫ Строчка с if немного съехала относительно остального кода внутри этого блока.

В IDEA есть удобный инструмент автоматического форматирования кода. Для этого достаточно нажать на горячие клавиши Ctrl+Alt+L (Command+Option+L)

System.out.println("Введите его стоимость в формате \"рубли.копейки\": ");
do {
String tmpStr = scanner.nextLine().trim();
if (tmpStr.matches("^[0-9]+\\.[0-9]{2}?$"))

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⏫ В данном случае мы слишком завязаны на формат "рубли.копейки". Было бы лучше добавить проверку вводимой строки через nextDouble. Но использование регулярного выражения - очень интересный способ)

private void finalCount() {
double sum = 0, res;

StringBuilder resStrB = new StringBuilder("Добавленные товары: \n");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍 Отличная идея использовать StringBuilder!

public class Calculator {
final private int numbPeople;
Scanner scanner = new Scanner(System.in);
final private ArrayList<Product> arrProducts = new ArrayList<>();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍 Хорошее применение списков!

@ArtemPsy

Copy link
Copy Markdown
Owner Author

Большое спасибо! )

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