Skip to content

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

Open
Hakushu21 wants to merge 3 commits into
mainfrom
dev
Open

Hakushu21 wants to merge 3 commits into
mainfrom
dev

Conversation

@Hakushu21

Copy link
Copy Markdown
Owner

No description provided.

@Hakushu21

Copy link
Copy Markdown
Owner Author

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

Comment thread src/main/java/Main.java Outdated
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, чтобы гарантировать, что значения полей не меняется после создания объекта

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

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 не допускается

Comment thread src/main/java/Main.java Outdated
@@ -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

ArturNurtdinov commented Jul 9, 2025 •

Copy link
Copy Markdown

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

Супер, молодец! В следующий раз присылай только ссылку на сам 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