refactor: collectingAndThen 사용하여 바로 반환

Author: sang5c

src/main/java/nextstep/ladder/domain/Lines.java

@@ -1,9 +1,11 @@
 package nextstep.ladder.domain;
 
 import java.util.List;
-import java.util.stream.Collectors;
 import java.util.stream.IntStream;
 
+import static java.util.stream.Collectors.collectingAndThen;
+import static java.util.stream.Collectors.toList;
+
 public class Lines {
 
     private final List<Line> lines;
@@ -20,10 +22,9 @@ private void validate(List<Line> lines) {
     }
 
     public static Lines of(int numberOfPlayers, int height) {
-        List<Line> lines = IntStream.range(0, height)
+        return IntStream.range(0, height)
                 .mapToObj(i -> LineBuilder.buildWithRandomPoints(numberOfPlayers))
-                .collect(Collectors.toList());
-        return new Lines(lines);
+                .collect(collectingAndThen(toList(), Lines::new));
     }
 
     public int getHeight() {

src/main/java/nextstep/ladder/domain/Players.java

@@ -1,9 +1,11 @@
 package nextstep.ladder.domain;
 
 import java.util.List;
-import java.util.stream.Collectors;
 import java.util.stream.IntStream;
 
+import static java.util.stream.Collectors.collectingAndThen;
+import static java.util.stream.Collectors.toList;
+
 public class Players {
     private final List<Player> players;
 
@@ -19,11 +21,9 @@ private void validate(List<Player> players) {
     }
 
     public static Players of(List<String> playerNames) {
-        List<Player> players = IntStream.range(0, playerNames.size())
+        return IntStream.range(0, playerNames.size())
                 .mapToObj(index -> new Player(index, playerNames.get(index)))
-                .collect(Collectors.toList());
-
-        return new Players(players);
+                .collect(collectingAndThen(toList(), Players::new));
     }
 
     public int count() {