Hvad skal der laves om?

Hvis firstName er en int kan du ikke bruge compareTo men skal bruge < og >.

compareTo sammenligner "Objecter", "int" er en primitiv datatype

Jeg formoder problemet ligger her:  return this.age.compareTo(p.age);

Prøv at rette til noget a la:  return new Integer(this.age).compareTo( new Integer(p.age) );

Okay - delte meninger om hvor problemet ligger, men enige alligevel :-)

carsten, smid et svar:-) Det virkede!

Kommer her!

Der er kund de 2 objekter der sammenlignes så hvorfor skriver du 'while'
        while(this.lastName.compareTo(p.lastName)==0)
med korrekte indrykninger kan du se hvorfor det er misvisende:

    public int compareTo(Object o)
    {
        Person p = (Person)o;
        if (this.lastName.compareTo(p.lastName)==0)
        {
            if(this.firstName.compareTo(p.firstName)==0)
                return new Integer(this.age).compareTo( new Integer(p.age) );
            return this.firstName.compareTo(p.firstName);     
        }
        return this.lastName.compareTo(p.lastName);  // denne linie manglede
    }

men jeg ville hellere lave det lidt mere forståeligt:

    public int compareTo(Object o)
    {
        Person p = (Person)o;
        if (this.lastName.compareTo(p.lastName)!=0)
            return this.lastName.compareTo(p.lastName);

                        // hvis lastname var ens skal vi kikke videre   
        if(this.firstName.compareTo(p.firstName)!=0)
            return this.firstName.compareTo(p.firstName);
     
                      // og hvis både last og first er ens er det alderen
        return new Integer(this.age).compareTo( new Integer(p.age) );
    }

mvh JakobA