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

주변 수거함 목록 조회 기능 요청 파라미터 객체화 #107

Closed
wants to merge 6 commits into from

Conversation

Hanjaemo
Copy link
Contributor

💡 작업 내용

  • �요청 파라미터 객체화

💡 자세한 설명

기존 �Controller 메서드에서는 위도와 경도를 원시 타입으로 전달 받은 뒤, 서비스 계층에서 이를 Point 타입으로 변환해주었습니다.
이를 개선하고자 ArgumentResolver를 이용해 요청 파라미터를 객체로 전달 받도록 수정했습니다.

그리고 tags에 대한 유효성 검증 로직이 서비스 계층 여러 곳에서 중복되는 것을 파악했으며, 이 또한 ArgumentResolver를 이용해 개선했습니다.

✅ 셀프 체크리스트

  • PR 제목을 형식에 맞게 작성했나요?
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있나요?
  • 이슈는 close 했나요?
  • Reviewers, Labels, Projects를 등록했나요?
  • 작업 도중 문서 수정이 필요한 경우 잘 수정했나요?
  • 테스트는 잘 통과했나요?
  • 불필요한 코드는 제거했나요?

closes #106

@Hanjaemo Hanjaemo added feature 구현, 개선 사항 관련 refactor 리팩토링 labels Jun 13, 2024
@Hanjaemo Hanjaemo requested a review from Doyeon04 June 13, 2024 07:17
@Hanjaemo Hanjaemo self-assigned this Jun 13, 2024
@Hanjaemo Hanjaemo closed this Jun 13, 2024
@Hanjaemo Hanjaemo deleted the feat/106-find-boxes-refactoring-and-test branch June 13, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 구현, 개선 사항 관련 refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

주변 수거함 목록 조회 기능 단위 테스트 작성 및 리팩터링
1 participant