[김수진]상세페이지 반응형 수정 및 리뷰 요청

[ksj0621]

수정 내용

• 상세페이지 safety 섹션 이미지 반응형 css 코드 수정

---

질문 사항

1. productdetail 페이지 컴포넌트와 하위 컴포넌트들의 전체적인 코드 구조와 짜임새의 개선할 부분을 짚어주시면 감사하겠습니다!
2. 지난 코드리뷰 때 남겨주셨던 ProductDetail.tsx의 useEffect, useState를 제거하여 불필요한 렌더링을 줄이라시는 코멘트에 대해 따로 직접 코멘트 남겨드린 대로, 두 번이나 짚어주셨던 부분이기에 제가 코드도 교체해보고 따로 공부도 해보았는데 코드 교체 시 하위 컴포넌트에서 갱신된 데이터를 받아 json 파일에서 데이터를 잘 못불러오는 문제가 있었습니다. useEffect, useState를 통해 상태를 저장하고 동적 데이터를 반영하도록 하여 리렌더링을 시켜야 하는 걸로 알고있는데 혹시 제가 잘못 이해하고 있거나, 코드를 잘 못 작성한 걸까요? 코멘트와 함께 확인해주시면 감사하겠습니다!!
3. violation과 관련한 코멘트도 따로 직접 남겨놓았습니다. 여러 violation 중 두가지의 violation이 지속적으로 뜨고 데이터도 늦게 불러와지는 바람에 중첩되어 렌더링 되는 현상도 지속되고 있어서 해결방법이 있을지 궁금합니다!
4. 스크롤 트리거 애니메이션 섹션의 반응형은 여전히 해결되지 않고 있습니다ㅠㅠ 제 노트북에서 화면 크기별로 확인해보면 문제가 없는데 다른 pc나 배포된 모바일 환경에선 요소들이 중첩되어 렌더링 됩니다. 혹시 제 반응형 코드에 문제가 있는 걸까요? 이 부분도 지난 번 코멘트에 직접 달아놓았습니다!!

---

매번 코드리뷰 남겨주실 때마다 하나하나 반영하려 노력하고 있습니다만 아직 부족한 점이 많은 것 같습니다. 그래도 하나하나 도움이 많이 되고 또 많이 배우고 있습니다! 마지막 코드리뷰인만큼 남겨주신 코멘트들 전부 공부해서 반영할 수 있도록 하고, 프로젝트도 잘 마무리하고 싶습니다. 항상 감사드립니다!!

[Bang-Kyungmin]

지난 코드리뷰 때 남겨주셨던 ProductDetail.tsx의 useEffect, useState를 제거하여 불필요한 렌더링을 줄이라시는 코멘트에 대해 따로 직접 코멘트 남겨드린 대로, 두 번이나 짚어주셨던 부분이기에 제가 코드도 교체해보고 따로 공부도 해보았는데 코드 교체 시 하위 컴포넌트에서 갱신된 데이터를 받아 json 파일에서 데이터를 잘 못불러오는 문제가 있었습니다. useEffect, useState를 통해 상태를 저장하고 동적 데이터를 반영하도록 하여 리렌더링을 시켜야 하는 걸로 알고있는데 혹시 제가 잘못 이해하고 있거나, 코드를 잘 못 작성한 걸까요? 코멘트와 함께 확인해주시면 감사하겠습니다!!

2번은 직접 코드를 수정해보니 데이터는 문제없이 가져오는 것 같은데 어떤 문제가 있으셨을까요? react-router를 통해 url 변경을 하게 되면 useParams 훅을 사용하는 컴포넌트들은 리렌더링됩니다. 그래서 정적데이터가 있더라도 carId가 변경되기 때문에 find를 통해 찾은 값이 다르게 할당될 것으로 기대됩니다.
console.log(carId)를 렌더링 코드에 넣어 디버깅해보셔도 좋을 것 같아요!