Gleiche Objektreferenz trotz clone()?

[renwal]

Hallo!

Ich brauche mal eure Hilfe beim Lösen folgenden Problems: In einem Quiz-Programm erzeuge ich eine Variable namens

currentQuestion

, die immer auf die aktuell angezeigte Frage verweist, die wiederum aus einem Array geladen wird.
Nun versuche ich, eine neue Frage zu laden, während ich in einem anderen Thread quasi die Aufräumarbeiten erledige. Während dem Laden der neuen Frage ändert sich auch die Referenz der Variable currentQuestion. Um deshalb aber keine Probleme mit dem Aufräum-Thread zu bekommen, der sich ja des "alten" Fragen-Objekts bedient, erstelle ich einen Klon von currentQuestion namens

oldCurrentQuestion

und übergebe diesen dem Aufräum-Thread.
Der Effekt ist dabei, dass das Klonen keinen Einfluss zu haben scheint. Sobald ich der Variable currentQuestion ein neues Objekt zuweise, ändert sich gleichermaßen die Referenz von oldCurrentQuestion. Ergebnis: Es fliegen reihenweise Exceptions und das GUI friert ein.

Um einen Rat wie das Problem zu lösen ist wäre ich euch sehr dankbar!

LG renwal

 

[Fab1]

Hi,

schaue dir mal folgendes (Java Insel) dazu an. Ich denke in dem Satz:

Felder erlauben standardmäßig clone(). Speichern die Arrays jedoch nicht-primitive Werte, liefert clone() nur eine flache Kopie, was bedeutet, dass das neue Feldobjekt, der Klon, die exakt gleichen Objekte wie das Original referenziert und die Einträge selbst nicht klont.

Steht das Problem beschrieben.

Weiteres siehe Link.

 

[bERt0r]

Das "Aufräumen" sollte eigentlich der Garbage Collector für dich übernehmen, da musst du keinen extra Thread dafür schreiben...

 

[renwal]

Hi!

Danke schon einmal für die Antworten. Leider bin ich durch den Artikel nicht wirklich weitergekommen. Besagtes shallow copy-Problem kannte ich. Nur verstehe ich nicht, wo Besagtes bei meinem Code auftreten soll. Ich habe mal so in etwa reproduziert und auf das Kernproblem reduziert, was ich tue:

// Die Variable mit der aktuellen Frage
Question currentQuestion = null;
// ArrayIterator ist eine Klasse, die es mir ermöglicht, Arrays in Iterator-ähnlicher Weise zu verarbeiten
ArrayIterator questions = new ArrayIterator(new Question[]{frage1, frage2, frage3});

// Methode, die eine neue Frage lädt
private void showQuestion()
{
    if (currentQuestion != null)
    {
        // Die aktuelle Frage klonen
        final Question oldCurrentQuestion = currentQuestion.clone();
        // Den Cleanup-Thread erzeugen und starten
        new Thread("cleanup")
        {
            public void run()
            {
                // Hier wird das Objekt oldCurrentQuestion benutzt (z.B. oldCurrentQuesion.getUIComponents(), etc.)
                // GUI aufräumen, Datenstrukturen verarbeiten, Berechnungen durchführen
            }
        }.start();
    }
    // Die neue Frage setzen
    currentQuestion = questions.next();
    // GUI erzeugen, etc.
}

@bERt0r: Das meinte ich mit "Aufräumarbeiten", siehe Z.19.

 

[Michael...]

Das Vorgehen hört sich irgendwie nach einer Verzweiflungstat an. Und inwiefern ein clone() bei Deinem Problem helfen soll ist mir auch nicht ganz klar ( helfen tut es ja ohnehin nicht)

Hierzu:

Sobald ich der Variable currentQuestion ein neues Objekt zuweise, ändert sich gleichermaßen die Referenz von oldCurrentQuestion. Ergebnis: Es fliegen reihenweise Exceptions und das GUI friert ein.

Nur weil man eine von zwei Variablen, die zuvor auf das selbe Objekt referenzieren, auf ein neues Objekt referenzieren lässt, wird die Referenz der zweiten Variable nicht automatisch mit umgebogen.
Da muss was ganz anderes passieren.

 

[D4rkscr43m]

wie sieht denn die

Question#clone

Methode aus? Vielleicht wird da ein Internes Objekt nicht geklont sondern 1zu1 in das neue Question-Objekt geschrieben?

 

[FArt]

Was wird aufgeräumt und warum in einem anderen Thread? Wieso musst du dafür ein Objekt klonen?

 

[renwal]

Naja, Michael hat Recht, etwas Anderes ist mir wirklich nicht mehr eingefallen.

Aufgeräumt wird das GUI, während die neuen Objekte im Hauptthread schon erzeugt werden. Außerdem kümmert sich der zusätzliche Thread um all die Aufgaben, die in der Speicherverwaltung in den Fragenobjekten anfallen. Es müssen z.B. die Verknüpfungen zwischen den Container- und GUI-Objekten entfernt werden und Variablen in diesen Container-Objekten aktualisiert werden.

Die Clone-Methode der Question-Klasse ist folgende:

@Override
public Evaluatable doClone() throws CloneNotSupportedException
{
    return (Evaluatable) this.clone();
}

 

[Volvagia]

Benutzt du Swing? Dann ist es keine gute Idee einen separaten Thread dafür zu verwenden. Ich verstehe auch nicht warum, den neuen Thread zu starten kostet wahrscheinlich mehr Zeit als ein paar GUI-Referencen zu kappen.

Du hast currentQuestion.
Dann lädst du eine neue Frage, also currentQuestion = questions.next();
Wäre das dann nicht einfacher?

Question oldQuestion = currentQuestion;
//Thread
currentQuestion = questions.next();?

Dann steht in oldQuestion die Frage zum Aufräumen und in currentQuestion die nächste/aktuelle Frage.

 

[FArt]

Aufgeräumt wird das GUI, während die neuen Objekte im Hauptthread schon erzeugt werden. Außerdem kümmert sich der zusätzliche Thread um all die Aufgaben, die in der Speicherverwaltung in den Fragenobjekten anfallen. Es müssen z.B. die Verknüpfungen zwischen den Container- und GUI-Objekten entfernt werden und Variablen in diesen Container-Objekten aktualisiert werden.

Und das ist so aufwendig, dass es in einem eigenen Thread (nicht Main-Thread und nicht EventQueue) augelagert werden muss? Wie lange brauchst du denn dafür?
Und warum überhaupt? Sinnvollerweise hast du doch ein Datenmodell (z.B. pro Frage). Wenn das nicht mehr referenziert wird, wird es auch collected. In der Regel gibt es da nicht viel zu tun.
Der richtige Ansatzpunkt für das Problem wäre wohl ein ordentliches Design deiner Datenstruktur in Verbindung mit der GUI.

 

[renwal]

Erst einmal sorry, dass das so lange gedauert hat!

Und das ist so aufwendig, dass es in einem eigenen Thread (nicht Main-Thread und nicht EventQueue) augelagert werden muss? Wie lange brauchst du denn dafür?

Da hängt ein Animationssystem drin. Damit beide Animationen möglichst gleichzeitig laufen können, wird die eine im Hintergrund gestartet und das GUI wird aufgeräumt, während der Hauptthread noch die neuen Daten lädt.

Sinnvollerweise hast du doch ein Datenmodell (z.B. pro Frage). Wenn das nicht mehr referenziert wird, wird es auch collected. In der Regel gibt es da nicht viel zu tun.
Der richtige Ansatzpunkt für das Problem wäre wohl ein ordentliches Design deiner Datenstruktur in Verbindung mit der GUI.

Ich verstehe die Formulierung nicht ganz. Ein Datenmodell existiert ja, ich speichere ja nicht irgendwo planlos Bytes. Es geht ja auch nicht darum, dass es nicht collected wird, sondern einfach nur, dass das Original und die Kopie des Objektes irgendwie immer noch "verbunden" sind, d.h. eine Änderung des einen zieht eine Änderung des anderen mit sich.

 

[renwal]

*schieb*

 

[FArt]

Du hast den Wink "keine weiteren Antoworten" nicht verstanden. Folge doch mal dem Link meiner Signatur.

Es wurde bereits von verschiedenen Seiten dezent darauf hingewiesen, dass dein Vorgehen seltsam und unnötig ist. Es bringt dir also nicht, an dem Symptom herumzuknubbeln. Dein Herangehensweise ist falsch.

Mit den bisher gelieferten Informationen sehe ich keinen Grund hier z.B. Objekte klonen zu müssen oder "Aufräumarbeit" erledigen zu müssen.

Halte dich an den Standard zur Programmierung zu Swing GUIs. Stößt du auf ein Problem, dann melde dich damit hier im Forum. Aber nicht mit einem Problem zu einem von dir warum auch immer eingeführten Workaround.