Array sortieren

[newtojava90]

Folgendes Problem stellt sich mir:

Ich habe ein Array gefüllt mit Objekten des Typs xmleventtype.

Dieser Typ beinhaltet verschiedene Feldvariablen u.a. Text. Ich möchte nun das Array sortieren und zwar alphabetisch nach dem Text. Mir ist klar dass ich dazu die Methode CompareTo in der Klasse xmleventtype implementieren muss.

    public int compareTo( XmlEventType xmleventtype ) {
        if( text.compareTo(xmleventtype.text) == -1)
            return -1;
        if( text.compareTo(xmleventtype.text) == 1 )
            return 1;
            
        return 0;

jedoch funktioniert trotz der Implementierung die Sortierung nicht.

Es wäre sehr nett wenn mir jemand sagen kann wie man den Fehler der java.lang.NullPointerException
korrigieren könnte.

LG und Danke

 

[Der Müde Joe]

Irgendetwas wird wohl null sein.

Wenn man auf ein Objekt das null ist eine Methode aufruft, wird eine NullPointerException geschmissen.

In deinem Fall wird wohl this.text null sein. Dann rufst du text.compareTo() auf und es knallt.
Aber ohne den gesamten StackTrace (die ganze Fehlermeldung) ist das nur eine Vermutung.

 

[Domi F]

Neben der NullPointerException hat deine Methode aber noch ein anderes Problem. Gemäss JavaDoc gibt compareTo folgendes zurück:

a negative integer, zero, or a positive integer as this object is less than, equal to, or greater than the specified object.

Überlege dir mal was passiert, wenn nicht -1 oder 1 sondern ein anderer Wert zurückgegeben wird.

Du kannst dir die ganzen if-Abfragen sparen und direkt den Wert aus text.compareTo() zurückgeben, dieser ist ja entweder negativ, positiv oder null.

 

[hüteüberhüte]

return this.str.compareTo(other.str);

Vorausgesetzt: this.str != null

 

[newtojava90]

Danke für eure Hilfe!

Ich musste den Vergleichsoperator noch erweitern.

   public int compareTo( XmlEventType xmleventtype ) { //Sortiert absteigend nach dem Gerätenamen
    	if(this.text.compareTo(xmleventtype.text)>0){
    	   return this.text.compareTo(xmleventtype.text);}
    	   else if (this.text.compareTo(xmleventtype.text)<0){
    		   return this.text.compareTo(xmleventtype.text);}
    	   else (this.text.compareTo(xmleventtype.text)==0)
    	   return this.timestamp.compare(xmleventtype.timestamp);
    	   
    }

Nun meint "er" (der compiler) dass die letze else abfrage nicht stimmt und ein ; fehlt. Warum? Bin echt schon am verzweifeln

 

[DrZoidberg]

public int compareTo( XmlEventType xmleventtype ) { //Sortiert absteigend nach dem Gerätenamen
    if(this.text.compareTo(xmleventtype.text) == 0) {
       return this.timestamp.compare(xmleventtype.timestamp);
     } else {
       return this.text.compareTo(xmleventtype.text);
     }
}

 

[newtojava90]

Danke!
Habe es gerade selbst gesehen
Bin noch ziemlich müde
Danke