Probleme bei toString Überschreibung

[Guest]

Hallo,

ich möchte 2 Objekte “addieren”:

Point point1=new Point(2,3);
Point point2=new Point(4,5);

4+2 und 3+5;

Ausgabe soll 6|8 sein. Bei mir kommt dauernd 0|0 raus. Ich weis nicht wieso. Konstruktoren habe ich natürlich bereits definiert.
Hier mal mein Code:

Klasse Point:

Point temp;
private int resultX, resultY;
……
……
public Point add(Point p)
   {
   temp = new Point(p.getX()+this.x,p.getY()+this.y);
   resultX=p.getX()+this.x;
   resultY=p.getY()+this.y;
   return temp;
   }

public String toString()
   {
   String result="Ergebnis: "+resultX+'|'+resultY;
   return result;
   }

public Point(int x, int y)	// Konstruktor
   {
   this.x=x;
   this.y=y;
   }
…..
…..

Main Klasse

……
……
Point point1=new Point(2,3);
Point point2=new Point(4,5);
System.out.println(point1.add(point2));
…..
…..

Es wäre super, wenn jamand sagen könnte was ich evtl falsch gemacht habe

Danke im Voraus

 

[meez]

Weil du halt nene neuen Point (tmp) erstellst, und resultXYZ in diesem Objekt noch nicht definiert wurden (nur initialisiert)...Schliesslich sind die ja nicht statisch...

 

[mattulla]

public String toString() 
{ 
   String result="Ergebnis: "+this.getX+'|'+this.getY; 
   return result; 
}

die Geschichte mit dem resultX und resultY kannst dir schenken

 

[Leroy42]

Ich vermute mal, daß du dir eine eigene Klasse Point geschrieben
hast, denn java.awt.Point besitzt keine add-Methode (warum
eigentlich nictt? )

Erstens mal, macht es keinen Sinn, tmp als Instanzvariable zu deklarieren;
Du brauchst sie ja nur lokal in add().

Desweiteren ist es so, daß bei Aufruf von add, der Point selbst verändert
wird durch resultX=... Ist das gewollt?

Schließlich lieferst du ein zusätzliches Element, daß ausgeben wird.

Und überhaupt? Wie heißen eigentlich die Instanzvariablen deiner
Klasse Point? Zur Berechnung rufst du getX() und x auf.
Bei der Ausgabe widerum gibst du resultX aus :shock: :shock:

Ich glaube, du hast dich irgendwie total verstrickt, darum

 

[Guest]

Vielen Dank matulla

@Leroy42 resultX und resultY sind doch unnötig und temp sollte als lokale Objektvariable definiert werden. Ihr hattet recht.

Vielen Dank nochmal, ich war lange am suchen

 

[bygones]

Ich vermute mal, daß du dir eine eigene Klasse Point geschrieben
hast, denn java.awt.Point besitzt keine add-Methode (warum
eigentlich nictt? )

warum auch - was für einen logischen sinn hätte es dass ein Punkt eine Add methode hätte ? einzig könnte man seine Koordinaten ändern, also den punkt verschieben - dafür gibts move bzw translate in der Point klasse

 

[Leroy42]

Die Methode soll natürlich nichts ändern.

In Smalltalk gibt es eine add-Methode, die ein neues Objekt liefert.
Also wie eine 2D-Vektoraddition.

Aber egal, haben muß ich die auch nicht.