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

[BE] 스프린트3 API 명세 변경 이후 전반적인 코드 리팩토링 #111

Closed
6 of 8 tasks
yoondgu opened this issue Jul 25, 2023 · 2 comments
Closed
6 of 8 tasks
Labels
BE 백엔드 관련 이슈 refactor 리팩토링 관련 이슈 우선순위 : 중

Comments

@yoondgu
Copy link
Collaborator

yoondgu commented Jul 25, 2023

🛠️ 작업 대상

핀 관련 리팩토링 #90 머지 이후 develop 브랜치에 반영된 모든 코드 (테스트 포함)

✅ 작업 내용

refactor/style

  • 코드 컨벤션 통일, 불필요한 빈 줄 등 스타일 검토

refactor/test

refactor/domain

  • Topic 생성자 대신 imageUrl null 여부에 따른 정적 팩토리 메서드 적용
  • Pin - PinImage 연관관계 매핑 로직 검토

refactor/exception

📎 참고 자료

[Tecoble] 인수 테스트에서 테스트 격리하기

⏰ 추정 시간

비관적 추정 : 8시간
낙관적 추정 : 5시간

한 번에 모든 사항을 수정하지 않고 세부 이슈 발급하여 진행하는 것이 좋아보입니다.
우선순위가 더 높은 이슈 진행 중에 병행할 것이므로 7/28 밤까지 수정하는 것을 목표로 하겠습니다.

@yoondgu
Copy link
Collaborator Author

yoondgu commented Jul 27, 2023

커스텀 예외는 추후 구현, 현재는 API 사용을 위한 ControllerAdvice까지만 적용
JPA Fetch Type 지연로딩은 전체 도메인 구현 후 성능 검토 시 적용

테스트 관련 리팩토링 : @kpeel5839
예외 관련 리팩토링 : @yoondgu
나머지는 도메인 구현 진행 시 병행

@yoondgu
Copy link
Collaborator Author

yoondgu commented Aug 9, 2023

리팩터링 관련 논의된 내용들은
#182
#237
#240
#238
위 PR의 코드리뷰 참고할 것

@yoondgu yoondgu closed this as completed Aug 9, 2023
@github-project-automation github-project-automation bot moved this from Todo to Done in 2023-map-befine Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈 refactor 리팩토링 관련 이슈 우선순위 : 중
Projects
Status: Done
Development

No branches or pull requests

1 participant