2020-NAVER-CAMPUS-HACKDAY / Influencer

🧏🏻 간단한 사용자 인터랙션을 통한 상품 추천 및 전시 개발 (a.k.a 쇼핑의 신) 🚚
109 stars 7 forks source link

Get Interaction Product Data Randomly #145

Closed Seogeurim closed 4 years ago

Seogeurim commented 4 years ago

관련 이슈

변경 사항 및 이유

이는 아주 간단한 PR입니다 ㅎㅎ 그래서 좀 더 구현 후 올릴까 고민을 했으나, PR을 통해 고민들을 나눠보면 좋을 것 같아서 우선 올려봅니다.

인터랙션 시, 원래는 : 카테고리 별로 pagination을 통해 데이터를 받아옵니다. 바꾼 것은 : 요청에 따라 랜덤으로 10개씩의 데이터를 받아옵니다. ([이전 카테고리], [이후 카테고리] 버튼을 누름에 따라 사용자에게 매번 새로운 데이터를 보여주고 싶으니까요!)

PR Point

고민 1. 본래 코드에서는 Redux로 page를 두어, 데이터를 다 읽으면 page 값을 바꿔주며 새로운 요청을 했습니다. 바꾼 코드에 대해서도 page 변수를 증가시키면서 요청을 하도록 두었는데, 변수의 의미상 괜찮은 것일까요?

고민 2. 사용자가 이미 인터랙션 [좋아요]/[싫어요]를 진행한 데이터에 대하여, 중복으로 수행하지 않도록 하는 별도의 처리가 필요할까요?

참고 사항

jominjimail commented 4 years ago

고민 1. 본래 코드에서는 Redux로 page를 두어, 데이터를 다 읽으면 page 값을 바꿔주며 새로운 요청을 했습니다. 바꾼 코드에 대해서도 page 변수를 증가시키면서 요청을 하도록 두었는데, 변수의 의미상 괜찮은 것일까요?

listCategory --> getCategoryProducts ex) 5000 카테고리의 아이템을 가져오는데 page로 list 순서대로 0-10개를 가져오는것보다 random으로 10개를 가져오고 싶다는거죠!

저도 그림님이 구현한것처럼 2번째가 더 맞다고 생각해요! 변수 의미라는건 list에서 get으로 바뀐 부분을 말씀하시는건가요? 그렇다면 괜찮은것 같습니다.

고민 2. 사용자가 이미 인터랙션 [좋아요]/[싫어요]를 진행한 데이터에 대하여, 중복으로 수행하지 않도록 하는 별도의 처리가 필요할까요?

중복을 처리한다는게 어느 부분에서의 중복을 말씀하시는건가요? 카테고리를 (500, 600, 700)을 선택했다면, 500 - 10개 랜덤 600 - 10개 랜덤 700 - 10개 랜덤

이라 중복이 없을것 같은데요. 어느 부분을 고민하고 계시는건가요?

참고 사항