[1주차/그린] 워크북 제출합니다

[chazy-d]

✅ 워크북 체크리스트

• 모든 핵심 키워드 정리를 마쳤나요?
• 핵심 키워드에 대해 완벽히 이해하셨나요?
• 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
• 미션을 수행하셨나요?
• 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

• 디렉토리 구조 컨벤션을 잘 지켰나요?
• pr 제목을 컨벤션에 맞게 작성하였나요?
• pr에 해당되는 이슈를 연결하였나요?(중요)
• 적절한 라벨을 설정하였나요?
• 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
• 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

[YoungJJun]

1주차 피드백

1번 쿼리 좋아요~ 그런데 리뷰를 가게에 다는건지 자신이 진행한 미션에 대해 다는건지 피그마만 보고 진행하다 보니 모호한 부분이 있는데 기존 의도 자체는 가게에 다는게 맞는것 같아용. 일단 그린은 미션에 다는걸로 생각하신 것 같으니 이 기준으로 리뷰 드릴게요! 이후에 수정하셔도 됩니다.

2번도 좋습니다 해당 페이지에 필요한 모든 데이터는 조회된 것 같아요!
그런데 화면상에 당장 사용하지 않는 컬럼들도 있는 것 같은데 user_id는 따로 불러오신 이유가 있을까요? created_at과 region_id는 이후에 다른 페이지 이동시 필요할까봐 미리 조회하신걸까요??
이 부분 한 번 체크해주세요~

3번
tab_type을 전달함으로써 각 상황에 맞는 데이터들이 올바르게 조회되는 쿼리인것 같아요! 미션이 쿼리를 이용해 상황에 맞게 조회하라는 느낌이어서 하나의 쿼리가 각 상황에 맞게 동작할 수 있도록 조금 더 범용성 있게 작성해주신것 같습니다.
우선 해당 코드 동작하고 의도까지 이해되는데 이 경우 모든 row에 대해 불필요한 비교가 필요한 것 같아서 실제 코드로 구현할떄는 어플리케이션에서 분기처리 하는게 가독성이나 성능측면에서 더 좋을 것 같아요.

4번

• 도전 가능한 미션 목록에 대해 not exists 사용하신 점 좋아요~

• 가게 카테고리랑 마감일이 빠진것 같아요.

• 우선 쿼리는 정상동작 하는 것 같습니다! 그런데 ERD 설계가 다른 분들이랑 꽤 달라서 약간 이해를 못한 부분이 있는데..
  혹시 mission_goal_count라는게 해당 지역에서 10개 완료해야 보상 받을 수 있는 구조에서 10개를 의미하는게 맞을까요? 맞다면 이 컬럼을 저장한건 나중에 해당 값으로 모듈러 연산해서 완료까지 남은 미션 수를 카운팅 하는건가요?? 이부분은 생각하지 못했는데 좋은거 같아요 그린!

---

그린 수고하셨습니다~ 2주차도 화이팅하시고 스터디때 뵐게용
2주차 진행전에 해당 PR 머지 후 진행해주세요.