youngJun99님 말씀에 동의합니다!
그러나 Hwanvely님 말대로 호출할 때는 가독성을 위해 변수명을 `customSeparatorHandler` 대신 `handler`로 쓸 수 있을 것 같아요! 위 프로젝에서 handler는 하나밖에 없고 만약 해당 클래스에서 어떤 클래스인지 궁금하다면 속성에 선언된 부분을 보면 단번에 알 수 있기 때문입니다!
김이님 코드 잘 읽었습니다. 전체적으로 상수 문자열에 대해서 privat static final ~ 로 선언하신 부분은 좋았던것 같아요. 다른 상수들도 그렇게 선언해보면 어떨까요? 몇몇 함수들은 하나 초과의 책임을 지고 있는 것 같은데, 이 부분도 고려해보시면 좋을 것 같습니다. 1주차 수고하셨습니다.
김이님 코드 잘 읽었습니다. 전체적으로 상수 문자열에 대해서 privat static final ~ 로 선언하신 부분은 좋았던것 같아요. 다른 상수들도 그렇게 선언해보면 어떨까요? 몇몇 함수들은 하나 초과의 책임을 지고 있는 것 같은데, 이 부분도 고려해보시면 좋을 것 같습니다. 1주차 수고하셨습니다.
단순히 상속뿐만 아니라, 코드 스멜을 놓칠 수 있다는 단점도 있다고 생각해요.
저는 개인적으로 정적 팩토리 메서드를 도메인에서 사용하는 것을 좋아하지는 않는데요, `new` 생성자로도 충분히 표현할 수 있다고 생각하기 때문이예요. 오히려 `new`가 **생성한다**는 개념과 더 호응한다고 생각하기도 하고요.
만약 클래스에 이미 정적 팩토리 메서...