Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User package 변경점 #7

Open
zetq516 opened this issue Jun 6, 2022 · 2 comments
Open

User package 변경점 #7

zetq516 opened this issue Jun 6, 2022 · 2 comments

Comments

@zetq516
Copy link
Collaborator

zetq516 commented Jun 6, 2022

  1. UserDAO에 Admin/normal user의 공통 메서드를 추상 -> 일반으로 변경하고 UserDAO 내에 선언하였습니다.
  2. UserDAO에 taskType(+get/set)을 추가하였습니다. +Tasks(+get/set) 또한 UserDAO로 옮겨서 선언하였습니다.
  3. 2에 따라 Admin/normal user에 taskType을 선언하며 객체 생성하는 constructor를 추가하였습니다.
  4. 2에 따라 UserDAO, Admin/normal의 task pacakge 관련 import 선언문이 import oodp_task.*; 로 변경되었습니다.

user_issue1
user_issue1-1
user_issue1-2
user_issue2

@alsco1234
Copy link
Collaborator

  1. UserDAO에 Admin/normal user의 공통 메서드를 추상 -> 일반으로 변경하고 UserDAO 내에 선언하였습니다.

이 부분에서 제가 UserDAO의 Admin/normal의 공통 메서드를 추상 메서드로 만든 이유가 composite pattern을 적용하기 위한 것이었어서 바꿔도 될지는 모르겠네요.. 일단 발표때 교수님 피드백 들어보고 안되면 고쳐도 될까요?

@zetq516
Copy link
Collaborator Author

zetq516 commented Jun 6, 2022

아하 그렇군요... 만약 안되면 다시 바꾸셔도 좋을 것 같습니다! 그렇게 되면 제가 너무 번거롭게 한 거 같아서 죄송하네요ㅠㅠ 일단 확인했습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants