-
Notifications
You must be signed in to change notification settings - Fork 1
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
의료진 도메인 기능 리팩토링 #8
Comments
2dongyeop
added a commit
to 2dongyeop/Avocado-backend-with-hexagonal
that referenced
this issue
Jun 4, 2023
2dongyeop
added a commit
to 2dongyeop/Avocado-backend-with-hexagonal
that referenced
this issue
Jun 4, 2023
2dongyeop
added a commit
to 2dongyeop/Avocado-backend-with-hexagonal
that referenced
this issue
Jun 4, 2023
1. 의료진 비밀번호 변경 유스케이스 작성 비밀번호를 변경할 의료진 번호와 이전 비밀번호, 새 비밀번호를 입력받습니다. 의료진 번호를 이용해 조회한 의료진 엔티티를 도메인으로 매핑합니다. 의료진의 비밀번호와 입력으로 받은 비밀번호가 다를 경우, 예외를 발생시킵니다. 의료진 비밀번호가 일치시 도메인에서 변경 비즈니스 로직을 실행하지만, 도메인과 엔티티를 분리해놓았기에 변경감지가 일어나지 않습니다. 따라서 해당 도메인을 엔티티로 변환하고 다시 저장합니다. 현재 비밀번호 암호화를 적용하지 않은 상태이므로, 로그인을 비롯한 인증인가 처리 작업시 함께 진행할 예정입니다. 2. 의료진 정보(프로필 이미지 및 소속 병원) 수정 유스케이스 작성 의료진 프로필 이미지 변경과 의료진 소속병원 변경 로직을 분리할 경우, 중복된 코드가 남발할 수 있습니다. 따라서 `@RequestParam(required = false)`로 지정하여, 쿼리 스트링으로 들어온 해당 값만 변경하도록 합니다. Related to : HBNU-Avocado#8
2dongyeop
added a commit
to 2dongyeop/Avocado-backend-with-hexagonal
that referenced
this issue
Jun 4, 2023
의료진 탈퇴 기능을 개발합니다. 입력으로 받은 의료진 번호를 이용해 의료진 엔티티를 조회하고, JPA repository에 해당 의료진 엔티티를 전달해 삭제하도록 요청합니다. 원래는 엔티티를 조회한 후 도메인으로 매핑을 하고, 도메인의 비즈니스 로직을 실행한 뒤 다시 엔티티로 매핑하여 요청을 전달하는 방식이지만 해당 유스케이스의 경우에는 실행되는 도메인의 비즈니스 로직이 없습니다. 따라서 불필요한 엔티티-도메인 간의 매핑을 줄이기 위해 삭제용 엔티티 조회 메서드를 함께 작성하였습니다. Related to : HBNU-Avocado#8
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description 🔍
의료진 도메인 기능을 개발합니다.
Todo 🗓️
Etc
The text was updated successfully, but these errors were encountered: