-
Notifications
You must be signed in to change notification settings - Fork 332
Open
Description
친절하면서 멋지게 작성된 소스 내용을 보던 중 의도가 너무 궁금해서 이렇게 문의 드립니다.
만약 불쾌함을 드렸다면 정말 사과 드립니다.
질문1. MemberFindDao에서 @repository가 아닌 @service를 사용하신 이유가 뭘까요?
질문2. MemberFindDao에 있는 두 개의 findBy* 메소드는�MemberRepository에서 제공이 가능 한데도 불구하고 DAO 클래스를 선언하신 이유는 뭔가요? (추측하기로 리턴값이 Optional.empty인 경우 적절한 exception 사용을 위한게 아닐까 생각했습니다.)
질문3. 만약 [질문2]에 대한 저의 추측이 맞다면, 비슷한 기능(?)을 하는 findById, findByEmail의 메소드는 두 군데(Dao, Repository)에 존재 하게 될테고 메소드가 사용되는 관점에서 봤을때 같은 혹은 비슷한 기능의 메소드가 중복으로 존재하는게 될텐데 의도하신 건가요?
(물론 리턴값(Optional, Member)이나 리턴되는 exception은 차이가 있지만 비즈니스는 유사해 보입니다.)
여기까지 읽어 주신것 만으로도 감사 드립니다 ^^
(후에 더 여쭤보고 싶은데 이메일이라도 알려주시면 감사하겠습니다 godngu@gmail.com)
Metadata
Metadata
Assignees
Labels
No labels