[사다리 - 함수형 프로그래밍] 박진형 리팩터링

[jinhyeongpark]

사다리 게임 리팩터링

1. 객체의 책임 분리 및 일급 컬렉션 적용

• Lines 및 Points 도입: 기존 Ladder와 Line에 집중되어 있던 리스트 관리 및 생성 로직을 각각 Lines와 Points라는 일급 컬렉션으로 분리
• 캡슐화 강화: 외부에서 내부 리스트에 직접 접근(getValues())하는 대신, Iterable 인터페이스를 구현하고 size(), get() 메서드를 제공하여 객체 스스로가 데이터를 관리

2. 의존성 주입(DI)을 통한 결합도 해제

• Ladder가 전략(BooleanSupplier)을 직접 받아 하위로 전달하던 방식에서, 상위 레이어에서 생성된 Lines를 주입받는 방식으로 변경
• Ladder는 이제 사다리 생성 방식을 몰라도 되며, 주입된 Lines를 통해 결과 산출 역할에만 집중

3. 네이밍 변경

• 정적 팩터리 메서드에 대해 단순 of나 from이 아닌 generate를 사용하여 내부적으로 별도의 로직이 수행됨을 명시

4. Enum 적용

• Point와 같은 특정 상태(다리가 있음, 없음)를 표현하는 클래스에 대해 Enum으로 변경하여 객체보다는 상태임을 명시

[CodingMasterLSW]

안녕하세요 진형님, 리뷰가 늦어졌네요ㅜㅜ

우선 미션을 하나 드리려고 합니다. 지금 사용하고 있는 모든 정적 팩토리 메서드 코드들을 삭제하고, 외부에서 주입받는 형식으로 리팩토링을 진행해보시죠. 간단히 말하자면,

생성자는 외부 주입을 통해 생성만 해봅시다.

public class Points implements Iterable<Point> {
    private final List<Point> points;

    public Points(List<Point> points) {
        this.points = points;
    }

리팩토링을 진행하면서 막히는 부분이 있다면 말해주세요. 그리고 리팩토링 전, 후 코드를 비교해보면서 느낀점을 작성해주세요. ex) 정적 팩토리를 썼을 때의 장단점, 외부 주입을 했을 때의 장단점, 내가 앞으로 선택할 방식은?

두 번째로 Controller에 대한 답변을 드리려고 합니다.

이런 미션을 하기 전에는 Controller-Service-Repository의 구조에서만 개발을 하던 입장이라..
API 요청을 받아서 필요한 Service를 호출하는 역할로만 생각해왔습니다.
자체적으로 로직 수행이나 기능을 담지는 않고 로직을 담은 클래스들을 연결하는 책임으로 생각됩니다!

• 그러면 현재 코드에서는 api 요청 자체가 존재하지 않는데, controller는 불필요한거 아닐까요?? 아예 없애보시죠!
• 로직을 담은 클래스들을 연결하는 건, 별도의 Manager라는 Domain 객체를 만들어 위임해도 괜찮을 것 같네요.

해당 내용 관련해서는 제가 이전에 다른 리뷰이에게 달아놓은 리뷰가 있어서, 참고해보면 좋을 것 같아요.
next-step/java-lotto-clean-playground#151

두 작업 모두 시간이 꽤 걸리는 리팩토링 작업일 수도 있지만, 꼭 해보는걸 추천드립니다! 어차피 마지막 미션이니 늦게 리뷰 요청을 해도 상관 없어요ㅎ_ㅎ 파이팅입니다!

[CodingMasterLSW]

이건 반복문 쓰는게 더 코드가 직관적일 것 같은데요??

public Map<Integer, Integer> generateREulsts2() {
        Map<Integer, Integer> results = new LinkedHashMap<>();
        for (int index = 0; index < width; index++) {
            results.put(index, climb(index));
        }
        return results;
    }

스트림을 선택한 이유가 있었나요??

[CodingMasterLSW]

이 친구는 default로 설정한 이유가 있나요?

[CodingMasterLSW]

Collections.unmodifiableList를 사용하면 모든 경우에서 불변을 유지할 수 있는걸까요??