Skip to content

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

Open
AmI2077 wants to merge 1 commit into
mainfrom
dev
Open

AmI2077 wants to merge 1 commit into
mainfrom
dev

Conversation

@AmI2077

@AmI2077 AmI2077 commented Aug 4, 2025

Copy link
Copy Markdown
Owner

No description provided.

Comment thread src/main/java/Car.java
Comment on lines +2 to +3
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/Car.java
Comment on lines +14 to +16
public void setName(String name) {
this.name = name;
}

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

public static void main(String[] args) {
System.out.println("Hello world!");
ArrayList<Car> cars = 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.

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

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

//хотел сделать так, чтобы название не состояло только из цифр, но если ввести число,
//которое не вмещает тип long, то проверка не срабатывает, хотелось бы понять как это обойти

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Числа, которые не вмещаются в формате Long, можно хранить в формате BigInteger или BigDecimal (если нужны с плавающей точкой)

Comment thread src/main/java/Main.java
private static boolean isNumeric(String str) {
try {
Long.parseLong(str);
return 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.

Здесь на самом деле из функции проще вернуть long, чтобы потом повторно не вызывать parseLong после проверки

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