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.

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

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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

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