Как создать клон объекта в Java?

Question

[Александр]

Нашел такое решение: stackoverflow.com/questions/869033/how-do-i-copy-a...

Оно не подошло. Не знаю как и почему, но все переменные в новом объекте становятся ссылками на переменные старого. Т.е. в нужном месте я создал копию объекта:

MyObject oldObject = new MyObject();
oldObject.someField = 1;
MyObject newObject = new MyObject(oldObject);
oldObject.someField = 2;

и переменная someField у объекта newObject тоже стала равняться 2.

Пробовал написать мегакостыль: в классе, который надо будет копировать, создал функцию clone() и присвоил в ней всем переменным значения из существующего объекта:

public Pot clone() {
        Pot newPot = new Pot();

        Field[] fields = this.getClass().getDeclaredFields();
        for (Field declaredField : fields) {
            try {
                if (declaredField.get(this) == null) {
                    newPot.setValue(declaredField.getName(), null);
                } else {
                    newPot.setValue(declaredField.getName(), declaredField.get(this));
                }
            } catch (IllegalArgumentException | IllegalAccessException ex) {
                Logger.getLogger(PotType.class.getName()).log(Level.SEVERE, null, ex);
            }
        }

        return newPot;
    }

    public boolean setValue(String key, Object value) {
        try {
            if (!key.equals("initP")) {
                Field declaredField = this.getClass().getField(key);
                declaredField.set(this, value);
            }
        } catch (IllegalArgumentException | IllegalAccessException ex) {
            Logger.getLogger(PotType.class.getName()).log(Level.SEVERE, null, ex);

            return false;
        } catch (NoSuchFieldException | SecurityException ex) {
            Logger.getLogger(PotType.class.getName()).log(Level.SEVERE, null, ex);

            return false;
        }
        return true;
    }

MyObject oldObject = new MyObject();
oldObject.someField = 1;
MyObject newObject = oldObject.clone();
oldObject.someField = 2;

И опять все тоже самое! someField в новом объекте стало равно двум. Что за чертовщина? :) Прошу помощи.

Answer 1

[DR_Demons]

В Java для клонирования объектов используется интерфейс Cloneable, он Вам не подходит?

example

package my.cloneable;

class User implements Cloneable {
       private String name;
       private int age;
      
       public String getName() {
             return name;
       }
       public void setName(String name) {
             this.name = name;
       }
       public int getAge() {
             return age;
       }
       public void setAge(int age) {
             this.age = age;
       }
      
       public User clone() throws CloneNotSupportedException {
             return (User)super.clone();
       }
}

public class App {

       public static void main(String[] args) {
            
             User user = new User();
            
             user.setName("Иванов");
             user.setAge(25);
            
             System.out.println("Данные до клонирования: " +
user.getName() + " - " + user.getAge() + "лет");
            
             User clone;
             try {
                    clone = user.clone();
                    clone.setName("Петров");
                    clone.setAge(30);
                    System.out.println("Клон после изменения данные: " +
                           clone.getName() + " - " + clone.getAge() + "лет");
                   
             } catch (CloneNotSupportedException e) {
                    System.out.println("Объект не может быть клонированным.");
             }
            
            
             System.out.println("Оригинал, после манипуляций с клоном: " +
 user.getName() + " - " + user.getAge() + "лет");
       }
}

/* результат:
Данные до клонирования: Иванов - 25лет
Клон после изменения данные: Петров - 30лет
Оригинал, после манипуляций с клоном: Иванов - 25лет
 */

Comments

[Александр]

Через него тоже пробовал. Та же самая канитель! В комменте ниже я написал почему, как мне кажется, ни один из этих методов не подошел. По факту они все рабочие, пока дело касается примитивных типов.

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

[DR_Demons]

Александр: не знаю что происходит в Вашем коде, я добавил в объект который нужно клонировать не примитивный тип и он так же клонировался, сохранил состояние.

[DR_Demons]

Александр:

changed code

package my.cloneable;

public class SomeClass {
	
	public SomeClass(String name) {
		this.name = name;
	}
	
	String name;

}

class User implements Cloneable {
    private String name;
    private int age;
    private SomeClass custom;
   
    public String getName() {
          return name;
    }
    public void setName(String name) {
          this.name = name;
    }
    public int getAge() {
          return age;
    }
    public void setAge(int age) {
          this.age = age;
    }
   
    public User clone() throws CloneNotSupportedException {
          return (User)super.clone();
    }
	public SomeClass getCustom() {
		return custom;
	}
	public void setCustom(SomeClass custom) {
		this.custom = custom;
	}
}

public class App {

       public static void main(String[] args) {
            
             User user = new User();
            
             user.setName("Иванов");
             user.setAge(25);
             user.setCustom(new SomeClass("Иван"));
            
             System.out.println("Данные до клонирования: " +
user.getName() + " " + user.getCustom().name + " - " + user.getAge() + "лет");
            
             User clone;
             try {
                    clone = user.clone();
                    System.out.println("Клон : " +
                            clone.getName() + " " + user.getCustom().name + " - " + clone.getAge() + "лет");
                    clone.setName("Петров");
                    clone.setAge(30);
                    clone.setCustom(new SomeClass("Степан"));
                    System.out.println("Клон после изменения данные: " +
                           clone.getName() + " " + clone.getCustom().name + " - " + clone.getAge() + "лет");
                   
             } catch (CloneNotSupportedException e) {
                    System.out.println("Объект не может быть клонированным.");
             }
            
            
             System.out.println("Оригинал, после манипуляций с клоном: " +
 user.getName() + " " + user.getCustom().name + " - " + user.getAge() + "лет");
       }
}

/* результат:
Данные до клонирования: Иванов Иван - 25лет
Клон : Иванов Иван - 25лет
Клон после изменения данные: Петров Степан - 30лет
Оригинал, после манипуляций с клоном: Иванов Иван - 25лет
 */

[Николай Павлов]

Александр: вы внутри clone делает setValue на ссылочный тип => ссылки у вас в двух обьектах одинаковые. В таком случае нужно клонировать делать deep clone. Если вам не важна критично скорость и объект сериализуемый, то используйте к примеру commons.apache.org/proper/commons-lang/javadocs/ap..., насколько я помню там будет сделать deep clone. Но если вы хотите быстрый, клоне вам нужно сделать примерно так

User clone(){
User temp = (User)super.clone();
temp.setValue(value.clone());
return temp;
}

[Александр]

DR_Demons: эх, даже не знаю как бы так выдрать кусок кода, который не клонируется... наклепал в нетбинсе проект примерно с такой же структурой, как в рабочем проекте. В нем снова переменные переприсваиваются =) wxmaper.ru/temp/JavaApplication9.zip (pastebin.com/8bjcSdsc)

В примере создается 2 клона (сначала думал проблема в том, что я клонированный объект хранил в клонируемом объекте). Затем по таймеру в самый первый объект заносятся изменения, и они тут же оказываются в обоих клонированных объектах....

[Александр]

Может я что-то не учел? Может у меня уже перегруз мозга случился и я не замечаю элементарной ошибки? =)

Answer 2

[Владимир Смирнов]

У вас 2 варианта:

1. Использовать какую-либо библиотеку для глубокого клонирования. Плюс в гораздо меньшем количестве бездумного кодинга. Минус в падении производительности и повышенном потреблении ресурсов. Если количество операций клонирования в приложении менее сотни в секунду, то можно не беспокоится по поводу производительности.
   
2. Писать всё клонирование самому, поле за полем. Плюс в высокой производительности - ни одна универсальная библиотека не сравнится в скорости с таким подходом. Минус в массе говнокода, который еще и поддерживать надо (при любом изменении класса не забывать дорабатывать методы clone())
   

Comments

[Александр]

Первый метод в принципе бы устроил, т.к. автоматом мне нужно создать всего одного клона, один раз :) в последующем новые клоны будут создаваться только действиями пользователя. А это даже не 2 раза в секунду.

О втором методе я тоже думал, но сразу же перекрестился и отбросил этот вариант.

Пока все-таки оставлю свой костыль, который просто создает 2 одинаковых объекта одновременно и выставляет у них одинаковые параметры до определенного момента... это, конечно, не совсем удобно, т.к. пришлось дублировать код в некоторых местах (не функции, слава богу, а только их вызовы :)) Но за то быстро и просто в реализации ^_^