java.util.optional z mapą, problem

Hej,

Mam problem z nową funkcjonalnością javy, z która nigdy nie miałem styczności, a muszę ją wykorzystać :(
A mianowicie mój problem:
mam taką oto funkcję:

 Optional<Map<Integer, Integer>> advisor(List<PatientExt> patients) {
Map<Integer, Integer> mapForOptional = new HashMap<Integer, Integer>();
		Optional<Map<Integer, Integer>> advisorMapOptional = new Optional<HashMap<Integer, Integer>>();
		// wtf
		
		
		return advisorMapOptional;
 }

i kompletnie nie wiem jak się za nią zabrać, próbowałem tak jak powyżej, ale mam błędy kompilacji.
Byłbym wdzięczny za jakiekolwiek sugestie, szukając w internecie nie znalazłem przykładów, które by pasowały do powyższego mojego problemu.

W skórcie:

1. jak zainicjalizować Optional mapą gdzie później Optional<Map<..,..>> zwrócę?
2. Pusty obiekt Optional jak będzie wyglądał, jak nic nie spełni moich warunków w funkcji advisor kiedy już będzie zaimplementowana?

Dzięki za pomoc :)

A dokumentacje, lub cokolwiek o Optional czytał ???.Optional.of itp

Nie robi się new Optional tylko Optional.of albo Optional.ofNullable albo Optional.empty

Czyli przykładowo
Optional.of( HashMap )
przypisuje do optional obiekt jak rozumiem ?

Swoją drogą dziwne miejsce jak dla mnie na opcjonala. Ja bym zwrócił mape (najwyżej po prostu pustą) zamiast optionala ;)
Edit:
dopiero zauważyłem że musisz ją wykorzystac chociaż w sumie zastanawiam się czemu :D

Optionale zostały wprowadzone żeby ograniczyć występowanie NullPointerException w programach. Optionalem opakowujesz obiekt lub jego brak - może na przykładzie łatwiej: dwie metody dla porównia z optionalem i bez - przypadek pesymistyczny mamy null-a;

public static Optional<Map<Integer, Integer>> advisor() {
		Map<Integer, Integer> mapForOptional = null;// new HashMap<Integer, Integer>();
		Optional<Map<Integer, Integer>> advisorMapOptional = 
				Optional.ofNullable(mapForOptional);

		return advisorMapOptional;
	}

public static Map<Integer, Integer> getMap() {
		Map<Integer, Integer> mapForOptional = null;
		return map;
	}

mamy tu null-a bo mapa nie została zainicjalizowana przez new HashMap<Integer, Integer>(); gdyby została zainicjalizowana wtedy nie byłoby null-a tylko pusta mapa. Optional.ofNullable(object) jest wygodny bo jeśli opakujemy nim null-a zwróci Optional.empty(), jeśli object nie jest null-em zwróci Optional.of(object).
I teraz możemy z tego z korzystać np tak:

Optional<Map<Integer, Integer>> optional = advisor();
Map<Integer, Integer> mapFromOptional = optional.orElse(new HashMap<Integer, Integer>());
System.out.println(mapFromOptional.size()); // 0

// bez optionala
Map<Integer, Integer> mapFromMethod = getMap();
System.out.println(mapFromMethod.size()); // NullPointerException 

mam nadzieję, że trochę rozjaśniłem

Przykro mi, ale twój argument jest inwalidą. Równie dobrze możesz Optional zainicjować nullem i mieć NPE.

Generalnie praktyka z Optionalem jest bardzo dobra, ale jego używanie go do opakowywania pustych kolekcji to nadużycie. Zupełnie nic nie zyskujesz w takiej sytuacji. W przypadku kolekcji dużo lepiej chornić się przed NPE z wykorzystaniem pustych kolekcji które de facto realizują Null Object Pattern.

@airborn inicjowanie optionala nullem powinno być wykrywane i autor powinien być poddawany natychmiastowej utylizacji ;)
Jeśli chodzi o Optional dla pustej kolekcji to nie do końca się zgodzę. Pusta kolekcja to jednak trochę coś innego niż brak kolekcji (sugerowany przez pusty Optional). Często faktycznie niczego to nie zmieni jak sobie na tej kolekcji robimy map czy filter, ale może się zdarzyć że robi nam różnice czy coś zostało zwrócone czy też nie. Więc nie robiłbym z tego jakiejś ogólnej zasady - trzeba po prostu myśleć i rozumieć co robimy i użyć jakiegoś Collections.emptyList() czy .emptyMap() kiedy ma to sens a Optional.empty() dla kolekcji, kiedy koniecznie chcemy sygnalizować że czegoś brakowało.

Z tą utylizacją to się jak najbardziej zgadzam ;)

Z NULLowaniem kolekcji już ciężej mi się zgodzić.W takiej sytuacji już zdecydowanie lepiej opakować kolekcję w jakąś klasę i zwracać implementacje interfejsu np. SearchResult i EmptyResult. Ale fakt, to zależy od konkretnego przypadku i może faktycznie czasem lepiej było by to zapakować w Optional, ale moim zdaniem to raczej rzadkie przepadki.

@airborn no ja całkowicie się zgadzam, że za świadome inicjalowanie/inicjowanie nulla Optionalem powinno być w dyby i do lochu :). Chodzi raczej o sytuację kiedy nie ma tej pewności czyli odpowiedź na pytanie Are you really sure that this variable can never ever be null nie jest twierdząca.
Temat jest akurat z kolekcją, mi chodziło ogólnie o ideę używania Optionala- jak już go dostajemy to tu powinna zapalić się lampka, że może być pusty i coś z tym trzeba zrobić i imo to jest ich największa zaleta - sygnalizowanie, że może być pusto.