Die Prüfung, ob eine Liste leer ist, in java 8

Ich bin neu in Java8. Ich habe implementiert, Listen und Filter. Ich habe eine null-check in meinem code. Ich würde schätzen etwas Hilfe mit, wie Sie überprüfen, wenn die Liste nicht leer ist, in der gleichen code-snippet. Wenn die Liste nicht leer ist, dann sollte der code gehen Sie mit der Behauptung.

    list.stream().filter(listElement -> listElement != null).
    forEach((listElement) -> Assert.assertEquals(
        ValueType.CANDY,
        listElement.getCandyType()));

InformationsquelleAutor Drools_user | 2015-12-22

20

Du fragst eine obsolete Frage. Streams verarbeiten alle Elemente einer Quelle, daher, falls es keine Elemente, keine Maßnahmen ergriffen werden. Daher brauchen Sie nicht zu überprüfen, ob die Liste leer ist.

Immer noch, können Sie den code vereinfachen:
```
list.stream().filter(Objects::nonNull)
    .map(listElement -> listElement.getCandyType())
    .forEach(candyType -> Assert.assertEquals(ValueType.CANDY, candyType));
```
oder
```
Assert.assertTrue(list.stream().filter(Objects::nonNull)
                      .map(listElement -> listElement.getCandyType())
                      .allMatch(Predicate.isEqual(ValueType.CANDY));
```
allMatch folgt den Regeln erforderlich ist, die für diese Prüfung. Wenn es keine Elemente gibt es keine unvereinbaren element, daher alle match -. Beachten Sie, dass listElement -> listElement.getCandyType() auch ersetzt werden kann durch ein Verfahren, Verweis auf die form ClassName::getCandyType; ich habe es nicht hier, wie ich weiß nicht die richtige Klasse name.

Gibt es keine performance-Unterschied zwischen den beiden Varianten. allMatch wird sofort wieder bei der Begegnung mit dem ersten nicht-übereinstimmenden element, und assertEquals werfen auf dem ersten nicht übereinstimmenden Elemente. Im zweiten Fall, der stack-trace nicht angezeigt werden Artefakte der stream-API-Implementierung.

Die zweite ist vorzuziehen, wenn Sie die Prüfungen für die Produktion von code eher als unit-tests und lassen Sie diese Prüfungen aus, wie mit der Java-Sprache assert Funktion, z.B.
```
assert list.stream().filter(Objects::nonNull)
           .map(listElement -> listElement.getCandyType())
           .allMatch(Predicate.isEqual(ValueType.CANDY));
```
da diese form garantiert, dass es keinen overhead, wenn assertions abgeschaltet sind, in der Erwägung, dass die erste Variante, mit einer assert - Anweisung mit der forEach Aktion immer noch bewirken, dass der Iteration über alle Elemente und Durchführung der zwischen-Schritte der pipeline.
- Der Nachteil deines Ansatzes ist, dass im Falle des Scheiterns der original-code anzeigen eines tatsächlichen nicht-übereinstimmenden Wert zu suchen, wie "erwartete SÜßIGKEITEN, aber war: SHMANDY", während der code würde nur sagen, dass die assertion fehlgeschlagen ist.
- Valeev: meine erste Variante noch funktioniert. Aber es ist fraglich, ob diese Informationen wirklich hilft. Vielleicht ist der index wäre hilfreich, vielleicht ist die andere nicht-übereinstimmende Elemente, wenn es einige... in der Regel, die Geschichte, die led des fehlerhaften Zustand ist, was Sie brauchen, aber das ist etwas, weder Variante liefern kann.
- Danke.Ich wählte Ihre Antwort.Es war sehr klar und verbesserte sich auf meinen code zu.
InformationsquelleAutor Holger

Die gewählte Antwort ist Super, nur eine kleine Anregung zum behandeln von null-Fall mit Optional.ofNullable des neu eingeführt Java8 Optional Klasse:

  Optional.ofNullable(list)
            .orElseGet(Collections::emptyList)
            .stream().filter(Objects::nonNull)
            .map(listElement -> listElement.getCandyType())
            .forEach(candyType -> Assert.assertEquals(ValueType.CANDY, candyType)););

InformationsquelleAutor Stas

7

Check-out die isEmpty Methode
```
if (list.isEmpty()) { ... }
```
Hier ist der doc: https://docs.oracle.com/javase/7/docs/api/java/util/List.html#isEmpty()
- Danke aber ich wollte wissen, die syntax, wie es könnte geprüft werden, in der Strom von Java8
- Wenn die Liste leer ist, dann ist die Assertion wird nicht passieren, jedenfalls, da es keine Elemente, die es zu Durchlaufen
InformationsquelleAutor Buddy
1

Behaupten, dass die Liste leer ist, ist eine andere Sache durchzusetzen, dass ein bestimmtes element der Liste hat eine form (d.h. getCandyType() == ValueType.CANDY). Versuchen, binden Sie die zwei Ideen, die immer zusammen ist einfach nicht der richtige Weg, Dinge zu tun.

Was Sie tun müssen, ist, um eine zweite Behauptung, die entweder vor oder nach der forEach, die nicht
```
assertThat(list.isEmpty(), not(equalTo(Boolean.FALSE)))
```
... oder etwas ähnliches.

Einer der wichtigsten Punkte, über leere Listen ist, dass Sie verarbeitet werden können in der gleichen Weise wie Listen, die Elemente enthalten. Dies ist, warum Dinge wie die Optional Klasse und die Null-Muster vorhanden. Also, wenn Sie wirklich darauf, dass die Liste leer ist Sie wirklich sollte machen es eine eigenständige Behauptung.
- Wenn diese Methoden das tun, was Ihre Namen suggerieren, die not(equalTo(Boolean.FALSE)) ist durchaus eine verschleierte Art und Weise zu sagen: true...
- Ich denke, dass Sie auch in der Lage, format-Fehlermeldung automatisch (so etwas wie "es war nicht gleich FALSCH"). Obwohl diese Ausführlichkeit ist absolut unnötig hier.
- du hast Recht, die beste Behauptung würde wahrscheinlich verwenden die hamcrest IsEmptyCollection.empty() matcher - assertThat(list, not(empty())).
InformationsquelleAutor sisyphus

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.