Konwersja Array iteracji do wyrażenia lambda

Question

Mam poniżej kod, który wykonuje iteracje nad Cookies zresetować ciasteczko, którego nazwa pasuje CookieSession.NAME

Cookie[] cookies = httpServletRequest.getCookies(); 
     LOGGER.info("Clearing cookies on welcome page"); 
     if (cookies != null) 
      for (Cookie cookie : cookies) { 
       if (cookie.getName().equals(CookieSession.NAME)) {      
       cookie.setValue(null); 
       cookie.setMaxAge(0); 
       cookie.setPath("/"); 
       httpServletResponse.addCookie(cookie); 
       } 
      } 

może ktoś uprościć używając Java 8 lambda wyrażenia

Answer 1

Nie wiem, czy to będzie zostać uproszczone, ale można to zrobić tak:

Arrays.stream(cookies) 
     .filter(c -> c.getName().equals(CookieSession.NAME)) 
     .forEach(cookie -> { 
      cookie.setValue(null); 
      cookie.setMaxAge(0); 
      cookie.setPath("/"); 
      httpServletResponse.addCookie(cookie); 
     }); 

Answer 2

pętla for mogłyby zostać zastąpione forEach wyrażenie:

Arrays.stream(cookies) 
     .filter(c -> c.getName().equals(CookieSession.NAME)) 
     .forEach(c -> {c.setValue(null); 
        c.setMaxAge(0); 
        c.setPath("/"); 
        httpServletResponse.addCookie(c); 
        }); 

Answer 3

Arrays.stream(httpsServletRequest.getCookies()) 
    .filter(cookie -> CookieSession.NAME.equals(cookie.getName())) 
    .forEach(cookie -> { 
     cookie.setValue(null); 
     cookie.setMaxAge(0); 
     cookie.setPath("/"); 
     httpServletResponse.addCookie(cookie); 
    }); 

Answer 4

Inne odpowiedzi wydają się być ignorowane if (cookies != null). Również lubię peek dla kilku operacji pośrednich, a nie blok. Wydaje mi się jaśniejszy.

Optional.ofNullable(httpServletRequest.getCookies()) 
    .ifPresent(cookies -> Arrays.stream(cookies) 
     .filter(cookie -> cookie.getName().equals(CookieSession.NAME)) 
     .peek(cookie -> cookie.setValue(null)) 
     .peek(cookie -> cookie.setMaxAge(0)) 
     .peek(cookie -> cookie.setPath("/")) 
     .forEach(httpServletResponse::addCookie)); 

Answer 5

Nie można tego uprościć za pomocą lambda lub forEach.

Ponadto, wiele osób uważa, że ​​nie należy używać forEach mutować stanu w linii tak:

cookies.forEach(cookie -> httpServletResponse.addCookie(cookie)); 

Jednak jest to naprawdę kwestia stylu. Dopóki forEach zużywa elementy sekwencyjnie (a nie równolegle), nic nie może pójść źle w wyniku takiej linii.

Here is Oracle's advice on the subject.

Moim zdaniem, należy pozostawić swój kod w niezmienionym stanie.