Метод клонирования Java

Question

Метод клонирования Java

1

Я пытаюсь создать метод clone, который возьмет уже созданный Rational и сделает копию, чтобы в моем основном я мог сделать что-то вроде этого:

Rational r3 = new Rational(r1);

До сих пор я создал метод в моем классе Rational, который будет принимать рациональное значение как параметр; однако я не уверен, как сделать новый экземпляр похожим на параметр.

public Rational copy(Rational aRational) {
int newNum = aRational.n;
 int newDenom = aRational.d;
 return (new Rational(newNum, newDenom));

}

Любая помощь или понимание очень ценятся!

Bhetzie 21 апр. 2014, в 23:14

Источник

1

Это выглядит очень хорошо для меня. Что-то не работает?
Keppil 21 апр. 2014, в 21:15
0

«Я не уверен, как сделать новый экземпляр похожим на параметр» <- а?
fge 21 апр. 2014, в 21:17
0

Вы называете copy(r1) или new Rational(r1) ?
Mike 21 апр. 2014, в 21:17
0

Ваш метод копирования не использует это. Либо сделайте его статичным, либо удалите параметр и сделайте копию этого.
Maurice Perry 21 апр. 2014, в 21:23

Показать ещё 2 комментария

Теги:

java

class

copy-constructor

rational-numbers

1 ответ

Ещё вопросы

Это выглядит очень хорошо для меня. Что-то не работает?
«Я не уверен, как сделать новый экземпляр похожим на параметр» <- а?
Вы называете copy(r1) или new Rational(r1) ?
Ваш метод копирования не использует это. Либо сделайте его статичным, либо удалите параметр и сделайте копию этого.

dasblinkenlight · Accepted Answer · 2014-04-21T18-42-00.000Z

в моем основном я [хочу] сделать что-то вроде этого: Rational r3 = new Rational(r1);

Это не клонирование, это делает конструктор, который принимает Rational как параметр. Вы можете сделать это следующим образом:

public Rational(Rational other) {
    n = other.n;
    d = other.d;
}

Это позволит использовать код для создания новых экземпляров Rational из существующих экземпляров.

Примечание. Похоже, что n и d в вашем классе подвергаются пользователям вашего класса как общедоступные переменные. Это не идеально, потому что у вас нет контроля над изменениями, которые пользователи вашего класса могут выполнять с его переменными экземпляра. Лучший подход заключается в том, чтобы сделать их private, и добавить геттеры и сеттеры в ваш класс. Такой подход позволит вам обнаружить попытки установить знаменатель на ноль, что может быть желательным.

Этот код работает, даже если n и d являются частными.
@ Абсурд-Разум Абсолютно, это так. Моя точка зрения заключалась в том, что OP не должен делать n и d приватными, чтобы контролировать процесс их установки, а не делать его конструктором. Я отредактировал ответ, чтобы сделать эту заметку отдельным пунктом.
Да, вы также можете использовать другой конструктор: this (other.n, other.d);