Java Stream API -> Optional<String> get

Question

Hej

Ćwiczę sobie stream API z Javy.

Wymyśliłem sobie taki zadanie - mam listę Pracowników (Klasa Employee ma pola: String name, String surname, LocalDate birthDate, boolean hired, boolean male, Position position, private int salary) - ale to nie ważne (chodzi tylko o podgląd klasy.)

 

W tej liście jest kilkanaście osób. Chciałbym za pomocą API stream wyciągnąć imię kobiety, która ma największą pensję w firmie. Oto moje rozwiązanie:

 

String maxSalaryWomanName = employeeList.stream()
        .filter(employee -> !employee.isMale())
        .max(Comparator.comparing(Employee::getSalary))
        .map(Employee::getName)
        .get();

 

Jest ok, ale Intellij podkreśla że użycie get() z opcjonala [jak nie ma żadnej kobiety to jest wyjątek:/ ], bez sprawdzenia isPresent(). Można zrobić to jakoś lepiej? Staram się oglądać sporo przykładów z tego streamApi, bo czasem fajnie się przydaje do działań na kolekcjach, ale trochę nie rozumiem tych bardziej zaawansowanych "zapytań".

Comments

To zależy. Co chcesz zrobić z tą osobą która ma najwięcej pieniędzy?

Answer 1

Wszystko zależy od tego, do czego Ci byłoby potrzebny rezultat streamu. W dokumentacji masz kilka metod które mógłbyś użyć, w zależności od zastosowania. 

W przypadku gdy nieznalezienie obiektu, np. podczas pobieranie obiektu z bazy danych w celu jego zmiany możesz zastosować orElseThrow - fakt, nie jest to do końca poprawne zastosowanie wyjątków, poczytaj o "coding by exception". 

W przypadku gdy możesz zastosować jakąś defaultową wartość możesz wykorzystać metodę: orElseGet().Ze względu na to, że Java nie wspiera możliwości zwracania kilku rezultatów (jak np. Go), możesz też to wykorzystać tworząc jakąś klasę która wrappuje dwa obiekty: Error i Obiekt (albo wykorzystać vavrowy Either), a następnie w getOrDefault() zwrócić jakiś obiekt zawierający kod błędu. 

Na upartego, możesz też zastosować ifa z isPresent() ale lepiej zastosować którąś z wcześniej wspomnianych metod - bardziej funkcyjne podejście.

https://docs.oracle.com/javase/8/docs/api/java/util/Optional.html

Comments

"coding by exception"

Masz może jakieś sprawdzone artykuły/ksiązki? 

---

Wszystko to, co da się znaleźć np. w Google pod frazą "Coding by exception" czy "Exception Driven Development". Fakt, dużo z tych artykułów jest dość starych, ale imo dalej powinny być aktualne - tutaj np. jest dyskusja pod jednym z postów na https://softwareengineering.stackexchange.com/questions/189222/are-exceptions-as-control-flow-considered-a-serious-antipattern-if-so-why . 

Czy jest to sprawdzone? Nie wiem, ale według mnie, patrząc na argumenty przytoczone w wątku - dużo racji w tym jest, że nie powinno się stosować wyjątków w ten sposób.

Answer 2

Wg. mnie to powinieneś owrapować zmienną w optional Optional<String> maxSalaryWomanName. Dalej w kodzie używając tej zmiennej możesz korzystać z ifPresent lub (jak pozwoli werssja Javy) ifPresentOrElse. Jak dopuszczasz sytuacje w której przypisujesz wartość domyślną to skorzystaj z orElseGet/orElse.
Unikaj używania get() na optionalu.

Answer 3

Jeśli chcesz to tylko wyświetlić w konsoli to tak będzie najprościej:

employeeList.stream()
        .filter(employee -> !employee.isMale())
        .max(Comparator.comparing(Employee::getSalary))
        .map(Employee::getName)
        .ifPresent(System.out::println);