toString-Methode

[Heinrich500]

Hallo,
ich habe folgendes Klasse Rechteck programmiert:

class Rechteck {
   
    final double hoehe;
    final double breite;
   
    Rechteck(double breite, double hoehe){
        this.breite=breite;
        this.hoehe=hoehe;
    }
   
    public  double breit(){
        return breite;
    }
    public double hoch(){
        return hoehe;
}
public double fläche(){
        return breite*hoehe;
}
  public boolean equals(Rechteck a){
        if(a.breite==breite && a.hoehe==hoehe) return true;
        else return false;
    }
}

Wenn ich jetzt eine toString-Methode schreiben will, wie mache ich das genau? Die Methode hat doch die Aufgabe hier, mein Rechteckobjekt "benutzerfreundlich" auszugeben.
D.h wenn ich mein Rechteck Objekt der print-Methode übergebe, bekomme ich eine von mir erzeugte Darstellung.
Also ich hätte folgende Idee:

public String toString(){
        return "breite" + breite + "\n" + "höhe: " + hoehe + "\n";
       
    }

Aber wie bringe zum Bsp die equals-Methode in der toString-Methode unter?

 

[mihe7]

Aber wie bringe zum Bsp die equals-Methode in der toString-Methode unter?

Gar nicht, wozu sollte das gut sein?

 

[mihe7]

Weil ich es gerade sehe:

        if(a.breite==breite && a.hoehe==hoehe) return true;
        else return false;

Das ist äquivalent zu:

        return a.breite == breite && a.hoehe == hoehe;

 

[Heinrich500]

Weis ich nicht. Ich hätte gedacht, dass man vielleicht sowas wie "sind gleich" ausgibt statt dem true?
Kann man meine aktuelle toString-Methode noch besser machen oder ist das in Ordnung?
Achso
Ja das ist besser. Danke

 

[fhoffmann]

public boolean equals(Rechteck a)

Diese Methode überschreibt nicht (!) die Methode equals der Klasse Objekt. Das sollte sie aber tun!

 

[Heinrich500]

Aber warum nicht. Was muss ich denn ändern?

 

[mihe7]

Ich hätte gedacht, dass man vielleicht sowas wie "sind gleich" ausgibt statt dem true?

Oh, ich denke, Du verwechselst da verschiedene Dinge miteinander.

toString soll eine String-Darstellung des betreffenden Objekts liefern. Das Objekt ist in Deinem Fall ein einziges Rechteck. Deine toString()-Methode wäre also prinzipiell in Ordnung.

Wenn Du zwei Rechteck-Objekte r1, r2 hast und r1.equals(r2) aufrufst, dann gibt das zunächst ein boolean zurück (s. Rückgabetyp von equals). Dieses wird bei der Ausgabe von Java automatisch zu Zeichenketten ("true" bzw. "false") konvertiert.

Du willst nun zwei Rechtecke miteinander vergleichen und das Ergebnis (ggf. inkl. der Rechtecke) ausgeben.

// stellt den Vergleich der beiden Rechtecke r1 und r2 als
// Zeichenkette dar
public String vergleichsDarstellung(Rechteck r1, Rechteck r2) {
    String nicht = "";
    if (r1.equals(r2)) {
        nicht = " nicht";
    }
    return "Das Rechteck\n" + r1.toString() + "und das Rechteck\n" + r2.toString() + " unterscheiden sich" + nicht + ".\n";
}

Das könntest Du jetzt für eine Ausgabe benutzen:

System.out.println(vergleichsDarstellung(r1, r2));

 

[mihe7]

Aber warum nicht. Was muss ich denn ändern?

Die Signatur der Methode anpassen:

public boolean equals(Object o) {

 

[Heinrich500]

Danke
D.h ich verwende dann meine schon geschriebene toString-Methode, um die Rechtecke zu vergleichen.
Meine toString gibt dann also nur gewisse einfache Attribute aus,während schwierigere Dinge in den Methoden dann geregelt werden?

D.h ich muss meinen Datentyp zu Rechteck ändern.

 

[mihe7]

D.h ich muss meinen Datentyp zu Rechteck ändern.

Nein, Du kannst das außerhalb machen.

 

[Heinrich500]

Wo außerhalb?
Ich verstehe nicht, was du meinst.

 

[mihe7]

public class Test {
    // stellt den Vergleich der beiden Rechtecke r1 und r2 als
    // Zeichenkette dar
    public String vergleichsDarstellung(Rechteck r1, Rechteck r2) {
        String nicht = "";
        if (r1.equals(r2)) {
            nicht = " nicht";
        }
        return "Das Rechteck\n" + r1.toString() + "und das Rechteck\n" + r2.toString() + " unterscheiden sich" + nicht + ".\n";
    }

    public void run() {
        Rechteck r1 = new Rechteck(...);
        Rechteck r2 = new Rechteck(...);
        System.out.println(vergleichsDarstellung(r1, r2));
    }

    public static void main(String[] args) {
        Test test = new Test();
        test.run();
    }
}

Hier wird einfach das bestehende Rechteck verwendet.

 

[Heinrich500]

Ok aber dann überschreibe ich doch die equals Methode nicht oder worauf willst du hinaus?

 

[mihe7]

Ach so, dann habe ich Dich falsch verstanden. Dir ging es ums Überschreiben der equals-Methode? Die musst Du in Rechteck abändern:

public boolean equals(Object o){
    if (o == null || o == this || !(o instanceof Rechteck)) {
        return o == this;
    }
    Rechteck a = (Rechteck) o;
    return a.breite==breite && a.hoehe==hoehe;
}

 

[Xyz1]

@Heinrich500 ... Wie alt bist Du ungefähr - und wie lange lernst du schon ungefähr? Zeige uns Bitte nochma den von dir produzierten Code.

 

[Heinrich500]

Achso jetzt verstehe ich es.

Mein Code nochmal:

lass Rechteck {
 
    final double hoehe;
    final double breite;
 
    Rechteck(double breite, double hoehe){
        this.breite=breite;
        this.hoehe=hoehe;
    }
 
    public  double breit(){
        return breite;
    }
    public double hoch(){
        return hoehe;
}
public double fläche(){
        return breite*hoehe;
}
 
public boolean equals(Object o){
    if (o == null || o == this || !(o instanceof Rechteck)) {
        return o == this;
    }
    Rechteck a = (Rechteck) o;
    return a.breite==breite && a.hoehe==hoehe;
}
}
public String toString(){
        return "breite" + breite + "\n" + "höhe: " + hoehe + "\n";
}
     
    }

Ich bin 19 und lerne seit diesem Semester Java als einführende Programmiersprache.

 

[Xyz1]

19 Tage?

aufe Schnelle kann ich oben keine Semantischen dafür aber viele (einfache) syntaktische Fehler erkennen.

 

[Heinrich500]

Ich habe das nur zusammenkopiert. Wsl habe ich Klammern vergessen usw.
Mir ging es um die Semantik. Vor allem, dass man in der toString Methode einfach nur die offensichtlichen Attribute ausgibt.

 

[Xyz1]

Sorry, @Heinrich500 , ich dachte du seist ein Spam-Bot oder ne künstliche Intelligenz. sorry

 

[Heinrich500]

Was meinst du damit?

 

[Xyz1]

Schlafenzeit!
@mihe7 kannst Du übernehmen?

 

[Heinrich500]

Ich verstehe nicht, was das Problem gerade ist?

 

[mihe7]

@Heinrich500 alles gut. Ab und zu erstellen Bots im Forum automatisiert Beiträge und @horstiii2 dachte, Dein Beitrag wäre so einer.

 

[Heinrich500]

Ok ich verstehe nicht, wie er darauf kommt. Würde mich interessieren.
Bei der Aufgabe haben wir glaube ich alles geklärt

 

[Xyz1]

Ich verstehe nicht, was das Problem gerade ist

alles gut, hatte versucht eine Art Turing-Test an dir auszuprobieren....

Ich weiß leider gar nicht wie "schlau" künstliche Intelligenzen heute schon sind