compareTo

[jack26]

Hallo,

ich hab grad ein kleines Verständnisproblem.

Ich habe folgenden Code, der glaube ich ziemlich falsch ist:

public int compareTo(Equipment arg0) {
		if(this.abschreibungswert()<arg0.abschreibungswert())
			return -1;
		if(this.abschreibungswert()>arg0.abschreibungswert())
			return +1;
		if(this.abschreibungswert()==arg0.abschreibungswert())
			this.getInvNr().compareTo(arg0.invNr);
                                                ....		
	}

Hierbei ist Equipment einfach ein Objekt einer selbstgeschriebenen Klasse Equipment.
Ich möchte eigentlich bloß folgendes:

Ich möchte 2 Equipments miteinander vergleichen, also so zum beispiel testEquipment.compareTo(anderesEquipment)
Dabei soll die selbstgeschriebene abschreibungswert()-Methode von beiden aufgerufen werden.
Anschließend sollen die beiden Rückgabewerte (ist jeweils eine int-Zahl) der abschreibungswert()-Methode miteinander verglichen werden und
-1 zurückgegeben werden, falls der abschreibungswert von testEquipment kleiner als der abschreibungswert von anderesEquipment ist
+1 zurückgegeben werden, falls der abschreibungswert von testEquipment größer als der abschreibungswert von anderesEquipment ist

und falls beide Rückgabewerte von abschreibungswert() gleich sind,

wird mit Hilfe des getters getInvNr() von beiden Equipments die InventarNummer (ist ein int-Attribut) aufgerufen und miteinander verglichen, falls hierbei die inventarNummer von testEquipment kleiner als die inventarNummer von anderesEquipment ist wird -1, ansonten +1 zurückgegeben.

Wäre für jede Hilfe dankbar.
Vielen Dank.
Gruß
jack26

 

[byte]

Wenn getInvNr einen int zurückliefert, dann ist die Zeile falsch. Denn primitive Typen (wie int) haben keine Methoden (somit auch keine compareTo()). Du musst das dann schon explizit prüfen und die Werte zurückgeben. Ausserdem in Zeile 6 das + entfernen.

 

[maki]

und falls beide Rückgabewerte von abschreibungswert() gleich sind,

wird mit Hilfe des getters getInvNr() von beiden Equipments die InventarNummer (ist ein int-Attribut) aufgerufen und miteinander verglichen, falls hierbei die inventarNummer von testEquipment kleiner als die inventarNummer von anderesEquipment ist wird -1, ansonten +1 zurückgegeben.

Dir ist klar das damit keine 2 Equipments gleich sein können (= compareTo gibt 0 zurück), oder?

Auch sollte die equals Methode genauso funktioniern, muss aber nicht zwingend.

 

[jack26]

Danke für die Antworten.
Ich habs schon gelöst mein Problem, stand wieder mal neben mir...

So sollts aussehen nachher:

public int compareTo(Equipment arg0) {
		if(abschreibungswert()<arg0.abschreibungswert())
			return -1;
		else if(abschreibungswert()==arg0.abschreibungswert()) 
			if(getInvNr()<arg0.getInvNr())
				return -1;
			
		return 1;
		
	}

Gruß
jack26

 

[Beni]

Für diese Methode muss gelten:

a.compareTo( a ) == 0

Falls das nicht gilt (wie bei dieser Implementation), werden einige Algorithmen (z.B. der Sortieralgorithmus von Java) ziemlichen Müll produzieren :wink: