인프런 커뮤니티 질문&답변

안녕하세요! 프로젝트와 관련해 코드를 작성하다 궁금한 점이 생겨 질문 드립니다..!

혹 선생님께서는 정적 팩토리 메소드의 역할이 어디까지라고 생각하실까요?

저는 팩토리 메소드이기 때문에 1. 단순히 파라미터를 받아 생성자를 통해 인스턴스를 생성하는 것 뿐만 아니라,

인스턴스를 생성하기 위해 2. 파라미터 valiation을 확인하고, 인스턴스 생성을 위한 데이터들을 가공하는 과정 또한 포함해도 괜찮은 것이 아닌가 하였는데,

저와 다른 의견 중에서는 팩토리 메소드는 가공이 전부 완료된 값을 단순히 받아서 인스턴스를 new 하기만 해야 한다는 의견이 있더라구요. 가공은 서비스 클래스에서 수행하구요!

예를 들면 아래와 같은 메소드가

validation과 데이터 가공을 처리하는 팩토리 메소드의 예시입니다 ::

public static SearchRequest from( AnotherRequest request) {
55
//AnotherRequest 클래스로부터 SearchRequest 클래스를 생성함.
        if(request.get고객리스트().size() > 9) {
56
//validation? 혹은 비즈니스 로직?
            throw new Exception();
57

        }
58
//아래부터 데이터 가공
        List<Passenger> 새로운 고객리스트 = request.get고객리스트().stream()
59

                                    .map(p -> new Passenger(p.get생일(), p.get성별()) )
60

                                    .collect(Collectors.toList());
61

62
//인스턴스 생성 후 return
        return new SearchRequest(새로운 고객리스트);
63

    }

2. 또한 webClient를 이용해 외부로 API 요청을 보내고자 할 때, path 값을 관리하는 과정에서,

1. 해당 path 값 ( ex. /a/b/xxx ) 을 반드시 webClient를 변수로 주입받아 실제 통신을 하는 class에서 가지고 있는 것과

    

2. 혹은 통신 시 사용되는 request 데이터 dto 에서 그 변수 값을 가지고, 실제 통신을 하는 class에서는 request dto에서 getPath()를 통해 가져와 사용하는 것

어느 부분이 더 적절하다고 생각하실까요?

저는 2번의 경우 webClient를 사용하는 클래스에서 추가적으로 여러 개의 path에 대한 상수 변수를 가지고 있을 필요가 없고 getPath()라는 메소드를 통해서 request dto의 종류를 신경쓰지 않고 일관되게 가져올 수 있다는 점에서 2번이 더 괜찮다고 생각을 하였는데,

다른 의견에서는 패스 값은 통신에 사용되는 값이기 때문에 dto가 아닌 통신을 하는 클래스가 가져야 하는게 더 자연스럽다. 그래서 새 기능이 추가되었을 때 통신을 하는 클래스 ( webClient를 변수로 가진 ) 에 패스에 대한 상수변수를 추가하고 패스에 따른 통신 메서드를 추가하는 것이 더 낫다.

라는 의견이 있었습니다.

코드 스타일에 따라 정답은 없겠지만, OOP의 입장에서 선생님의 의견을 여쭙고자 합니다! 답변 해주시면 정말 감사하겠습니다!