[1주차] 기본/심화과제 #8
Conversation
|
|
||
| public class ChequingAccountService implements AccountService { | ||
|
|
||
| private final AccountRepository accountRepository = new AccountRepository(); |
There was a problem hiding this comment.
이 부분은 저번 스프링 스터디에서 들었던 강의대로 AppConfig라는 설정 파일을 만들고 생성자 주입으로 외부에서 AccountRepository 구현체를 넘겨주는 방식으로 구현해도 괜찮을 것 같습니다
물론 해당 과제를 받았던 1차 세미나 당시에는 아직 안배웠던 내용이긴 합니다!
| view.printCommandMessage(MEMBER_REGISTER_SIGN); | ||
|
|
||
| UserRegisterDto userRegisterDto = view.getUserRegisterInfo(); | ||
| userController.register(userRegisterDto); |
There was a problem hiding this comment.
AutoController가 userController나 accountController를 호출하도록 구현한 이유가 따로 있으신가요?? 저라면 AutoController가 다른 conntroller를 의존하지 않고 userService.register()나 ChequingAccountService.deposit() 메소드를 AutoController에서 바로 호출하도록 구현했을 것 같긴 합니다
There was a problem hiding this comment.
뭔가 단순히... 서비스 레이어마다 컨트롤러가 있어야 하지 않을까.. 하는 생각이었던 것 같아요! 그리고 지금 클라이언트의 입력을 처리해서 적절한 핸들러로 매핑해주는..? 서블릿이 없으니까 중간에 컨트롤러가 더 있으면 어떨까.. 했던 것 같긴 한뎅.. 크게 깊게 고민해본 부분은 아니었던 것 같네요!! 만약 제 생각에 조금이라도 어색한 부분이 있으면 알려주세요!
|
멋진 은행 프로그램 잘봤습니다! 고생하셨습니다! |
|
enum을 사용해서 전체적으로 코드 가독성이 너무 좋아요! 저도 다음부터는 따라 해보겠습니다~! 고생하셨습니다 |
📝 관련 이슈
closed #4
closed #5
👩🏻💻 내용
😸 중점을 둔 부분
😹 해결하지 못한 부분
AcountController에서 계좌의 타입에 따라서 다른 서비스의 메서드를 호출해야 하는데, 지금 방식이 너무 이상한 것 같아서 고민입니당..
하위디렉토리라서 그런지 .gitginore 가 적용이 안돼서 전부 다 올라간 것 같은데 이런 문제는 어떻게 해결해야 하나요..? ㅠ-ㅠ
해결하지 못한 부분 중심으로 코멘트 주시면 감사하겠습니당