refactor: use more concise APIs to collect streams (#5031)

Author: vursen

articles/flow/component-internals/visibility.adoc

@@ -52,9 +52,7 @@ container.add(label);
 
 // prints 1 - the server-side structure is preserved
 // regardless of whether the component is visible or not
-System.out.println("Number of children: "
-        + container.getChildren().collect(
-                Collectors.counting()));
+System.out.println("Number of children: " + container.getChildCount());
 ----
 ====
 

articles/flow/routing/registered-routes.adoc

@@ -27,7 +27,7 @@ To retrieve all the routes defined by parent layout, use:
 List<RouteData> routes = RouteConfiguration.forSessionScope().getAvailableRoutes();
 List<RouteData> myRoutes = routes.stream()
         .filter(routeData -> MyParentLayout.class.equals((routeData.getParentLayout())))
-        .collect(Collectors.toList());
+        .toList();
 ----
 
 

src/main/java/com/vaadin/demo/component/datepicker/DatePickerIndividualInputFields.java

@@ -14,7 +14,6 @@
 
 import java.time.LocalDate;
 import java.time.Month;
-import java.util.stream.Collectors;
 import java.util.stream.IntStream;
 
 @Route("date-picker-individual-input-fields")
@@ -29,8 +28,7 @@ public DatePickerIndividualInputFields() {
         LocalDate now = LocalDate.now(ZoneId.systemDefault());
 
         List<Integer> selectableYears = IntStream
-                .range(now.getYear() - 99, now.getYear() + 1).boxed()
-                .collect(Collectors.toList());
+                .range(now.getYear() - 99, now.getYear() + 1).boxed().toList();
 
         yearPicker = new ComboBox<>("Year", selectableYears);
         yearPicker.setWidth(6, Unit.EM);
@@ -80,8 +78,8 @@ private void updateDayPicker() {
                 monthPicker.getValue(), 1);
         int lengthOfMonth = startOfMonth.lengthOfMonth();
 
-        dayPicker.setItems(IntStream.range(1, lengthOfMonth + 1).boxed()
-                .collect(Collectors.toList()));
+        dayPicker.setItems(
+                IntStream.range(1, lengthOfMonth + 1).boxed().toList());
     }
     // end::snippet[]
 

src/main/java/com/vaadin/demo/component/grid/GridDragDropFilters.java

@@ -26,12 +26,10 @@ public class GridDragDropFilters extends Div {
     public GridDragDropFilters() {
 
         List<Person> people = DataService.getPeople();
-        managers = people.stream().filter(Person::isManager)
-                .collect(Collectors.toList());
+        managers = people.stream().filter(Person::isManager).toList();
         staffGroupedByMangers = people.stream()
                 .filter(person -> person.getManagerId() != null)
-                .collect(Collectors.groupingBy(Person::getManagerId,
-                        Collectors.toList()));
+                .collect(Collectors.groupingBy(Person::getManagerId));
 
         // tag::snippet[]
         TreeGrid<Person> treeGrid = setupTreeGrid();

src/main/java/com/vaadin/demo/component/grid/GridHeaderFooterStyling.java

@@ -7,7 +7,6 @@
 import com.vaadin.flow.router.Route;
 
 import java.util.List;
-import java.util.stream.Collectors;
 
 @Route("grid-header-footer-styling")
 public class GridHeaderFooterStyling extends Div {
@@ -35,8 +34,7 @@ public GridHeaderFooterStyling() {
 
     private static List<PersonWithRating> createDataSet() {
         return DataService.getPeople().stream()
-                .map(PersonWithRating::generateFromPerson)
-                .collect(Collectors.toList());
+                .map(PersonWithRating::generateFromPerson).toList();
     }
 
     public static class Exporter // hidden-source-line

src/main/java/com/vaadin/demo/component/grid/GridStyling.java

@@ -7,7 +7,6 @@
 import com.vaadin.flow.router.Route;
 
 import java.util.List;
-import java.util.stream.Collectors;
 
 @Route("grid-styling")
 public class GridStyling extends Div {
@@ -39,8 +38,7 @@ public GridStyling() {
 
     private static List<PersonWithRating> createDataSet() {
         return DataService.getPeople().stream()
-                .map(PersonWithRating::generateFromPerson)
-                .collect(Collectors.toList());
+                .map(PersonWithRating::generateFromPerson).toList();
     }
 
     public static class Exporter // hidden-source-line

src/main/java/com/vaadin/demo/component/treegrid/TreeGridDragDrop.java

@@ -29,8 +29,7 @@ public TreeGridDragDrop() {
         managers = people.stream().filter(Person::isManager).toList();
         staffGroupedByMangers = people.stream()
                 .filter(person -> person.getManagerId() != null)
-                .collect(Collectors.groupingBy(Person::getManagerId,
-                        Collectors.toList()));
+                .collect(Collectors.groupingBy(Person::getManagerId));
 
         // tag::snippet[]
         TreeGrid<Person> treeGrid = setupTreeGrid();

src/main/java/com/vaadin/demo/fusion/security/authentication/UserInfoService.java

@@ -8,7 +8,6 @@
 import org.springframework.security.core.context.SecurityContextHolder;
 
 import java.util.List;
-import java.util.stream.Collectors;
 
 /**
  * Provides information about the current user.
@@ -24,8 +23,7 @@ public UserInfo getUserInfo() {
                 .getAuthentication();
 
         final List<String> authorities = auth.getAuthorities().stream()
-                .map(GrantedAuthority::getAuthority)
-                .collect(Collectors.toList());
+                .map(GrantedAuthority::getAuthority).toList();
 
         return new UserInfo(auth.getName(), authorities);
     }