comparable und equals

[sh33p]

habe mal aus langeweile eine Klasse Hund geschrieben,um Comparable und Equals zu
implementieren.doch irgendwie liegt noch ein kleiner fehler in equals.comparable liefert das richtige..
nur equals nicht.bsp:

Hund h1 = new Hund("Bello",5);
Hund h2 = new Hund("Bello",5);

comparable liefert 0,da name und alter gleich sind. equals aber false..

public class Hund implements Comparable<Hund>{

private String name;
private int alter;


public Hund(String name, int alter){
  this.name = name;
  this.alter = alter;
}
public int getAlter() {
  return alter;
}

public String getName(){
  return name;
}
public int compareTo(Hund h){   // vergleich auf alter und name

if( h.getAlter() == this.getAlter()){
  return 0;
}
if(h.getAlter() <this.getAlter()){
  return -1;
}
if(h.getAlter() > this.getAlter()){
  return 1;
}

  return getName().compareTo(h.getName());
}

// Equals definieren

public boolean Equals(Object o){
  if(o == null){
    return false;
  }
  if( o == this){
    return true;
  }
  if(! o.getClass().equals(this.getClass())){
    return false;
  }
  Hund that = (Hund) o;
  
  return this.getName() == that.getName() && this.getAlter() == that.getAlter();


}
}

 

[Murray]

Strings müssen mit equals verglichen werden.

 

[ARadauer]

comparable liefert 0,da name und alter gleich sind.

nein, da alter gleich ist, dann machst du schon dein return...
nenn sie mal unterschiedlich, dan wirst du auch 0 bekommen...

 

[firewarrior]

Methodennamen sind case-sensitive. Es sollte also equals() und nicht Equals() heissen. Die @Override Annotation kann bei sowas helfen.

 

[sh33p]

nein, da alter gleich ist, dann machst du schon dein return...
nenn sie mal unterschiedlich, dan wirst du auch 0 bekommen...

ok ich sehs..wie kann ich denn im comparable testen,ob mehrere objektattribute in den objekten gleich sind und nicht nur 1?

 

[ARadauer]

ok ich sehs..wie kann ich denn im comparable testen,ob mehrere objektattribute in den objekten gleich sind und nicht nur 1?

mhn ja verschachteln ist eine Möglichkeit.
also wenn das Alter gleich ist, gibst du den wert vom compare to der namen zurück

   public int compareTo(Hund h){   // vergleich auf alter und name
      
         if( h.getAlter() == this.getAlter()){
            return getName().compareTo(h.getName());
         }else if(h.getAlter() <this.getAlter()){
           return -1;
         }else (h.getAlter() > this.getAlter()){
           return 1;
         }
      }

Gibt von Apache Commons noch ein paar externe Hilfsbiblitheken, die das einwenig schöner machen, aber die NAmen fallen mir im Moment nicht ein und ich bin zu faul zum suchen

 

[Murray]

Immer ein Attribut hernehmen und auf Ungleichheit testen; in dem Fall entsprechendend Rückgabewert liefern. Ist das Attribut bei beiden Instanzen gleich, einfach das nächst Attribut vornehmen.

In deinem Fall muss das return 0 bei gleichem Alter weg.

 

[sh33p]

danke.