Najlepsza praktyka sprawdzania poprawności pustej i pustej kolekcji w Javie

Chcę sprawdzić, czy kolekcja jest pusta i null. Czy ktoś mógłby dać mi znać najlepszą praktykę.

Obecnie sprawdzam jak poniżej:

if (null == sampleMap || sampleMap.isEmpty()) {
  // do something
} 
else {
  // do something else
}

Jeśli używasz biblioteki Apache Commons Kolekcje w swoim projekcie, możesz użyć metod CollectionUtils.isEmptyi MapUtils.isEmpty(), które odpowiednio sprawdzają, czy kolekcja lub mapa jest pusta, czy pusta (tzn. Czy są „zerowo bezpieczne”).

Kod tych metod jest mniej więcej tym, co napisał użytkownik @icza w swojej odpowiedzi.

Niezależnie od tego, co robisz, pamiętaj, że im mniej kodu piszesz, tym mniej kodu musisz przetestować, ponieważ złożoność kodu maleje.

To najlepszy sposób, aby to sprawdzić. Możesz napisać metodę pomocniczą, aby to zrobić:

public static boolean isNullOrEmpty( final Collection< ? > c ) {
    return c == null || c.isEmpty();
}

public static boolean isNullOrEmpty( final Map< ?, ? > m ) {
    return m == null || m.isEmpty();
}

Jeśli używasz ram Spring, możesz użyć CollectionUtilsdo sprawdzenia zarówno kolekcji (lista, tablica), jak i mapy itp.

if(CollectionUtils.isEmpty(...)) {...}

Osobiście wolę używać pustych kolekcji zamiast nulli mieć algorytmy działające w taki sposób, że dla algorytmu nie ma znaczenia, czy kolekcja jest pusta, czy nie.

Kiedy używasz wiosny, możesz użyć

boolean isNullOrEmpty = org.springframework.util.ObjectUtils.isEmpty(obj);

gdzie obj to dowolna [mapa, kolekcja, tablica, wszystko ...]

w przeciwnym razie: kod to:

public static boolean isEmpty(Object[] array) {
    return (array == null || array.length == 0);
}

public static boolean isEmpty(Object obj) {
    if (obj == null) {
        return true;
    }

    if (obj.getClass().isArray()) {
        return Array.getLength(obj) == 0;
    }
    if (obj instanceof CharSequence) {
        return ((CharSequence) obj).length() == 0;
    }
    if (obj instanceof Collection) {
        return ((Collection) obj).isEmpty();
    }
    if (obj instanceof Map) {
        return ((Map) obj).isEmpty();
    }

    // else
    return false;
}

dla String najlepiej jest:

boolean isNullOrEmpty = (str==null || str.trim().isEmpty());

Jeśli chcesz sprawdzić wartość null, to jest właśnie ten sposób. Jeśli jednak masz nad tym kontrolę, po prostu zwróć pustą kolekcję, kiedy tylko możesz, i sprawdź tylko później, czy jest pusta.

Wątek ten dotyczy mniej więcej tego samego języka C #, ale zasady dotyczą równie dobrze Java. Jak wspomniano powyżej, wartość null powinna zostać zwrócona tylko wtedy, gdy

• null może oznaczać coś bardziej szczegółowego;
• Twój interfejs API (umowa) może zmusić Cię do zwrócenia wartości null.

Możesz użyć metody org.apache.commons.lang.Validate„ notEmpty ”:

Validate.notEmpty(myCollection)-> Sprawdź, czy podana kolekcja argumentów nie ma wartości null ani nie ma wartości zero (brak elementów); w przeciwnym razie zgłasza wyjątek.

Sprawdzimy, czy obiekt Collection jest pusty, zerowy lub nie. wszystkie te metody podane poniżej znajdują się w pakiecie org.apache.commons.collections4.CollectionUtils.

Sprawdź na liście lub ustaw typ kolekcji obiektów.

CollectionUtils.isEmpty(listObject);
CollectionUtils.isNotEmpty(listObject);

Sprawdź typ obiektów na mapie.

MapUtils.isEmpty(mapObject);
MapUtils.isNotEmpty(mapObject);

Zwrócony typ wszystkich metod ma wartość logiczną.

Dla wszystkich kolekcji, w tym mapy użyj: isEmptymetody, która znajduje się na tych obiektach kolekcji. Ale musisz zrobić kontrolę zerową przed:

Map<String, String> map;

........
if(map!=null && !map.isEmpty())
......