vielen Dank für die zusätzliche Information.

Gruß
Stefan

Der im Artikel gewählte Ansatz ist der Empfohlene.

wirft man einen Blick in den Quellcode von Float.compare stellt man fest, dass hier nicht die float-Werte sondern die Bit-Repräsentationen in Form von Integern verglichen werden. Sie haben also recht, dass Float.compare (und analog dazu auch Double.compare) etwas anderes als ein Vergleich mit == ist. In der Praxis fällt dies jedoch kaum ins Gewicht, weshalb ich den Artikel gerne unverändert, und Ihren Kommentar als zusätzlichen Hinweis da stehen lassen würde.

Gruß
Stefan

Gruß

m3

genau das ist der Grund, warum man überprüfen sollte, ob es sich um das identische Objekt handelt. Der “contract” ist der Vertrag bzw. die Vorgaben von Sun, wie equals implementiert werden sollte. Dieser wird auf der zweiten Seite dieses Kapitels unter dem Punkt Vertrag/contract für hashCode und equals besprochen.

Gruß

Stefan

Ich verstehe das ein Prüfung, ob es sich um das selbe Objekt handelt, sinnvoll sein kann – gerade wenn der Vergleichsalgorithmus zeitaufwendig ist (hier nicht der Fall).

Ich verstehe jedoch nicht, was du mit “contract” meinst.

Gruß

m3

danke für Ihren Kommentar! Da das Java Blog Buch ein Buch ist, sollten die Kapitel mehr oder weniger aufeinander aufbauen. Leider wird instanceof erst später im Kapitel 04.05 Vererbung angesprochen, weshalb ich mich gegen die Verwendung von instanceof entschieden habe. Aber selbstverständlich funktioniert es auch mit instanceof, da haben Sie natürlich Recht.

Eine Anmerkung zu Ihrem Code/Kommentar im Code habe ich aber trotzdem:

>> den Kram darüber braucht man nicht – null wird durch instanceof geregelt

Dennoch sollte überprüft werden, ob es sich beim übergebenen Objekt um das selbe Objekt wie this handelt, da dies vom contract so verlangt wird.

Gruß

Stefan

//den Kram drüber brauch man net - null wird durch instanceof geregelt

  if (obj instanceof bottle) {         //funktioniert jetzt auch für subklassen
    Bottle b = (Bottle)obj;
    if (this.capacity == b.getCapacity() && this.content.equals(b.getContent())) {
      return true;
    }
  }
  return false;