Closed belljun3395 closed 11 months ago
기존에 상속으로 구현된 Response를 수정하였습니다.
[AS-IS] ItResponse, MemberResponse를 상속하여 구현한 Response를 분리되도록 수정하였습니다.
[TO-BE]
오브젝트 책을 읽으며 상속은 타입 계층을 구현하기 위해 사용하는 것이라는 것을 알았고 현재 Response를 상속한 것은 이에 해당하지 않는다고 생각하였습니다.
오브젝트에서 제시한 상속을 사용하면 좋은 경우
위의 두 사항이 모두 만족할 때 상속을 사용하면 좋다고 하였습니다. 하지만 Response의 경우 2번의 경우를 만족하지 않는다는 판단을 하였고 수정하였습니다.
해당 부분은 필요하시면 넣어주세요~
There is no coverage information present for the Files changed
💁♂️ PR 내용
기존에 상속으로 구현된 Response를 수정하였습니다.
🙏 작업
[AS-IS] ItResponse, MemberResponse를 상속하여 구현한 Response를 분리되도록 수정하였습니다.
[TO-BE]
🙈 PR 참고 사항
오브젝트 책을 읽으며 상속은 타입 계층을 구현하기 위해 사용하는 것이라는 것을 알았고 현재 Response를 상속한 것은 이에 해당하지 않는다고 생각하였습니다.
오브젝트에서 제시한 상속을 사용하면 좋은 경우
클라이언트 입장에서 부모 클래스 타입으로 자식 클래스를 사용해도 무방한가?
위의 두 사항이 모두 만족할 때 상속을 사용하면 좋다고 하였습니다. 하지만 Response의 경우 2번의 경우를 만족하지 않는다는 판단을 하였고 수정하였습니다.
📸 스크린샷
해당 부분은 필요하시면 넣어주세요~
🤖 테스트 체크리스트