Prüfen Sie, ob alle Werte in einer Karte sind null

Brauche ich, um zu überprüfen, ob alle Werte in einer Karte sind null, ich habe diese Methode, die ich ersetzen wollen durch einen built-in ein, wenn möglich. Einschränkungen: Java 5 und Zugriff auf Apache-Commons-Bibliotheken.

/* Checks if all values are null */
public static boolean isEmpty(Map<Dboid,?> aMap){

    boolean isEmpty = true;

    Iterator<?> it = aMap.entrySet().iterator();

    while(it.hasNext() && isEmpty){
        Object value = it.next();
        if(value != null) {
            isEmpty = false;
        }
    }

    return isEmpty;
}
  • was meinst du mit native hier?
  • ich denke, OP native bedeutet, seine eigene Methode
  • Er meinte: "built-in", z.B. Teil des Map API, oder einen anderen one-liner.
  • Dies sollte nicht benannt werden isEmpty, da gibt es Tasten, aber die befestigt werden möglicherweise Werte null.
  • und sollten Sie brechen, sobald Sie Sie finden, ein nicht-null-Wert..
  • Er tut. Sehen isEmpty Bedingung in der Schleife.
  • Durchlaufen entrySet() ist immer wieder Map.Entry Objekte nie null, selbst wenn der Schlüssel oder der Wert in der map-Eintrag passiert null.
  • du hast Recht, es ist ein Fehler in meinem code
  • das tut er nicht
  • Er... er hat. Wenn value != null, isEmpty eingestellt ist false, und die Schleife wird nicht fortgesetzt (while (... && isEmpty)). Keine weiteren tests werden durchgeführt, sobald eine nicht-null Wert gefunden wird.
  • hmm du hast Recht, nicht sehen, die isEmpty-Zustand..

InformationsquelleAutor Lluis Martinez | 2014-03-14
  1. 3

    Als solche gibt es keine direkte Methode für diese, aber Sie können Apache Commons CollectionUtils.countMatches() - Methode, und übergeben Sie ein NullPredicate Instanz zu. Natürlich, Sie würde das tun, übergeben Sie die Werte in die map mit Map#values() Methode:

    public static <K, V> boolean hasAllNullValues(Map<K, V> map) {
    int size = map.size();
    return CollectionUtils.countMatches(map.values(), NullPredicate.INSTANCE) == size;
}

    oder noch besser, verwenden Sie CollectionUtils.exists() Methode, um zu überprüfen, gibt es mindestens ein element erfüllt, dass die NotNullPredicate übergeben, als zweites argument:

    public static <K, V> boolean hasAllNullValues(Map<K, V> map) {
    return !CollectionUtils.exists(map.values(), NotNullPredicate.INSTANCE);
}
    InformationsquelleAutor Rohit Jain
  2. 6

    Andere Lösung ohne Verwendung von Bibliotheken von Drittanbietern. 

    Collections.frequency(aMap.values(), null) == aMap.size()
    InformationsquelleAutor kapex
  3. 3

    Ich weiß, die Frage ist für Java 5, aber für diejenigen, die hierher kommen aus der google-Suche, wie ich es Tat:

    Für Java - >= 8, die Sie tun können:

    boolean allValuesAreNull = yourMap.values()
        .stream()
        .allMatch(Objects::isNull);

    mit einer nuance: es wird true für leere Karte.

    InformationsquelleAutor PetroCliff
  4. 2

    Es gibt keine integrierte Methode, dies zu tun. Insbesondere, es gibt nichts, dass ein Mittel "zu finden, ein element, das nicht gleich etwas".

    Allerdings, wenn eine Karte, die enthält nur null Werte definiert ist, die durch Ihre business-Regeln, die auf "leer", das scheint zu implizieren, dass null Werte bedeuten "nicht vorhanden", in welchem Fall Sie wollen, zu konstruieren Sie den code so, dass null Werte sind nie aufgenommen in den ersten Platz. Dann können Sie einfach die eingebaute isEmpty().

    InformationsquelleAutor Jason C
  5. 2

    wie etwa

    return CollectionUtils.find(aMap.values(),NotNullPredicate.INSTANCE).isEmpty();
    • Sieht entspricht Rohit Antwort.
    • Ja, aber das ist besser, zurückzukehren !CollectionUtils.vorhanden ist(in der Karte.Werte(), NotNullPredicate.INSTANZ);
    InformationsquelleAutor Abhijeet Kushe
  6. 1

    Gibt es keine API geben wird, dass Sie, aber könnten Sie die Optimierung, die Methode ein wenig.


    Keine Notwendigkeit, überprüfen Sie die isEmpty-variable auf jeder iteration.
    Das ist eine kleine Optimierung.

    /* Checks if all values are null */
public static <K,V> boolean isMapEmpty(Map<K,V> aMap){
    for (V v: aMap.values()) {
      if (v != null) { return false; }
    }
    return true;
}
    • typpo - got to be aMap.values(); Die Optimierung besteht darin, dass er nicht brauchen eine zusätzliche Variablen und zusätzliche Kontrolle jeder iteration .. das summiert sich.
    • hey, was die downvote für? wollen einfach nur, um zu helfen, den Kerl
    • Die meisten wahrscheinlich, es addiert sich nicht, und hat null Effekt. Bitte geben Sie benchmarks und Profil-Daten zu sichernden Forderung, sowie ein Vergleich mit der OP, die eigentliche performance-Anforderungen, bevor darauf hindeutet vorzeitige Mikro-Optimierungen, wie ein Gültiger Ansatz. Ermutigend ist, dass Denkprozess schadet mehr als es hilft, es ist genau das, warum wir so viele schlechte "performance" - Fragen abzuwehren, und hier.
    • Es ist eine kompaktere version von meinem aktuellen code.
    • Ich Stimme zu - es ist ein Mikro-Optimierung und Reinigung der code. Und das ist alles was ich angeboten... vielen Dank für die upvote.
    • überprüfen Sie die isEmpty-flag in der while-Schleife.
    • ja dein code hat, aber Sie nicht wirklich brauchen es. Sobald Sie sehen nicht den Wert null, können Sie verlassen auf der Stelle.
    • Ja, aber manchmal Folge ich der einzige return am Ende der Methode Technik. Nicht immer, obwohl 🙂

    InformationsquelleAutor Deian
Schreibe einen Kommentar