Skip to content

Сделал программу для определения победителя в гонках «24 часа Ле-Мана»!#1

Open
Hakushu21 wants to merge 3 commits intomainfrom
dev
Open

Сделал программу для определения победителя в гонках «24 часа Ле-Мана»!#1
Hakushu21 wants to merge 3 commits intomainfrom
dev

Conversation

@Hakushu21
Copy link
Copy Markdown
Owner

No description provided.

@Hakushu21
Copy link
Copy Markdown
Owner Author

Я создал этот Pull request для сдачи своей 1 проектной работы!

Comment on lines +4 to +5
private String name;
private 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, чтобы гарантировать, что значения полей не меняется после создания объекта

}

int speed = -1;
while (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.

Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода. Также по условию задачи скорость, равная 0 не допускается

@@ -1,6 +1,73 @@
import java.util.Scanner;

class Car{
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Классы лучше выделять в отдельные файлы, чтобы всё приложение не писать в одном файле

@Hakushu21
Copy link
Copy Markdown
Owner Author

Внес все изменения!

Также ввел изменения:
-нельзя вводить 0
-добавил константы мин и макс скоростей
-разделил по классам
@ArturNurtdinov
Copy link
Copy Markdown

ArturNurtdinov commented Jul 9, 2025

Внес все изменения!

Супер, молодец! В следующий раз присылай только ссылку на сам PR, а не на вкладку со всеми PRами, вот в таком виде - https://github.com/Hakushu21/Java-Module-Project-YP/pull/1 🚀

@Hakushu21
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