Skip to content

[Refactor] 코드 리뷰 후 ImageRedisService 리팩토링#295

Open
tigris24 wants to merge 2 commits intoKernel360:developfrom
tigris24:refactor/#293/refactor-redis
Open

[Refactor] 코드 리뷰 후 ImageRedisService 리팩토링#295
tigris24 wants to merge 2 commits intoKernel360:developfrom
tigris24:refactor/#293/refactor-redis

Conversation

@tigris24
Copy link
Collaborator

@tigris24 tigris24 commented Sep 5, 2024

#293
외부 메소드를 만들어서 처리한 부분 정리

  • saveNewColor
    • extractRGB : ClothAnalyzeData 에서 rgb 리턴
    • isNewColor : 해시 테이블에서 STANDARD_DIST 기준으로 입력된 RGB가 새로운 색인지 확인
    • saveColor : 새로운 색 이름과 RGB를 ZSET과 HASH TABLE에 각각 저장
  • updateExistingColor
    • getPopularPosts : findAllByPostStatusAndCreatedAtAfterAndLikeCountGreaterThanEqualAndHitsGreaterThanEqual 사용
    • selectColorsForUpdate : 각 이미지 기준으로 selectedColorHashMap에 업데이트 할 색 추가
      • 같은 색일 때 업데이트 조건은 기존 색에 대한 condition 과 새로 저장할 condition 을 calcScorePost로 비교해서 큰 값 쪽 채택
    • updateColorIfCloseEnough : selectedColorHashMap에 저장된 업데이트 할 색들을 해시 테이블에서 찾아서 RGB 값 업데이트

@tigris24 tigris24 self-assigned this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant