Verständnissproblem Objekte

[Sebastian_650]

Mein Programm hat mir gerade bewiesen das ich mich doch nicht so ganz durchblicke.. Am besten kopiere ich mal den ganzen fraglichen Code rein:

public class Render extends Thread{
    
    /** Creates a new instance of Render */
    private Graphics graph;
    private JPanel panel1;
    private int rate;
    private Datatank cache;
    private int pointcount;
    private Masspoint[] points;
    
    public Render(Graphics device, JPanel panel, int _rate, Datatank _cache, int _pointcount) {
        graph = device;
        panel1 = panel;
        rate = (int)(1d / (double)_rate * 1000d);
        cache = _cache;
        pointcount = _pointcount;
        points = new Masspoint[pointcount];
        points = cache.get(); <= hier..
    }
    
    public void run(){
        while (true){
            graph.clearRect((int)(-panel1.getWidth()/2), (int)(-panel1.getHeight()/2), panel1.getWidth(), panel1.getHeight());
            for (int i = 0; i < pointcount; i++){
                graph.fillOval((int)(points[i].x[0]*0.1), (int)(points[i].y[0]*0.1), 6, 6);
            }
            //graph.fillOval(0,0,10,10);
            try{
                sleep(rate);
            }
            catch (InterruptedException e){}
        }
    }
}

 public synchronized Masspoint[] get(){
        return points;
    }

Das "Problem" liegt an der mit <= hier markierten stelle. Points sind dabei im Prinzip ein Array aus Punkten, welche auch noch Geschwindigkeiten etc gespeichert haben. Ich habe mir das so gedacht: Mit points = cache.get() kopiere ich die Werte der Punkte in das neue points Array der Renderschleife. Dann dürfte aber in der while(true) Schleife in run() nie ein neues Bild aufgebaut werden, die Werte werden hier ja nicht verändert und auch nie wieder get() aufgerufen. Fakt ist aber das er scheinbar immer auf die aktuellen Punktpositionen zugreift, denn das Bild bewegt sich. Scheinbar kopiert er nicht die Werte sondern legt sich direkt eine "Verknüpfung" an, die direkt zugreift? Das gefällt mir nicht ganz..

Das würde auch meine gazen Rechenthreads doch ad absurdum führen, denn die sehen vereinfacht gesagt so aus:
aktuelle Punkte mit .get() in lokalpoints[] speichern
neue Werte berechnen.
zurückschreiben mit Punktspeicher.set(lokalpoints)..[/code]

Edit: Kurzgesagt, ich brauche eigentlich eine kopie der Werte und keine direkte Referenz. Nur wie?

 

[Dunedain]

Wenn Masspoint eine von dir geschriebene Klasse ist, java.lang.Object.clone überschreiben und verwenden.
Beispiel:

public Masspoint clone ()
{
    /** Ich gehe einfach mal von nem vorhandenen leeren Konstruktor aus... */
    Masspoint m = new Masspoint();

    /** Werte kopieren... */
    m.setWertA( getWertA() );
    m.setWertB( getWertB() );
    //...

    /** ...Und Kopie zurückgeben */
    return m;
}


public synchronized Masspoint[] get ()
{
    return points.clone ();
}

 

[SlaterB]

erstmal musst du einen Punkt kopieren,
z.B. new Punkt(Punkt other) und im Konstruktor die einzelen Felder kopieren,

dann in einer for-Schleife das Array kopieren

 

[Beni]

Wenn Masspoint eine von dir geschriebene Klasse ist, java.lang.Object.clone überschreiben und verwenden.
Beispiel:

public Masspoint clone ()
{
    /** Ich gehe einfach mal von nem vorhandenen leeren Konstruktor aus... */
    Masspoint m = new Masspoint();

    /** Werte kopieren... */
    m.setWertA( getWertA() );
    m.setWertB( getWertB() );
    //...

    /** ...Und Kopie zurückgeben */
    return m;
}


public synchronized Masspoint[] get ()
{
    return points.clone ();
}

Clone lässt sich einfacher implementieren, da sind ein paar fiese Tricks eingebaut:

public class Masspoint implements Cloneable{
  private int x, y, ...;

  public Masspoint clone(){
    return (Masspoint)super.clone();
  }
}

 

[Sebastian_650]

Wenn ich das in Masspoints einbaue, müsste ich jeden Punkt einzeln in einer Schleife clonen? Denn mit get gebe ich ja das Array return points (ein Masspointarray) zurück und wenn ich da points.clone(); schreibe ruft er nicht die clone() funktion in Masspoints auf..?

Edit:

public synchronized Masspoint[] get(){
        Masspoint[] retp = new Masspoint[points.length];
        for(int i = 0; i < points.length; i++){
            retp[i] = points[i].clone();
        }
        
        return retp;
    }

Mit überschriebener clone() funktion:

public Masspoint clone(){
        try{
            return (Masspoint)super.clone();
        }
        catch(CloneNotSupportedException e){
            System.out.println("Error");
            return null;
        }
    }

Liefert immernoch ein bewegtes Bild statt einmalig Punkte zu holen. Das kann doch nicht sein??

Edit 2:

Ja doch kann sein: http://www.dpunkt.de/java/Die_Sprache_Java/Die_Sprachelemente_von_Java/47.html
"Bei Arrays von Objekten und somit auch geschachtelten Arrays ist zu beachten, dass die Arrays sowohl bei Object.clone() als auch bei System.arraycopy() flach kopiert werden, d. h., es werden nur die Objektreferenzen kopiert und nicht die Objekte selbst."
Prächtig, das hab ich gemerkt ^^ Und wie umgeht man das am elegantesten?

Last Edit: Im endeffekt scheint es am einfachsten zu sein wenn man wirklich in einer Schleife alle Elemente einzeln kopiert..