fix : redis 랭킹 버그 수정

[kshired]

Issue Number

close: # N/A

작업 내역

구현 내용 및 작업 했던 내역

• ranking 버그 수정
• redis ranking 조회 리팩터링

변경사항

• 의존성 목록 : N/A

작업 유형

• 신규 기능 추가
• 버그 수정
• 리펙토링
• 문서 업데이트

PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

• zset().add 로 매번 존재하는건 replace하고, 없어진건 그대로 냅두는 방식으로 구현되어서 있어서 추가전 데이터를 지우고 추가하도록 변경
  • 데이터를 추가하거나 할 때 에러가 발생할 수 있으므로, transaction management를 받도록 수정
• redis를 통한 ranking 조회 로직을 함수형으로 리팩터링 함

[ekzm8523]

코멘트 몇개 남겼습니다!

[ekzm8523]

Suggested change

	val rankKey = redisTemplate.opsForZSet().reverseRangeByScore(RANKING, score, score, 0, 1)?.firstOrNull()
	if (rankKey != null) {
	rank = redisTemplate.opsForZSet().reverseRank(RANKING, rankKey)?.plus(1)
	val rank = rankKey?.let {
	redisTemplate.opsForZSet().reverseRank(RANKING, rankKey)?.plus(1)
	}
	val rankKey = redisTemplate.opsForZSet().reverseRangeByScore(RANKING, score, score, 0, 1)?.first()
	val rank = redisTemplate.opsForZSet().reverseRank(RANKING, rankKey!!)?.plus(1)

위에 score가 받아져왔다는 건 무조건 하나 이상의 점수가 레디스에 있다는 거니까 !!를 통해 무조건 존재한다는 걸 코드에 내포하는 건 어떨까요?
�위와 같이 정리하면 코드가 더 깔끔해질 것 같아요!

[kshired]

@ekzm8523 이건 사람마다 의견이 다를 수 있을 것 같아요. 저는 후자인듯?

rank의 score과 key가 확실하게 존재한다는 것을 의미하게 코드 짜기 vs ? 을 사용하여 null 안정성을 확보하면서 코드짜기.

어떻게 생각하시나요

[ekzm8523]

마지막에 변경된 rankKey?.let도 적절해보입니다~ 이렇게 가시져

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud