Skip to content

PR практика 1#1

Open
ElenaPupysheva wants to merge 5 commits intodevfrom
master
Open

PR практика 1#1
ElenaPupysheva wants to merge 5 commits intodevfrom
master

Conversation

@ElenaPupysheva
Copy link
Copy Markdown
Owner

Добавлено решение первой практической работы.

Comment on lines +2 to +3
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. С помощью final отмечаются переменные, которые инициализируются только один раз.

int speed;
while (true) {
speed = scanner.nextInt();
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.

🍏 максимальную и минимальную скорость можно вынести в константы

}
}
System.out.println("Гонка завершена!");
for (Car car : carList) System.out.println("В гонке участвовали: "+car.name+" со скоростью " +car.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.

🍏 лучше все-таки открыть фигурные скобки и написать System.out.println на следующей строчке, так код будет более читаем

Comment on lines +5 to +6
// ваш код начнется здесь
// вы не должны ограничиваться только классом Main и можете создавать свои классы по необходимости
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🍏 ненужные комментарии можно смело удалять)

@estetika-panelek
Copy link
Copy Markdown

Оставлю на будущее гайд по работе с ветками:

Снимок экрана 2024-06-30 в 12 32 08

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