[BLOOM-040] 온보딩 api 개발 #110
Conversation
Dompoo
commented
Sep 12, 2024
Dompoo
commented
- 질문과 응답이 있을 때, 이를 통해서 어떻게 여러 결과중 하나를 선택할지 고민을 많이 했습니다.
- 아직은 가중치 등 복잡한 결과 산정 로직이 필요없다고 생각하여, 간단하게 구현하였습니다.
- 하나의 응답이 하나의 결과에 연결됩니다.
- 사용자가 제출한 응답지의 여러 응답들에 따라 각 연결된 결과를 쿼리합니다.
- 그렇게 쿼리해온 여러 결과중 가장 많이 선택된 결과를 반환합니다.
- 온보딩 질문지 조회 api를 개발했습니다.
- 온보딩 응답 제출 api를 개발했습니다.
응답들 당 하나의 결과가 매칭이 되고, 그 응답이 선택되면 그 결과가 선택될 확률이 높아지는거죠! 결과1, 2, 3이 있을 때,
요런식으로 질문-응답이 짜일 수 있고 만약 제가 질문1-응답1, 질문2-응답1을 했다면
요런식으로 계산되어서 결과1이 반환되는 방식입니다! |
| return resultSet | ||
| .maxByOrNull { result -> | ||
| frequencyMap.getOrDefault(result, 0) | ||
| } ?: throw NoSuchElementException("No result found in the resultSet") |
There was a problem hiding this comment.
resultSet에서 frequencyMap 기준으로 최대 선택 Result를 찾게 되는데, 일치하는게 하나도 없는 경우에 터지는 예외입니다! 보통 resultSet이나 selectedResult를 이상하게 제공했을 경우에 발생해요.
There was a problem hiding this comment.
그럼 따로 Exception을 만들어서 클라이언트에게 반환하는건 어떨까요? 현재 ExceptionHandler에 NosuchElementException이나 IlegalArgumentException 같이 기본 예외가 아닌 커스텀 예외처리가 되어있으니까요
| private fun inResultPairs( | ||
| qOnboardingAnswer: QOnboardingAnswer, | ||
| resultPairs: List<Pair<Int, Int>>, | ||
| ): BooleanExpression? { | ||
| val conditions: List<BooleanExpression> = | ||
| resultPairs.map { (questionNumber, answerNumber) -> | ||
| qOnboardingAnswer.onboardingQuestion.questionNumber.eq(questionNumber) | ||
| .and(qOnboardingAnswer.answerNumber.eq(answerNumber)) | ||
| } | ||
| return if (conditions.isNotEmpty()) { | ||
| Expressions.allOf(*conditions.toTypedArray()) | ||
| } else { | ||
| null | ||
| } |
There was a problem hiding this comment.
resultPairs가 비어있는 경우입니다!
네 맞습니다! |
