Skip to content

Проектная работа №1#1

Open
775hg4y2fd-glitch wants to merge 2 commits into
mainfrom
dev
Open

Проектная работа №1#1
775hg4y2fd-glitch wants to merge 2 commits into
mainfrom
dev

Conversation

@775hg4y2fd-glitch

Copy link
Copy Markdown
Owner

No description provided.

Comment thread src/main/java/Main.java
Race race = new Race();
for (int i = 1; i <= 3; i++) {
String name;
while (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.

Не рекомендую писать бесконечные циклы через while (true) - лучше всегда явно прописывать условие выхода из цикла, чтобы уменьшить вероятность ошибиться и повысить читабельность кода

Comment thread src/main/java/Main.java
while (true) {
System.out.println("Введите название машины №" + i + ":");
name = scanner.nextLine();
if (!name.isEmpty()) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать

Comment thread src/main/java/Main.java
}
System.out.println("Название не может быть пустым");
}
int speed;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Код для считывания скорости с ввода тоже лучше вынести в отдельную функцию

Comment thread src/main/java/Main.java
}
speed = scanner.nextInt();
scanner.nextLine();
if (speed > 0 && speed <= 250) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread src/main/java/Main.java
Comment on lines +41 to +42
String name;
int speed;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Поля лучше пометить final, тем самым исключив воможность их модификации извне

Comment thread src/main/java/Main.java
}
}

class Tachka { //

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Код лучше писать всегда на латинице с неймингами на английском, чтобы код читался всегда только на одном языке

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