Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Java: Общие вопросы > Метод clone() и "лишние телодвижения"


Автор: agR 6.12.2008, 18:28
Вот стало интересно.... изменилось ли поведение метода clone() в JDK 1.6 по сравнению с 1.5. 
http://forum.vingrad.ru/index.php?showtopic=175052&view=findpost&p=1278412
Решил посмотреть как будет себя вести код в 1.6:

Код

import java.util.Date;
import java.util.GregorianCalendar;


public class DeepClone {
    public static void main(String[] args) {
        try {
            Birthday b1 = new Birthday("Alex");
            b1.setBirtday(1985, 10, 22);
            Birthday b2 = b1.clone();
            System.out.println("--- Before change ---");
            System.out.println("b1: "+b1);
            System.out.println("b2: "+b2);
            b2.setBirtday(1985,12,3);
            b2.setName("Bob");
            System.out.println("--- After change ---");
            System.out.println("b1: "+b1);
            System.out.println("b2: "+b2);
            }
            catch(CloneNotSupportedException e) {
                e.printStackTrace();
            }
    }
}
class Birthday implements Cloneable {
    public Birthday(String name) {
        this.name = name;
    }
    public Date getBirtdayDate() {
        return birthday;
    }
    public void setBirtday(int year, int month, int day) {
        birthday = new GregorianCalendar(year,month-1,day).getTime();
    }
    public String getName() {
        return name;
    }
    public void setName(String name) {
        this.name = name;
    }
    
    @Override
    public Birthday clone() throws CloneNotSupportedException {
        Birthday cloned = (Birthday)super.clone();

        //cloned.birthday = (Date)birthday.clone(); ДАННАЯ СТРОКА НЕ ДАЕТ НИКАКОГО ЭФФЕКТА

        return cloned;

    }
    @Override
    public String toString() {
        return "[name="+name
                +",birthday="+ birthday
                +"]";
    }
    private Date birthday;
    private String name;
}


Так вот, казалось бы для "глубокого" клонирования закомментированная строка необходима, но она ровным счетом ничего не делает - результат не меняется.
Может теперь при работе с методом clone() уже не надо "лишних телодвижений"? Или это я что-то не доглядел?  

Автор: _sten_ 7.12.2008, 15:55
А что не так?

Выводит:
Цитата

--- Before change ---
b1: [name=Alex,birthday=Tue Oct 22 00:00:00 EET 1985]
b2: [name=Alex,birthday=Tue Oct 22 00:00:00 EET 1985]
--- After change ---
b1: [name=Alex,birthday=Tue Oct 22 00:00:00 EET 1985]
b2: [name=Bob,birthday=Tue Dec 03 00:00:00 EET 


я так понимаю тебя смущает то что не изменилась дата в b1 After change ???

Так все правильно. После вызова метода clone() проперти birthday класса Birthday для b2 ссылается на тот же объект что и b1(как и описано в спеке). 
Ты вызвал:
Код

   b2.setBirtday(1985, 12, 3);

который выпонил метод:
Код

public void setBirtday(int year, int month, int day) {
        birthday = new GregorianCalendar(year, month - 1, day).getTime();
    }


то есть создал новый экземпляр класса Date и теперь проперти birthday ссылается на него.

Если бы ты сделал так:
Код

        b2.getBirtdayDate().setTime(0L);


то тогда  дата в b1  тоже изменилась.

Метод клон создает новый объект, и делает так что бы все его проперти ссылались на те же объекты. Для дип клонирования закоменченая строка нужна и от "лишних телодвижений" никуда не дется. Проста для классов Integer, String создается впечатление что дип клонирование происходит автоматически потому что они неизменяемые.

Автор: agR 7.12.2008, 17:04
Спасибо.
Вот этот момент:
Код

public void setBirtday(int year, int month, int day) {
        birthday = new GregorianCalendar(year, month - 1, day).getTime();
    }

не учел. В след. раз буду внимательней smile

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)