[#61] 최근 방문한 매장 조회 기능 구현 #62
Conversation
binaryyoung
commented
Jan 3, 2020
binaryyoung
commented
- 회원이 최근 방문한 매장을 조회하는 기능 추가
| @@ -18,7 +18,6 @@ | |||
| import org.springframework.data.redis.serializer.RedisSerializationContext; | |||
| import org.springframework.data.redis.serializer.StringRedisSerializer; | |||
|
|
|||
| @Configuration | |||
There was a problem hiding this comment.
이 부분은 레디스 설정파일이 2개일 때 오류가 나서 임시로 지웠는데 해결하면 다시 추가하겠습니다.
| @Bean | ||
| public LettuceConnectionFactory shopRedisConnectionFactory() { return new LettuceConnectionFactory(); } | ||
|
|
||
| @Bean |
There was a problem hiding this comment.
오류가 나는 이유는 동일 타입 Bean에 이름을 정해주지 않아서입니다. 해당 Bean에 이름을 정해준다면 오류를 해결할 수 있을것으로 보이네요. 다만 그렇다면 @Autowired 를 사용하는 RedisTemplate에 @Qualifire 어노테이션을 추가하여 어떤 Bean을 사용할지 정해주어야합니다.
There was a problem hiding this comment.
그렇군요. 참고해서 다시 적용해보겠습니다
| public class ShopRedisConfig { | ||
|
|
||
| @Bean | ||
| public LettuceConnectionFactory shopRedisConnectionFactory() { return new LettuceConnectionFactory(); } |
There was a problem hiding this comment.
LettuceConnectionFactory는 따로 옵션을 설정해 주자 않는건가요?
There was a problem hiding this comment.
application.propertise에 spring.redis.port=6379 와 같이 설정을 해놓으면 자동으로 적용되어 따로 옵션을 설정해주진 않았습니다.
There was a problem hiding this comment.
Bean이 제대로 주입되지 않아서 자동으로 적용되는것처럼 보이는 것 같습니다. 설정도 따로 분리하시는게 좋을 것 같네요. 명시적으로 config를 설정해주지않으면 자동으로 적용되지 않습니다
| * @param shopId 매장 아이디 | ||
| */ | ||
| public void add(String memberId, Long shopId) { | ||
| ZSetOperations<String, Long> zSetOperations = shopRedisTemplate.opsForZSet(); |
There was a problem hiding this comment.
최근 조회한 매장 목록을 불러와야 되는데 list를 사용하면 데이터의 중복이 발생하고 중복을 막기위해 그냥 set을 사용하면 최근 조회한 매장 순서로 불러올 수 가 없었기 때문에 정렬된 셋인 ZSet을 사용하였습니다.
| */ | ||
| public void add(String memberId, Long shopId) { | ||
| ZSetOperations<String, Long> zSetOperations = shopRedisTemplate.opsForZSet(); | ||
| zSetOperations.add(RECENT_SHOP_VIEW_KEY + memberId, shopId, System.currentTimeMillis()); |
There was a problem hiding this comment.
해당 키를 만드는 Factory가 있으면 좋을 것 같습니다. 여러 곳에서 같은 key를 참조해야할텐데 동일한 key 생성 로직을 하나의 메서드로 만들 수 있으니까요.
| public class ShopRedisConfig { | ||
|
|
||
| @Bean | ||
| public LettuceConnectionFactory shopRedisConnectionFactory() { return new LettuceConnectionFactory(); } |
There was a problem hiding this comment.
Bean이 제대로 주입되지 않아서 자동으로 적용되는것처럼 보이는 것 같습니다. 설정도 따로 분리하시는게 좋을 것 같네요. 명시적으로 config를 설정해주지않으면 자동으로 적용되지 않습니다
| redisTemplate.setConnectionFactory(shopRedisConnectionFactory()); | ||
| redisTemplate.setKeySerializer(new StringRedisSerializer()); | ||
| redisTemplate.setValueSerializer(new GenericToStringSerializer<>(Long.class)); | ||
| return redisTemplate; |
There was a problem hiding this comment.
각 옵션들을 설정한 의도를 주석으로 설명해주시면 좋을 것 같습니다~
