🚀 1단계 - 스트림, 람다, Optional (#2311)

* docs: 요구사항 추가

* feat: CarTest의 익명 클래스를 람다로 전환

* feat: sumAll, sumAllEven method 람다를 활용해 중복 제거

* feat: countWords method filter 적용

* feat: printLongestWordTop100 method filter 적용

* feat: sumOverThreeAndDouble 메서드 구현, printLongestWordTop100 메서드 구현

* feat: getUser 메소드 자바 8의 stream과 Optional을 활용해 구현

* feat: getUser 메소드 자바 8의 stream과 Optional을 활용해 구현

* feat: Expression의 of 메소드 구현

Author: gan-ta

README.md

@@ -6,4 +6,17 @@
 * 모든 피드백을 완료하면 다음 단계를 도전하고 앞의 과정을 반복한다.
 
 ## 온라인 코드 리뷰 과정
-* [텍스트와 이미지로 살펴보는 온라인 코드 리뷰 과정](https://github.com/nextstep-step/nextstep-docs/tree/master/codereview)
+* [텍스트와 이미지로 살펴보는 온라인 코드 리뷰 과정](https://github.com/nextstep-step/nextstep-docs/tree/master/codereview)
+
+## step1
+- [x] CarTest의 익명 클래스를 람다로 전환
+- [x] sumAll, sumAllEven method 람다를 활용해 중복 제거
+- [x] sumOverThreeAndDouble() 구현 - List에 담긴 모든 숫자 중 3보다 큰 숫자를 2배 한 후 모든 값의 합을 구한다 (map, reduce, filter를 활용)
+- [x] printLongestWordTop100() 메서드 구현 (map, reduce, filter를 활용)
+  - 단어의 길이가 12자를 초과하는 단어를 추출
+  - 12자가 넘는 단어 중 길이가 긴 순서로 100개의 단어를 추출
+  - 단어 중복을 허용하지 않는다. 즉, 서로 다른 단어 100개를 추출
+  - 출한 100개의 단어를 출력한다. 모든 단어는 소문자로 출력
+- [x] ageIsInRange1() 메소드 기능 Optional을 활용해 ageIsInRange2() 메소드에 구현
+- [x] getUser() 메소드를 자바 8의 stream과 Optional을 활용해 구현
+- [x] Expression의 of 메소드 구현

src/main/java/nextstep/fp/Condition.java

@@ -0,0 +1,5 @@
+package nextstep.fp;
+
+public interface Condition {
+  boolean test(Integer number);
+}

src/main/java/nextstep/fp/Lambda.java

@@ -27,30 +27,18 @@ public void run() {
     }
 
     public static int sumAll(List<Integer> numbers) {
-        int total = 0;
-        for (int number : numbers) {
-            total += number;
-        }
-        return total;
+        return sumAllByCondition(numbers, (number) -> true);
     }
 
     public static int sumAllEven(List<Integer> numbers) {
-        int total = 0;
-        for (int number : numbers) {
-            if (number % 2 == 0) {
-                total += number;
-            }
-        }
-        return total;
+        return sumAllByCondition(numbers, (number) -> number % 2 == 0);
     }
 
     public static int sumAllOverThree(List<Integer> numbers) {
-        int total = 0;
-        for (int number : numbers) {
-            if (number > 3) {
-                total += number;
-            }
-        }
-        return total;
+        return sumAllByCondition(numbers, (number) -> number > 3);
+    }
+
+    private static int sumAllByCondition(List<Integer> numbers, Condition condition) {
+        return numbers.stream().filter(condition::test).mapToInt(Integer::intValue).sum();
     }
 }

src/main/java/nextstep/fp/StreamStudy.java

@@ -15,19 +15,21 @@ public static long countWords() throws IOException {
                 .get("src/main/resources/fp/war-and-peace.txt")), StandardCharsets.UTF_8);
         List<String> words = Arrays.asList(contents.split("[\\P{L}]+"));
 
-        long count = 0;
-        for (String w : words) {
-            if (w.length() > 12) count++;
-        }
-        return count;
+        return words.stream().filter(w -> w.length() > 12).count();
     }
 
     public static void printLongestWordTop100() throws IOException {
         String contents = new String(Files.readAllBytes(Paths
                 .get("src/main/resources/fp/war-and-peace.txt")), StandardCharsets.UTF_8);
         List<String> words = Arrays.asList(contents.split("[\\P{L}]+"));
 
-        // TODO 이 부분에 구현한다.
+         words.stream()
+             .filter(word -> word.length() > 12)
+             .sorted((s1, s2) -> Integer.compare(s2.length(), s1.length()))
+             .distinct()
+             .limit(100)
+             .map(String::toLowerCase)
+             .forEach(System.out::println);
     }
 
     public static List<Integer> doubleNumbers(List<Integer> numbers) {
@@ -39,6 +41,6 @@ public static long sumAll(List<Integer> numbers) {
     }
 
     public static long sumOverThreeAndDouble(List<Integer> numbers) {
-        return 0;
+        return numbers.stream().filter(number -> number > 3).map(number -> number * 2).reduce(0, Integer::sum);
     }
-}
+}

src/main/java/nextstep/optional/Expression.java

@@ -1,5 +1,7 @@
 package nextstep.optional;
 
+import java.util.Arrays;
+
 enum Expression {
     PLUS("+"), MINUS("-"), TIMES("*"), DIVIDE("/");
 
@@ -14,12 +16,9 @@ private static boolean matchExpression(Expression e, String expression) {
     }
 
     static Expression of(String expression) {
-        for (Expression v : values()) {
-            if (matchExpression(v, expression)) {
-                return v;
-            }
-        }
-
-        throw new IllegalArgumentException(String.format("%s는 사칙연산에 해당하지 않는 표현식입니다.", expression));
+        return Arrays.stream(values())
+            .filter(value -> matchExpression(value, expression))
+            .findFirst()
+            .orElseThrow(() -> new IllegalArgumentException(String.format("%s는 사칙연산에 해당하지 않는 표현식입니다.", expression)));
     }
 }

src/main/java/nextstep/optional/User.java

@@ -1,5 +1,7 @@
 package nextstep.optional;
 
+import java.util.Optional;
+
 public class User {
     private String name;
     private Integer age;
@@ -33,7 +35,15 @@ public static boolean ageIsInRange1(User user) {
     }
 
     public static boolean ageIsInRange2(User user) {
-        return false;
+        Optional<User> optionalUser = Optional.ofNullable(user);
+        if(optionalUser.isEmpty()){
+            return false;
+        }
+        return optionalUser
+            .filter(targetUser -> targetUser.getAge() != null)
+            .filter(targetUser -> targetUser.getAge() >= 30)
+            .filter(targetUser -> targetUser.getAge() <= 45)
+            .isPresent();
     }
 
     @Override

src/main/java/nextstep/optional/Users.java

@@ -13,11 +13,9 @@ public class Users {
             new User("honux", 45));
 
     User getUser(String name) {
-        for (User user : users) {
-            if (user.matchName(name)) {
-                return user;
-            }
-        }
-        return DEFAULT_USER;
+        return users.stream()
+            .filter(user -> user.matchName(name))
+            .findFirst()
+            .orElse(DEFAULT_USER);
     }
 }

src/test/java/nextstep/fp/CarTest.java

@@ -8,24 +8,14 @@ public class CarTest {
     @Test
     public void 이동() {
         Car car = new Car("pobi", 0);
-        Car actual = car.move(new MoveStrategy() {
-            @Override
-            public boolean isMovable() {
-                return true;
-            }
-        });
+        Car actual = car.move(() -> true);
         assertThat(actual).isEqualTo(new Car("pobi", 1));
     }
 
     @Test
     public void 정지() {
         Car car = new Car("pobi", 0);
-        Car actual = car.move(new MoveStrategy() {
-            @Override
-            public boolean isMovable() {
-                return false;
-            }
-        });
+        Car actual = car.move(() -> false);
         assertThat(actual).isEqualTo(new Car("pobi", 0));
     }
 }