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

[FEAT] 샘플 이미지 업로드 API 구현, 이외 사진 도메인 리팩토링 #92

Merged
merged 9 commits into from
Aug 11, 2024

Conversation

bflykky
Copy link
Member

@bflykky bflykky commented Aug 11, 2024

❗️ 이슈 번호

Closes #87

📝 작업 내용

  1. 샘플 이미지 업로드 API 구현
  2. ShareGroupServiceImplFaceDetctionServiceImpl의 순환참조 문제 해결
  3. PhotoController, PhotoService, PhotoServiceImpl의 함수 순서를 수정하였다.
  4. PhotoConverter를 사용하지 않고 new 연산자로 dto 객체를 생성하는 코드를 converter를 사용하도록 리팩토링하였다.
  5. PhotoEsInfo, PagedPhotoEsInfo에서 'Es' 키워드를 제거하였다.

💭 주의 사항

💡 리뷰 포인트

사진 업로드 시, 새로 업로드된 사진에 대해 얼굴 인식을 위해 아래 코드를 호출한다.
faceDetectionService.detectFaceUploadPhoto(photoNameList, shareGroup.getId(), memberIdList);
기존에는 memberIdList 파라미터를 전달하지 않고, detectFaceUploadPhoto() 내부에서 memberIdList를 구했다. 이 과정에서 ShareGroupServiceImpl을 참조함으로써 순환 참조가 발생하였다. 이를 해결하기 위해, 파라미터로 memberIdList를 전달하도록 리팩토링하였다.

리팩토링 과정에서 해당 공유 그룹에 참여한 회원의 프로필에서 memberId를 추출할 때, 아직 회원이 참여하지 않은 프로필은 getMember()의 값이 null이므로 filter() 메소드를 통해 null이 아닌 값만 데이터 가공에 포함되도록 수정하였다.

- 샘플 이미지 업로드 API 구현
- ShareGroupServiceImpl과 FaceDetctionServiceImpl의 순환참조 문제 해결
  a. 프로필에서 memberId를 추출할 때, 아직 회원이 참여하지 않은 프로필은 getMember()의 값이 null이므로
  filter() 메소드를 통해 null이 아닌 값만 데이터 가공에 포함되도록 수정하였다.
- PhotoController, PhotoService, PhotoServiceImpl의 함수 순서를 수정하였다.
- PhotoConverter를 사용하지 않고 new 연산자로 dto 객체를 생성하는 코드를 converter를 사용하돌고 리팩토링하였다.
ElasticSearch를 뜻하는 es를 제거하였다.
@bflykky bflykky added ✨feature 기능 추가 🐛fix 버그 및 기능 에러 수정 ♻️refactor 코드 가독성 리팩토링 labels Aug 11, 2024
@bflykky bflykky self-assigned this Aug 11, 2024
@bflykky bflykky linked an issue Aug 11, 2024 that may be closed by this pull request
4 tasks
@bflykky bflykky merged commit 1caa571 into develop Aug 11, 2024
@bflykky bflykky deleted the feature/#87/sample-photo-api branch August 13, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛fix 버그 및 기능 에러 수정 ♻️refactor 코드 가독성 리팩토링 ✨feature 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 샘플 이미지 관련 API 구현
2 participants