Ungewollte Wertveränderung einer Variable

[julian.veh]

Hallo,

ich habe das Problem, dass in einer Methode ungewollt ein Wert verändert wird, was eigentlich nicht sein dürfte. Hier einmal der betroffene Codeausschnitt:

public void revise(Formula f) {
        //f hat ein array 'position' mit zwei Elementen (x und y Koordinate)
        Formula fnegated = (Formula)f.negate();
        //die Koordinaten sind unverändert
        contract(fnegated);
        //nach Aufruf dieser Methode ist das Vorzeichen der y-Koordinate verändert.
        expand(f);
        System.out.println("bla");
}

Wie kann das sein, wo kann ich den Fehler suchen? Die Methode negate() gibt ein neues Objekt zurück, es werden keine Änderungen an f vorgenommen, aber die Änderung ist ja sowieso erst nach der contract Methode da. Das neues Objekt, das in der negate-Methode erzeugt wird, übernimmt auch die Koordinaten vom ursprünglichen Objekt. Es wird ein weiteres Attribut vom Typ boolean verändert.
In Java werden Parameter doch immer mit Call by Value übergeben.
Wie kann ich rausfinden, wann sich der Wert ändert? Ich habe was zu Trackpoints gefunden, aber wo muss ich den setzen. Kenne mich im Debugger nicht so aus.
(Ich nutze Eclipse)

Danke schonmal für jede Hilfe.

LG Julian

 

[mkdrive2]

In Java werden Parameter doch immer mit Call by Value übergeben.

Das stimmt nicht. Objekte werden als Referenz übergeben, sodass die Attribute verändert werden können.

Ich gehe davon aus, dass bei "f.negate()" kein richtiger Klon des Objekts "f" zurückgegeben wird (d.h. nicht alle Attribute kopiert werden). Wenn Objekte einfach kopiert werden, dann haben danach beide Variablen Referenzen zum gleichen Objekt. Arrays (was ja Teil von Forumla ist) ist auch ein Objekt.

Gib mal den Code für Formula.negate() + Attribute von Formula.

 

[httpdigest]

Die Methode Formula.negate() sieht ganz bestimmt so aus:

public Formula negate(Formula f) {
  Formula res = new Formula();
  res.xyArray = f.xyArray;
  return res;
}

Das heißt, das Array mit den x und y Werten wird einfach "shallow/flach" kopiert/übernommen. Somit teilen sich dann beide Formula-Instanzen dasselbe Array.

 

[mkdrive2]

Das heißt, das Array mit den x und y Werten wird einfach "shallow/flach" kopiert/übernommen. Somit teilen sich dann beide Formula-Instanzen dasselbe Array.

Wenn du nicht möchtest, dass man mit der Methode "negate()" zurückgegebene Formula-Objekt das vorige Formula-Objekt verändern kann, dann musst du ein neues Array erstellen, anstatt einfach, wie im Code nur die Referenz zu kopieren.

Wenn das gewollt ist und "contract()" nichts verändern soll, dann bitte nochmal den Code für "contract()".

 

[httpdigest]

Wenn du nicht möchtest...

Häh? Ich möchte überhaupt nichts. Ich bin nicht der ursprüngliche Fragesteller.
Ich habe nur geschrieben, wie ich vermute, dass die Methode wohl aussieht, während du schon deine erste Antwort verfasst hattest.

 

[julian.veh]

Danke für die Antworten, das ging ja schnell.

Bisher so:

@Override
RoboFormula negate() {
        return new PlayerRunningFormula(this.uniformNumber, this.sameTeam, this.localVector, this.direction, !this.runsInDirection);
    }

Sollte es dann so funktionieren?

@Override
RoboFormula negate() {
  double[] directionClone = new double[2];
   directionClone[0] = ...;
    //und so weiter
        return new PlayerRunningFormula(this.uniformNumber, this.sameTeam, vectorClone, directionClone, !this.runsInDirection);
    }

Die anderen sind alles primitive Datentypen.

LG

edit:___________
oder vermutlich einfach mit direction.clone(); und localVector.clone(); ?!

Jedenfalls Danke. Konnte es noch nicht testen, aber das klingt so als wäre mein Problem gelöst!

 

[mkdrive2]

Sieht schon ganz gut aus. Die beiden Arrays werden geklont. Ich nehme an runsInDirection ist ein boolean. Welche Datentypen sind uniformNumber und sameTeam. Wenn du keine weiteren Überraschungen erleben willst, dann sollten diese beiden keine Objekte sein.

Häh? Ich möchte überhaupt nichts. Ich bin nicht der ursprüngliche Fragesteller.

Sorry, habe ich nicht gemerkt.

 

[julian.veh]

Genau, sameteam boolean und uniformNumber int.

 

[mrBrown]

Sollte es dann so funktionieren?

@Override
RoboFormula negate() {
  double[] directionClone = new double[2];
   directionClone[0] = ...;
    //und so weiter
        return new PlayerRunningFormula(this.uniformNumber, this.sameTeam, vectorClone, directionClone, !this.runsInDirection);
    }

Praktikabler ist es oftmals, die Kopie im Konstruktor zu erstellen, und nicht zu erwarten, dass eine Kopie übergeben wird

 

[mihe7]

Das stimmt nicht. Objekte werden als Referenz übergeben, sodass die Attribute verändert werden können.

Doch, das stimmt. Es wird die Objektreferenz als Wert übergeben.