CompareTo z nullami na końcu :: 4programmers.net

Rejestracja:ponad 11 lat
Ostatnio:ponad 2 lata
Lokalizacja:Kraków
Postów:233

Mam problem z metodą compareTo, chodzi o to, że mogą się pojawić nulle a te mają być na końcu. Analogicznie do order by ... nulls last. Proszę o pomoc bo albo mam gdzieś jeszcze coś źle albo to ta metoda, wyglądała tak a muszę zrobić, żeby nulle szły ostatnio i coś mi nie wychodzi.

Kopiuj

  
@Override
    public int compareTo(Object o) {
        if( getParam() == null) {
            return -1;
        }
        if ( o.getParam() == null ) {
            return 1;
        }
        return getParam().getDescription().compareTo(o.getParam().getDescription());
    }

edytowany 1x, ostatnio: Hepek 2014-03-05 14:20

szweszwe 2014-03-05 14:25

Ten pierwszy getParam() to bez żadnego obiektu?

Hepek 2014-03-05 14:26

z tego co wiem tam można dać this. ale tak też powinno być dobrze

Koziołek

2014-03-05 15:31

Koziołek2014-03-05 15:31

Moderator

Rejestracja:około 18 lat
Ostatnio:dzień
Lokalizacja:Stacktrace
Postów:6822

Nie masz rzutowania o na typ w ramach którego jest metoda.
compareTo jest generyczna po to by nie musieć używać Object i rzutowania
jeżeli getDescription zwróci null dla parametru do którego referencję trzyma obiekt na którym wywołujesz metodę to dostaniesz NPE.

A teraz co do implementacji:

Kopiuj

public int compateTo(TwojaKlasaWKtorejSiedziTaMetoda o){
   if(o==null || o.getParam() == null )
     return 1; // jesteśmy > niż null
   if(getParam()==null || getParam().getDescription()==null)
     return -1; // jesteśmy nullem i tym samym jesteśmy mniejsi
   return getParam().getDescription().compareTo(o.getParam().getDescription());
}

Hepek 2014-03-05 15:56

właśnie chodzi o to, że description też może być nullem i żeby wtedy ustawiało te nulle na końcu, nie da się uchronić od NPE?

Koziołek 2014-03-05 17:20

to tak jak ci napisałem. Jeżeli this.param.description jest null to zwracasz -1. Jeżeli nullem jest ten z o to obsługujesz na poziomie komparatora w description

Hepek 2014-03-07 06:40

niestety, dalej dostaje NPE

Koziołek 2014-03-07 07:30

w którym dokładnie miejscu

Hepek

2014-03-07 07:46

Hepek2014-03-07 07:46

Rejestracja:ponad 11 lat
Ostatnio:ponad 2 lata
Lokalizacja:Kraków
Postów:233

Poradziłem sobie w ten sposób:

Kopiuj

public int compareTo(ObiektKlasy... o) {
   
         if( getParam().getDescription() == null) {
            return 1;
        }
        if ( o.getParam().getDescription() == null ) {
            return -1;
        }
        
        if(getParam().getDescription() == null && o.getParam().getDescription() == null)
            return 0;
        else if(getParam().getDescription() > o.getParam().getDescription())
            return 1;
        else if(getParam().getDescription() < o.getParam().getDescription())
            return -1;

        return getParam().getDescription().compareTo(o.getParam().getDescription());
 
    }