Cloneable Probleme

[Otz]

hallo

ich versuche gerade ein java objekt zu kopieren. nach einigen vergeblichen versuchen bin ich auf das interface cloneable gestoßen.


hier mein codefragment:


ich habe in dem objekt, welches ich clonen möchte eine clone methode gebaut.
[Java]
class MeinObjekt implements Cloneable{
....

public MeinObjekt clone1(){
MeinObjekt objektname= null;
try {
objektname= (MeinObjekt ) this.clone();
} catch (CloneNotSupportedException e) {
e.printStackTrace();
}
return objektname;
}
}
[/Java]


diese Methode rufe ich aus einer anderen klasse auf:

[Java]
this.tempcopy=MeinObject.clone1();
[/Java]

Jedoch kopiert er das objekt nicht sonder er verweist nur darauf ;(


wo hat sich mein fehler versteckt? vielen dank

 

[SlaterB]

funktioniert bestens, siehe Test unten,

Paradebeispiel für unvollständige Frage, ohne Testprogramm, ohne Erklärung für 'Jedoch kopiert er das objekt nicht sonder ein verweist nur darauf'

innere referenzierte Objekte werden natürlich übernommen, nicht selber auch noch kopiert,
dafür gibts z.B.
http://www.informatik-student.de/2006/12/10/deep-copy-von-java-objekten/

public class Test2
{
    public static void main(String[] args)
    {
        MeinObjekt a = new MeinObjekt();
        a.k = 5;
        MeinObjekt b = a.clone1();
        a.k = 6;
        System.out.println("a: " + a.k + ", b: " + b.k);
    }
}

class MeinObjekt   implements Cloneable {
    public int k;

    public MeinObjekt clone1()
    {
        MeinObjekt objektname = null;
        try
        {
            objektname = (MeinObjekt)this.clone();
        }
        catch (CloneNotSupportedException e)
        {
            e.printStackTrace();
        }
        return objektname;
    }

}

 

[Otz]

ahhhh ich verstehe das Problem. Ok...

wenn ich das Objekt clone, erzeugt er mir zwar ein "neues" aber die verweise im "inneren" des objektes zeigen immer noch auf die gleichen objekte wie beim ursprünlichen.

ist bischen wie bei c++ mit den zeigern oder?!

 

[SlaterB]

da habe ich nichts mehr beizutragen

 

[Otz]

da habe ich nichts mehr beizutragen

ja bedeutet es du stimmst mir zu oder ist es totaler müll?!


ich verstehe jetzt den Fehler jedoch bekomme ich das Problem leider noch nicht gelöst. ich habe es mit der copy/clone methode probiert aber leider funktioniert es immer noch nicht. es ist genau das selbe fehler bild.


wenn ich im geclonten objekt ein attribut ändere, wird dies im ursprünglichen objekt geändert. daraus lässt sich schliesen, dass keine kopie erzeugt wurde sondern eher eine referenz odeR?!

 

[SlaterB]

zu 'c++ mit den zeigern' kann ich nichts sagen, habe zu wenig c++,
kann aber vermuten dass das selbst dann zu ungenau formuliert wäre, da kann man beliebiges antworten

'funktioniert es immer noch nicht'
was hatte ich noch geschrieben? kein (kompletter) Code, keine Erklärungen im ersten Posting
nun dasselbe..,

ich kann jetzt wieder ein Beispiel zusammenzimmern, verzichte aber doch lieber drauf

den Kopiercode kopieren, die Klasse mit Attributen erweitern, Testmethode usw.,
'schlimmstenfalls' stellst du selber fest, dass es im Standard funktioniert, in deinem Programm aber nicht,

das kann noch kompliziert zu suchen sein, durchaus, aber die Funktionalität an sich im Forum anzuzweifeln hilft dabei ja wenig

 

[Otz]

bin leider immernoch nicht zum gewünschten ergebnis bekommen

ich versuche nun nochmal das problem zu schildern, diesmal mit etwas mehr code

Also Object1 soll kopiert/geclont werden:

[Java]
public class object1{

private CheckBox check = null;
private String name; //only uppercase letters

...(getter/setter)
[/Java]

Object1 befindet sich in einem Container(VaadinFramework), dies sollte aber nichts mit dem Problem zu tun haben
[Java]
class Object1Container....

public Object1Container deepContainerCopy() {

Object1Container cloneContainer = new Object1Container ();

if(cloneContainer.size()!=0) {
cloneContainer.removeAllItems();
}
//Container ist leer!

Object1 runner = new Object1();
runner = (Object1 ) this.firstItemId();

for(int i = 1; i<=this.size();i++){
Object1 cloneType = new Object1 ();
CheckBox check = new CheckBox();
check.setValue(runner.getCheck().getValue());
check.setCaption(runner.getCheck().getCaption());
cloneType.setCheck(check);
cloneType.setNameUpperCase(runner.getNameUpperCase());
cloneContainer.addItem(cloneType);

runner=this.nextItemId(runner);
}
return cloneContainer;
}
[/Java]

[Java]
Object1Container container = new Object1Container();
Object1Container cloneContainer= container.deepContainerCopy();

[/Java]

so, ich hoffe jetzt kann man es etwas besser verstehen und ihr könnt mir bei meinem fehler weiterhelfen

 

[Crian]

Ich habe für so etwas Copy-Konstruktoren. Wie man das letztlich löst, ist natürlich Geschmackssache. Wichtig ist, dass du alle tieferen Datenstrukturen eben wieder kopieren musst. Daher haben bei mir alle der dort vorkommenden Klasen auch wieder Copy-Konstruktoren, die sich dann wiederum darum kümmern, den Inhalt zu verdoppeln und nicht nur Verweise zu kopieren.

 

[SlaterB]

das ist vollkommen anderer Code als in meinem Link, wie sollte je irgendjemand dazu etwas geahnt haben können?

auf mich macht das einen zuverlässigen andere-Objekte-Eindruck,
was genau siehst du denn nun als Problem an?

Object1 ausgangsobject = new Object1();
 
Object1 cloneobject=ausgangsobject.deepContainerCopy();

gibt es die Methode deepContainerCopy() in Object1 oder in Object1Container, oder besteht Vererbung?
bisschen verwirrend,
kommt überhaupt die gepostete Methode dran, gibt es innere Checkboxen usw?
wenn du nur ausgangsobject in einer Zeile erstellst, ist ja nicht so viel Inhalt zu vermuten,

wie auch immer, jetzt wo so viele Details da sind, wo genau liegt denn der vermutete Fehler?
cloneobject wird ja wohl != ausgangsobject sein, oder dort schon ein Problem?
wenn Container, wie sieht es mit den inneren Objekten aus, wo tritt Gleichheit auf, wo genau letzlich ein Problem?

> wenn ich im geclonten objekt ein attribut ändere, wird dies im ursprünglichen objekt geändert.

was genau bedeutet das hier, Value in CheckBox, Name in Object1, oder oder oder?
sind auch keine statischen Attribute betroffen die sich natürlich für alle gleich auswirken?

nur (noch mehr) Details helfen

am besten immer ein vollständiges Testprogramm, siehe mein Beispiel,
main-Methode welche Objekte erstellt, kopiert, evtl. Werte ändert, vergleicht, ALLES dabei,
wenn mit mehreren Unterklassen, dann sicher bisschen schwieriger zu machen, aber hast ja auch alle Zeit der Welt

 

[Otz]

hab mal versucht deine fragen zu beantworten

gibt es die Methode deepContainerCopy() in Object1 oder in Object1Container, oder besteht Vererbung?

ja, sorry war ein fehler von mir. wurde bereits geändert. ist natürlich im container.

kommt überhaupt die gepostete Methode dran, gibt es innere Checkboxen usw?
wenn du nur ausgangsobject in einer Zeile erstellst, ist ja nicht so viel Inhalt zu vermuten,
was meinst du mit innere checkboxen? ob das object ausgansobject checkboxen besitz? ja, es besitz checkbox objecte

wie auch immer, jetzt wo so viele Details da sind, wo genau liegt denn der vermutete Fehler?
naja, wenn ich das wüsste. ich seh nur, dass es nicht das macht was es soll

cloneobject wird ja wohl != ausgangsobject sein, oder dort schon ein Problem?
ne, da stimmts noch. also cloneobject!=ausgangsobject=true

> wenn ich im geclonten objekt ein attribut ändere, wird dies im ursprünglichen objekt geändert.
nein

was genau bedeutet das hier, Value in CheckBox, Name in Object1, oder oder oder?
wie gesagt, dass sind eigenschaften von vaadin. value ist ob die checkbox gecheckt oder nicht ist... sollte ja aber nichts mit dem fehler zu tun haben oder?!

nochmal zu meinem problem:

ich hab ne gui in der der inhalt des containers angezeigt werden soll. diese objekte können auch geändert werden. z.b. checkbox checken oder unchecken.

dann gibt es 2 buttons. ok und cancel. ok speichert die änderungen im container und cancel stellt wieder den ausgangszusand her.
deshalb dachte ich mir, wenn ich das fenster erstelle clone ich im konstruktor des fensters den container und wenn anstatt der ok button der cancel button gedrückt wird "lade" ich den im konstruktor geclonten container.

 

[SlaterB]

'vaadin' lese ich jetzt quasi zum ersten Mal, ok früher einmal kurz erwähnt,
keine Ahnung was mit diesen Checkboxen los ist,
vielleicht werden alle die den gleichen Typ/ Value/ Caption haben, was immer das ist, im Hintergrund auf gleiche Weise geändert,
komische Framework-Objekte bieten sich nicht für solche Tests an,
kopiere eigene Dummy-Klassen, Listen, Arrays usw. und erfahre, dass Kopien in der Tat Kopien sind,

was mit deinem Framework ist, kann ich nicht sagen,
GUI-Elemente bieten sich normalerweise auch nicht für Kopien an, z.B. Swing-Buttons auch nicht

wenn dann kopiere nur die Konfigurations-Daten an sich, String name, Position x/y, wiederum in einfachen Java-Objekten,
und lasse daraus auf strukturierten Wege GUI-Elemente erstellen, falls du ähnliche Formulare oder so brauchst

 

[Otz]

ich denke aber es macht relativ wenig aus ob ich ein framework objekt kopieren will oder ein "normales" oder?!

für beide sollte im großen udn ganzen das selbe gelten.

 

[SlaterB]

was genau der Fehler ist, das verschweigst du ja weiterhin, ich hatte zuletzt

> wenn ich im geclonten objekt ein attribut ändere, wird dies im ursprünglichen objekt geändert.

von einem deiner früheren Postings kopiert und da schreibst du nun prompt 'nein' dran

wenn die CheckBoxen gleichzeitig in die GUI eingebunden sind, dort durch irgendwelche Prozesse gecheckt werden oder nicht,
kann ich mir vorstellen, dass gleiche Caption, was immer das ist, eine Auswirkung hat,

aber wie gesagt egal, du musst schon sagen was überhaupt das Problem ist

 

[Otz]

Problem:

wenn ich im geclonten objekt ein attribut ändere, wird dies im ursprünglichen objekt geändert.

nein

aber wenn ich im ursprünglichen element was ändere, ändert es sich im geclonten

 

[SlaterB]

aber was, wie genau, welcher Container, welches Object1, welche CheckBox, welches Attribut darin mit welchem Wert, was für ein Methodenaufruf?

hole dir die beiden Objekte direkt nebeneinander her in zwei Variablen,
CheckBox a = ..;
CheckBox b = ..;
Ausgabe ob es dieselben sind: ==
Ausgabe der Zustände vorher,
welcher Aufruf?
Ausgabe der Zustände nachher,

wenn es ein Framework-Objekt ist, kann man doch gar nicht sagen, was da intern alles passiert,

wie kannst du das mit einem eigenen Dummy-Objekt a la MeinObjekt mit Attribut k aus Posting #2 vergleichen?
nur eigene Objekte zeigen exakt nachvollziehbar das Java-Verhalten: wenn die Objekte != sind, dann kann man selbstverständlich die Attribute einzeln verändern,

was in einem Framework-Objekt wie CheckBox los ist kann man im guten Glauben zunächst als normal annehmen,
ein unerwartetes Verhalten dort hat aber evtl. nichts mit Java an sich zu tun, sondern kann immer durch die interne Implementierung gut begründet sein


edit:
mir fällt auf dass meine Postings langsam 3x so lang werden wie deine, obwohl es andersrum sein sollte,
in dieser Häufung nicht mehr zu vertreten,
wenn du weiter nur einzelne Sätze postest werde ich wohl gezwungen sein, auch nur noch 'ja aber Java geht' zu antworten,
falls damit geholfen ist..

 

[Otz]

ich glaube wirklich du hast recht

ich habe mir jetzt die beiden objekte geholt und verglichen sind beide !=
genauso bei den checkboxen. die sind auch !=
meine deepCopy methode sollte also funktionieren.


da muss wohl noch irgendwsa im hintergrund ablaufen was ich (noch) nicht herausgefunden habe.

ich werde wohl noch etwas herumprobieren müssen.



sorry dass mein beitrag nicht größer als deiner geworden ist, es gibt eben nicht mehr zu berichten sollte ich eine lösung gefunden haben, werde ich auch mehr von mir geben können.
Danke für die Gedult und Hilfestellungen ich meld mich wenns was neues gibt

 

[Warrior7777]

Zum Thema Objekte kopieren hab ich das hier gefunden: Kopieren von Objekten Das scheint hier jedoch nicht das Problem zu sein, aber evt. für jemanden, der mit einem ähnlichen Problem auf den Thread hier stößt.

Noch einen schönen Sonntag!