[Programmiererstammtisch] "Zum ächzenden Compiler"

[alpha civ]

Was macht denn Node.setPosition genau?

[Tiramisu]

Laut GraphNode.java soll das die Position des Knotens ändern:

...
public class GraphNode {
...
public Vector2D getPosition() {
        return position;
    }

    public void setPosition(Vector2D position) {
        this.position = position;
    }

    public Vector2D getDisplacement() {
        return displacement;
    }

    public void setDisplacement(Vector2D displacement) {
        this.displacement = displacement;
    }
}
...

[alpha civ]

Dann ist das Problem klar: jeder Knoten greift auf denselben Deltavektor zu. Änderst du den Deltavektor, so änderst du die Position jedes Knotens.

[Tiramisu]

Danke! Aber warum? Und wie kann ich das Problem umgehen? Habe ich wieder etwas mit den Pointern falsch gemacht?

[Wild Haggis]

Initialisiere ihn in der Schleife.

[alpha civ]

Collection<GraphNode> GraphNodes = Graph.getNodes();
...

for (GraphNode Node : GraphNodes) {
Vector2D DeltaVector = new Vector2D();
...//Berechne DeltaVector
DeltaVector.setX(0); 
Node.setPosition(DeltaVector);
}

Die Initialisierung von DeltaVector gehört in die Schleife, damit jedes Mal auch ein neuer Vektor erzeugt wird.

[Tiramisu]

Oh, Mann! Danke euch!

[Strat]

In C++ gibt es ja list mit konstanten Iteratoren. Vermutlich ist es in Java ähnlich

[alpha civ]

Danke! Aber warum? Und wie kann ich das Problem umgehen? Habe ich wieder etwas mit den Pointern falsch gemacht?

Bei einer Zuweisung "a = b" wird (außer bei Grundtypen) a die Objektreferenz b zugewiesen. D.h. nach der Zuweisung kann man sowohl mit a als auch b auf das selbe Objekt zugreifen. Insbesondere wird das Objekt nicht kopiert!

[Tiramisu]

Aber in meine Code gibt es nirgends eine Stelle, wo ich sowas wie "DeltaVector = Node.getPosition()" verwendet hätte.
Ich habe höchstens
DeltaVector = DeltaVector.subtract(OtherNode.getPosition())
verwendet, um DeltaVector - OtherNode.getPosition() zu berechnen. Aber das sollte doch keine ungewollten Fehler verursachen, oder?

[alpha civ]

Es geht um das hier:

public void setPosition(Vector2D position) {
        this.position = position;
    }

[Tiramisu]

Aaah! Danke!

[Shakka]

Das muss man ein paar mal falsch machen damit man's lernt.

Wenn man ein Buch über Herplatten liest, hat man auch keine Vorstellung davon, was Hitze sein kann. Das lernt man auch nur durch Schmerz.

[Tiramisu]

Ja, man muss ein paar Wochenenden in den Sand setzen, um Java richtig zu hassen.

[alpha civ]

Das muss man ein paar mal falsch machen damit man's lernt.

Wenn man ein Buch über Herplatten liest, hat man auch keine Vorstellung davon, was Hitze sein kann. Das lernt man auch nur durch Schmerz.

Oja. Mein "Lieblingsfehler" in meiner Anfangszeit mit Python war dieser hier:

xs = [3,1,2]
for x in xs.sort():
# irgendwas

Das habe ich ein paar Mal gemacht, seitdem nie wieder.