compareTo

[gast]

hallo,

wie vergleicht man richtig? so:

	public int compareTo(Kunde kunde)
	{
		Kunde neuerKunde = new Kunde();
		if(kunde == null)
			return 1;
		if(name.compareTo(neuerKunde.name) == 0 && vorname.compareTo(neuerKunde.vorname)   ==0) 
			return 0;
		
		return 1;
	}

oder macht man das so?

if(kunde.name.compareTo(neuerKunde.name) == 0 usw

danke!

 

[HLX]

Du brauchst keinen neuen Kunden anzulegen. Du vergleichst mit dem Objekt, das compareTo() implementiert.

Verwende außerdem Getter für den Zugriff.

 

[Leroy42]

public int compareTo(Kunde kunde) {
  int res1 = name.compareTo(kunde.name);
  int res2 = vorname.compareTo(kunde.vorname);
  return res1<0 ? res1 : res1==0 ? res2 : res1;
}

 

[gast]

also dann so?

  public int compareTo(Kunde kunde) 
   { 
            if(kunde == null) 
         return 1; 
      if(getName().compareTo(kunde.getName()) == 0 && getVorname().compareTo(kunde.getVorname())   ==0) 
         return 0; 
       
      return 1; 
   }

 

[Leroy42]

Nein!

Lies mal wie compareTo definiert ist (Klicken)

 

[FelixB]

Leroy hat dir doch schon ne Lösung geschrieben, was gefällt dir denn daran nicht?

 

[HLX]

Leroy hat dir doch schon ne Lösung geschrieben, was gefällt dir denn daran nicht?

eine mögliche NullPointerException?

 

[Leroy42]

eine mögliche NullPointerException?

Okay!

public int compareTo(Kunde kunde) { 
  if (kunde==null) return -1;
  int res1 = name.compareTo(kunde.name); 
  int res2 = vorname.compareTo(kunde.vorname); 
  return res1<0 ? res1 : res1==0 ? res2 : res1; 
}

Besser? :wink:

 

[Gast]

was bedeutet das:

return res1<0 ? res1 : res1==0 ? res2 : res1;
???

 

[Leroy42]

return res1<0 ? res1 : res1==0 ? res2 : res1;

Wenn res1 kleiner als 0 ist (nachname 1 kleiner) dann res1 zurückgeben
sonst
wenn res1 gleich 0 (name==kunde.name) dann res2 (ergebnis des vornamenvergleichs) zurückgeben
sonst (res1 ist größer als 0) res1 zurückgeben

 

[merlin2]

Das ist der ternäre Operator.

 

[FelixB]

Leroy hat dir doch schon ne Lösung geschrieben, was gefällt dir denn daran nicht?

eine mögliche NullPointerException?

wobei man auch darüber streiten kann, wie sich ein Objekt zu einem anderen verhält, das gar nicht existiert...

 

[Leroy42]

wobei man auch darüber streiten kann, wie sich ein Objekt zu einem anderen verhält, das gar nicht existiert...

Eben, deshalb würde ich in diesem Fall auch
gerne ne Exception geworfen haben um den
(Programmier-)Fehler überhaupt zu bemerken.