feat: 리뷰어 리뷰이 매칭 및 매칭 결과 공지

[davidolleh]

작업 내용

• 리뷰어 리뷰이 매칭 및 결과 공지 기능을 구현하였습니다.

SlashCommand 안에 자동완성 기능이 포함되어 있습니다. 그래서 자동완성 기능을 가진 SlashCommand를 그림과 같이 상속을 통해서 표현했습니다. ReviewMatchListener는 SlashCommand를 상속받은 AutoCompleteInteraction를 구현했기에 자동으로 이벤트로 등록 됩니다.

참고 사항

• 테스트 코드의 방향성을 정하지 않아 테스트에서 랜덤 Shuffle을 처리하기 위해 Shuffle하는 기능을 추상화 하였고 테스트 코드에서는 직접 mock 클래스를 만들어 테스트 하였습니다.

관련 이슈

• closes feat: 리뷰어 리뷰이 매칭 및 매칭 결과 공지 #14

PR 체크리스트

• 테스트는 모두 통과했나요?
• 빌드는 성공했나요?
• 코드 포맷팅을 진행했나요?
• PR 내부의 예시는 삭제하셨나요?

[TaeyeonRoyce]

고생 많으셨어요~!
복잡한 로직일텐데 꼼꼼하게 신경 써주셨군요.
인터페이스 구조도 상황에 맞게 잘 잡아주신 것 같아요.

몇가지 코멘트 남겼으니 확인하시고 의견 남겨주세요!

[davidolleh]

마지막으로 질문 있습니다! For Each와 for loop은 취향차이인지 상황마다 확실한 장단점이 있는지 궁금합니다.

[TaeyeonRoyce]

마지막으로 질문 있습니다! For Each와 for loop은 취향차이인지 상황마다 확실한 장단점이 있는지 궁금합니다.
확실한 장단점 보단 특징이 달라요.
for-loop은 Index를 조작 할 수 있고, continue나 break 같은 흐름 제어가 가능하죠.
forEach()는 그런게 불가능 하지만, 선언적인 표현으로 좀 더 직관적이고 간결하게 나타낼 수 있습니다.
성능도 Java21을 사용하는 상황에서는 유의미한 차이는 전혀 없을 것 같아요!

[TaeyeonRoyce]

LGTM 🚀
고생하셨어요!