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.

Sign up for GitHub

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

Jump to bottom

✨ [Feat] 유저의 입력 값 바탕 제일 높은 가중치의 개발자 3명 반환하는 API #151

Merged
merged 13 commits into from
Jul 21, 2024
Merged

✨ [Feat] 유저의 입력 값 바탕 제일 높은 가중치의 개발자 3명 반환하는 API #151

merged 13 commits into from
Jul 21, 2024

Conversation

88dldl
Copy link
Member

@88dldl 88dldl commented Jul 20, 2024

📍 PR 타입

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

❗️ 관련 이슈 링크

Close #143

🔁 변경 및 추가사항

  • part, 개발 경험, 학번 모두 충족 시 가중치 검사하도록 설정해두었습니다.
  • 현재 developer_match요청 값으로 받은 developer의 가중치를 임의로 세팅해두었습니다.
    -> 논의 후 적절한 값으로 세팅 필요합니다.
    [DeveloperServiceImpl][DeveloperRepositorympl] 중점적으로 피드백 주시면 좋을 듯합니다!!

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

https://rst0070.github.io/java/jackson-json-node-read-n-write.html
-> developer_match 요청보내고 받은 값을 파싱하는 방법 위의 자료를 참고했습니다

✅ 체크 리스트

  • PR 템플릿에 맞추어 작성했어요.
  • 변경 내용에 대한 테스트를 진행했어요.
  • 프로그램이 정상적으로 동작해요.
  • PR에 적절한 라벨을 선택했어요.
  • 불필요한 코드는 삭제했어요.

@88dldl 88dldl added the ✨ Feat 새로운 기능 추가 label Jul 20, 2024
@88dldl 88dldl self-assigned this Jul 20, 2024
@88dldl 88dldl linked an issue Jul 20, 2024 that may be closed by this pull request
✨ [Feat] 팀원매칭 봇 API 구현 #143
Closed
1 task
chanmin-00
chanmin-00 reviewed Jul 21, 2024
View reviewed changes
Copy link
Member

@chanmin-00 chanmin-00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

긴 코드 작성하시느라 수고하셨습니다!! 👍

src/main/java/ssuPlector/repository/developer/DeveloperRepositoryImpl.java Outdated
queryFactory
.selectFrom(developer)
.where(
part1Eq(requestDTO.getPart())
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

part1만 조회하는 이유가 있을까요?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아니요.. 그냥 제가 까먹은겁니다 ... 추가하도록 할게요!!

src/main/java/ssuPlector/service/developer/DeveloperServiceImpl.java Outdated
() -> new GlobalException(GlobalErrorCode.DEVELOPER_NOT_FOUND));

weight.put(developer.getId(), weight.get(developer.getId()) + w);
w -= 0.3;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

JSON 데이터를 읽어오는 과정이 복잡하긴 하지만 각 개발자들의 매칭 정도를 가중치로 표현해서 정렬하는 좋은 방식인 것 같습니다. 오류 부분을 수정하고 나서 머지하면 될 것 같습니다. 수고하셨습니다

chanmin-00
chanmin-00 approved these changes Jul 21, 2024
View reviewed changes
Copy link
Member

@chanmin-00 chanmin-00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!!

@88dldl 88dldl merged commit 97cc7f5 into develop Jul 21, 2024
@88dldl 88dldl deleted the issue/143-matching-bot branch July 21, 2024 08:48
@88dldl 88dldl restored the issue/143-matching-bot branch July 21, 2024 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chanmin-00 chanmin-00 chanmin-00 approved these changes

Assignees

@88dldl 88dldl

Labels
✨ Feat 새로운 기능 추가
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

✨ [Feat] 팀원매칭 봇 API 구현
2 participants
@88dldl @chanmin-00