clone()

[Kassel]

Hi Community,

habe ein Problem bei einem JUnit Test .. immer rot:

TEST:

@Test
	public void testMessageSimpleClone()
	{
		Message m = new Message();
		m.setFrom("Alice");
		m.setTo("Bob");
		m.setContent("Hallo, wie gehts?");
		
		Message copy = (Message) m.clone();
		
	
		assertNotNull(copy);
		assertEquals(m, copy); -> FEHLER
		assertNotSame(m, copy);
		
		assertEquals(m.getFrom(), copy.getFrom());
		assertEquals(m.getTo(), copy.getTo());
		assertEquals(m.getContent(), copy.getContent());
	}

MESSAGE:

public class Message implements Cloneable
{

	private String from;
	private String to;
	private String content;
	
	private Message message;
	
	private Collection<Attachment> attachments = new HashSet<Attachment>();
....
getter setter ...

public Object clone(){
//		try{
//		Message cloned=(Message)super.clone();
//		cloned.content=getContent();
//		cloned.from=getFrom();
//		cloned.to=getTo();
//		return cloned;
//		}
//		catch(CloneNotSupportedException e){
//			  System.out.println(e);
//			  return null;
//			  }
		try {
			return super.clone();
		} catch (CloneNotSupportedException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
			return null;
		}
	}

Was ist bei meiner clone zu beachten damit mein Test nicht fehlschlägt ?

 

[faetzminator]

Du musst [c]clone()[/c] selbst implementieren. Zuerst ein neues Objekt erzeugen und dann alle nicht primitiven, mutable Felder mit [c]clone()[/c] rüberkopieren. Die primitiven (int, ...) und immutable Felder (String, ...) können einfach rüberkopiert werden.

 

[Kassel]

Du meinst so was ?

public Message clone(){
                      Message cloned=(Message)super.clone();
		cloned.content=getContent();
		cloned.from=getFrom();
		cloned.to=getTo();
		return cloned;
}

Dies klappt jedoch nicht ...

 

[faetzminator]

Nein eben nicht [c]super.clone()[/c] sondern [c]new Message()[/c]. Oder was verstehst du sonst unter Zuerst ein neues Objekt erzeugen ?

 

[Kassel]

Sry ... stimmt so - jedoch ist mein Test immer noch nciht grün da hier ein Fehler auftritt:

assertEquals(m, copy);

Jetzt frage ich mich warum dass so ist ... alle Attribute des einen Objektes stimmen mit den Attributen des anderen Objektes überein ... komisch

 

[SlaterB]

clone ist super, egal in welcher Form, aber überlege mal auf welche Weise assertEquals(m, copy); die Gleichheit feststellt,
ergeben die Buchstaben e, q, u, a, l, s in irgendeiner Kombination etwas was du kennst?

@faetzminator
super.clone() geht! zumindest technisch, ob so gedacht ist eine andere Frage

> dann alle nicht primitiven, mutable Felder mit clone() rüberkopieren.
warum nicht dieselben nehmen? ob so oder so kann man ganz nach belieben entscheiden, beides macht Sinn

 

[njans]

Wennn du willst, dass die Objekte gleich sind, dann solltest du Equals und hashCode überschreiben

 

[Kassel]

Danke Community,

jetzt habe ich es auch kapiert ... wie schon SlaterB sagte ... mit

super.clone();

geht es auch ...
ich musste halt sicherstellen dass:

assertEquals(m, copy);
assertNotSame(m, copy);

zwei Objekte gleich sind und identisch ( equals ), jedoch nicht die selben Objekte sind also kein

Object obj1=obj2;

...
dies hat natürlich geklappt indem ich den Rat von njans befolgt habe in dem ich die equals Methode überschrieben habe - z.B so :

public boolean equals( Object obj )
	{
		boolean flag = false;
		Message emp = ( Message )obj;
		if( emp.from == getFrom() )
			flag = true;
		return flag;
	}

Jetzt fuktioniert es auch ... Danke

 

[faetzminator]

@faetzminator
super.clone() geht! zumindest technisch, ob so gedacht ist eine andere Frage

Natürlich gehts, aber seine Klasse erbt von Object, welches AFAIK einfach eine Exception wirft. Wenn sich also der Supertyp nicht darum bemüht, etwas zu klonen, dann muss mans selber machen

 

[SlaterB]

nein, das clont wirklich

 

[faetzminator]

nein, das clont wirklich

Wer wo wie was?

The class [c]Object[/c] does not itself implement the interface [c]Cloneable[/c], so calling the [c]clone[/c] method on an object whose class is [c]Object[/c] will result in throwing an exception at run time.

 

[SlaterB]

tja, hängt vom Interface ab, ich habe nicht gesagt dass es schön ist, nur dass es funktioniert
Java Tips - How to implement Cloneable Interface

----

und zur equals-Methode: String ebenfalls mit equals vergleichen, nicht mit == !

 

[faetzminator]

Hab nochmal in der Javadoc gelesen - und ich muss dir hier wohl recht geben! Allerdings erstellt es keine Deep Copy - was wohl der Grund dafür ist, dass ich das nicht kannte. Ich hab bis jetzt immer nur Deep Copy benötigt.