Niedawno opanowałem wyrażenie Lambda, które zostało wprowadzone w java 8. Odkryłem, że ilekroć używam funkcjonalnego interfejsu, zwykle używam wyrażenia Lambda zamiast tworzyć klasę, która implementuje funkcjonalny interfejs.

Czy jest to uważane za dobrą praktykę? A może ich sytuacje, w których użycie Lambdy jako funkcjonalnego interfejsu jest nieodpowiednie?

Istnieje wiele kryteriów, które powinny skłonić Cię do rozważenia niestosowania lambda:

• Rozmiar Im większa lambda, tym trudniej jest podążać za otaczającą ją logiką.
• Powtórzenie Lepiej jest utworzyć nazwaną funkcję dla powtarzalnej logiki, chociaż dobrze jest powtarzać bardzo proste lambdy, które są rozłożone.
• Nazewnictwo Jeśli potrafisz wymyślić świetną nazwę semantyczną, powinieneś jej użyć, ponieważ zapewnia ona dużą przejrzystość kodu. Nie mówię imion jak priceIsOver100. x -> x.price > 100jest tak samo jasne jak ta nazwa. Mam na myśli, isEligibleVoterże takie imiona zastępują długą listę warunków.
• Zagnieżdżanie Zagnieżdżone lambdy są naprawdę bardzo trudne do odczytania.

Nie idź za burtę. Pamiętaj, że oprogramowanie można łatwo zmienić. W razie wątpliwości napisz to w obie strony i sprawdź, który z nich jest łatwiejszy do odczytania.

To zależy. Ilekroć znajdziesz tę samą lambdę w różnych miejscach, powinieneś rozważyć wdrożenie klasy, która implementuje interfejs. Ale gdybyś użył anonimowej klasy wewnętrznej, inaczej uważam, że lambda jest znacznie lepsza.

Popieram odpowiedź Karla Bielefeldta, ale chcę przedstawić krótkie uzupełnienie.

• Debugowanie Niektóre IDE zmagają się z zasięgiem w lambdzie i próbują wyświetlać zmienne składowe w kontekście lambda. Chociaż mam nadzieję, że ta sytuacja ulegnie zmianie, może być denerwujące utrzymywanie czyjegoś kodu, gdy jest on wypełniony lambdami.

Dostęp do zmiennych lokalnych obejmującego zakres

Przyjęta odpowiedź Karla Bielefeldta jest poprawna. Mogę dodać jeszcze jedno rozróżnienie:

• Zakres

Kod lambda zagnieżdżony w metodzie wewnątrz klasy może uzyskiwać dostęp do dowolnie efektywnych zmiennych końcowych znalezionych w tej metodzie i klasie.

Utworzenie klasy, która implementuje funkcjonalny interfejs, nie daje takiego bezpośredniego dostępu do stanu kodu wywołującego.

Aby zacytować Java Tutorial (podkreślenie moje):

Podobnie jak klasy lokalne i anonimowe, wyrażenia lambda mogą przechwytywać zmienne; mają taki sam dostęp do zmiennych lokalnych obejmującego zakres . Jednak w przeciwieństwie do klas lokalnych i anonimowych, wyrażenia lambda nie mają żadnych problemów z cieniowaniem (więcej informacji można znaleźć w części Cieniowanie). Wyrażenia lambda mają zasięg leksykalny. Oznacza to, że nie dziedziczą żadnych nazw z nadtypu ani nie wprowadzają nowego poziomu zakresu. Deklaracje w wyrażeniu lambda są interpretowane tak, jak w otaczającym środowisku.

Chociaż wyciąganie długiego kodu i nazywanie go ma wiele zalet, należy to porównać z prostotą bezpośredniego dostępu do stanu otaczającej metody i klasy.

Widzieć:

• Dostęp do lokalnych zmiennych w zakresie obejmującym ( samouczki Java autorstwa Oracle)
• Jaka jest korzyść ze skutecznego wykorzystania zmiennych końcowych w wyrażeniach lambda Java 8
• Różnica między finałem a efektywnie finałem

Może to być wybieranie nitów, ale do wszystkich innych doskonałych punktów zawartych w innych odpowiedziach dodałbym:

Preferuj referencje metod, jeśli to możliwe. Porównać:

employees.stream()
         .map(Employee::getName)
         .forEach(System.out::println);

przeciw

employees.stream()
         .map(employee -> employee.getName())
         .forEach(employeeName -> System.out.println(employeeName));

Użycie odwołania do metody pozwala uniknąć konieczności nazwania argumentów lambdy, które zwykle są zbędne i / lub prowadzą do leniwych nazw, takich jak elub x.

Opierając się na odpowiedzi Matta McHenry'ego, może istnieć związek między lambda a referencjami metod, przy czym lambda może być przepisana jako seria referencji metod. Na przykład:

employees.stream()
         .forEach(employeeName -> System.out.println(employee.getName()));

vs

employees.stream()
         .map(Employee::getName)
         .forEach(System.out::println);