Liste überschreibt alte Elemte

[Lisa0709]

Hallo ihr Lieben,
ich bin kompletter Java Anfänger, brauche es allerdings jetzt für die Uni.
Wir haben ein Projekt, bei dem ich über die GUI einen Helikopter von Punkt A nach Punkt B fliegen lassen soll. Dies soll Kästchenweise über Pixel laufen.
Also muss ich den kürzesten Weg approximieren, sodass der Heli quasi im Zickzack fliegt.
Alles etwas kompliziert. Mein Problem ist nun, dass sich meine Liste, in der ich die Pixelkoordinaten speichere, immer wieder überschreibt.
Sobald die While-Schleife das zweite Mal aufgerufen wird, wird zwar korrekt ein Array eingefügt, allerdings verändert sich der Eintrag vorher in der Liste auch. Wie kann ich das verhindern? Danke schon mal für jegliche Hilfe! 🙂

ArrayList<int[]> pixelliste = new ArrayList<int[]>();

int[] pixel = new int[2];
int[] pixel1 = new int[2];

//x_start_akt sind die Pixelkoordinaten, die sich ständig aktualisieren.

while (x_start_akt != x_ende && y_start_akt != y_ende) {
if (x_start_akt != x_ende && y_start_akt != y_ende) {
pixel[0] = x_start_akt - 20;
pixel[1] = y_start_akt;
x_start_akt = x_start_akt - 20;

pixelliste.add(pixel);
}
if (x_start_akt != x_ende && y_start_akt != y_ende) {
pixel1[0] = x_start_akt;
pixel1[1] = y_start_akt - 20;
y_start_akt = y_start_akt - 20;

pixelliste.add(pixel1);

}

 

[stg]

Du erstellst zu Beginn ein Array und fügst anschließend in der Schleife immer wieder das selbe Array in deine Liste ein.

Reicht dir das als Hinweis schon?

 

[Lisa0709]

Ja, aber ich verstehe leider nicht, warum die Liste überschrieben wird. Ich dachte, wenn ich adde wird einfach nur hinten dran gefügt.

 

[Xyz1]

Dafür gibt es einen Algorithmus für: https://en.wikipedia.org/wiki/Bresenham's_line_algorithm . Pseudocode und Grafiken sind auch dabei. Ich kann Dir, wenn Du willst das später mal ausprogrammieren.

 

[Lisa0709]

Ach, danke für den Tipp. Werde es damit später nochmal probieren, ansonsten würde ich drauf zurückkommen. Danke! 🙂

 

[mrBrown]

Ja, aber ich verstehe leider nicht, warum die Liste überschrieben wird. Ich dachte, wenn ich adde wird einfach nur hinten dran gefügt.

Die Liste wird nicht überschrieben, sondern sie enthält einfach N mal das selbe Array - und dieses eine Array änderst du in deiner Schleife.

 

[Lisa0709]

Die Liste wird nicht überschrieben, sondern sie enthält einfach N mal das selbe Array - und dieses eine Array änderst du in deiner Schleife.

Danke! Aber wie kann ich das denn umgehen? Ich verstehe nicht, wie ich immer neue Daten in mein Pixel Array speichern kann und die dann in die Liste adden kann.

 

[fhoffmann]

Du musst in deiner while-Schleife immer wieder aufrufen:

pixel = new int[2];

 

[Lisa0709]

Achsooooo, ja na klar. Da stand ich aber wirklich lange auf dem Schlauch.
Jetzt funktioniert alles, ich danke euch! 🙂

 

[mrBrown]

Als genereller Tipp zur Vermeidung solcher Fehler: Den Scope der Variablen möglichst klein halten 🙂

 

[mrBrown]

Diskussion verschoben nach https://www.java-forum.org/thema/scope-vs-gueltigkeitsbereich.185281/