-
Notifications
You must be signed in to change notification settings - Fork 0
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 #38 매칭 필터링 및 알고리즘 구현 #38
base: dev
Are you sure you want to change the base?
The head ref may contain hidden characters: "feat/#30/\uB9E4\uCE6D-\uD544\uD130\uB9C1-\uBC0F-\uC54C\uACE0\uB9AC\uC998-\uAD6C\uD604"
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨어요!!
Members user = memberRepository.findById(userId) | ||
.orElseThrow(MemberNotFoundException::new); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MemberService에 findById 메서드가 있습니다!!
해당 메서드를 사용하는게 좋아보여요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MemberService에 예외처리까지 되어있는 메서드를 사용하면,
불필요한 메서드 추가랑 예외처리를 안해줘도 될거같네요. 수정 완료했습니다 !
📌 관련 이슈
관련 이슈 번호 #30
Close #
🚀 작업 내용
*거리 조건은 300m 이내로 설정했고, 태그가 비어있는 경우에도 처리할 수 있도록 설정하였습니다
📸 스크린샷
SSE 구독 후 포스트맨에서 테스트 진행했습니다
DB에 매칭방 생성까지 확인했습니다
추가로 동일한 유저가 똑같은 조건의 방을 생성하려고 할시, 이미 존재하는 매칭방이라는 예외를 던져주도록 설계했습니다
📢 리뷰 요구사항
주영님과 충돌을 방지하고자 매칭시 필터링 로직이랑 전체 매칭방 리스트 조회 먼저 해당 PR에서 작업해서 머지하도록 하고,
블랙리스트와 친구추가는 별도의 이슈를 생성해서 추후에 작업하도록 협의했습니다
현재 저희가 출발지(가천대학교 정문, 1번출구) 와 도착지(가천대학교 기숙사, AI 공학관)로 정해져있는 설계였기에
Haversine이랑 Radian을 통한 위도 경도 계산법은 사용하지 않았습니다
하지만 추후에 앱으로 마이그레이션 후 출발지와 도착지를 지도에서 동적으로 선택할 수 있도록 확장되는 경우에는 리팩토링이 필요해보입니다