VPF::Метод clone() и "лишние телодвижения"

agR

Дата 6.12.2008, 18:28 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 140
Регистрация: 19.7.2006
Где: Киев

Репутация: 2
Всего: 2

Вот стало интересно.... изменилось ли поведение метода clone() в JDK 1.6 по сравнению с 1.5.
Поведение clone() ver 1.5 когда-то описал LSD
Решил посмотреть как будет себя вести код в 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 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 184
Регистрация: 29.9.2006

Репутация: нет
Всего: 3

А что не так?

Выводит:

Цитата

--- 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 создается впечатление что дип клонирование происходит автоматически потому что они неизменяемые.

Это сообщение отредактировал(а) _sten_ - 7.12.2008, 15:57

--------------------

WARNING: Condition 'to_be || !to_be' is always 'true' !

agR

Дата 7.12.2008, 17:04 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 140
Регистрация: 19.7.2006
Где: Киев

Репутация: 2
Всего: 2

Спасибо.
Вот этот момент:

Код


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

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

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Java: Общие вопросы \| Следующая тема »