오늘은 실전프로젝트 중간 발표가 있었다.
먼저 한 발표자들을 보니 더욱 긴장됐다.. 그리고 우리 차례가 늦어서 중간에 최종 점검하는데 자잘한 오류가 발생해 급하게 수정하고 그래서 걱정됐는데 다행스럽게도 오류를 잘 처리하고 발표할 수 있었다.😀
발표 후 멘토님께 받은 피드백
-
카드 모달이 뜰때 url이 바뀌지 않기때문에 해당 디테일한 정보를 다른사람한테 공유할 수가 없다. 디테일한 페이지에 url이 변경되어야 그 링크를 다른 사람들에게 공유해줄 수 있다. -> 디테일 페이지 열었을때는 history API 사용해서 url을 바꾸거나 query string이나 해쉬로 url에 디테일 페이지를 열었다는 것을 꼭 표기해서 공유를 가능한 구조로 만들어야 바이럴도 잘 탈것 같고 sns 공유도 할 수 있다.
-
해당 정보가 양도 많지 않은 상황이여서 연관검색어에 필요한 정보를 소스에 하드코딩 했지만 그런 상황이 바뀌는 경우가 꼭 생기기 때문에 자주 바뀌지는 않지만 그래도 이것을 관리 할 수 있는 API와 스토리지, DB에 테이블을 만드는 방법 또는 관리할 수 있는 어드민을 만드는것도 좋을 것 같다.
-
마이페이지에서 뷰데이터 리덕스에 저장하는 부분 재고해야 함 -> 리덕스에서는 최대한 비즈니스 로직에 관련한 엔티티들, 데이터들, 모델들을 저장해서 활용하면 좋을 것 같다. ui만을 위해서 관리하는 스테이트를 리덕스에 저장하게되면 아무래도 ui의 변경사항이 훨씬 자주일어나니까 계속 업데이트가 되고, 이뮤터블 데이터를 사용하지 않기 때문에 ui를 한번 껐다켰다하는 간단한 연산이 전체 데이터셋을 다 재생산할수도 있는 성능적으로도 조금 이슈가 될 수 있는 부분도 있다. 그리고 가장 중요한 부분은 코드 유지보수상 화면에서 껐다 켜는 되게 지엽적인 데이터가 왜 글로벌 스코프에 노출되어있는지가 유지보수하기 힘들어질 것 같다. -> 그래서 자식이 부모데이터를 바꾸도록 접근하는 것이 아니라 자식이 부모의 데이터를 바꿔라라는 이벤트를 나타낼 수 있도록 다시 접근을 해서 부모와 자식간에 관계에서 끝내는 것이 좋을 것 같다.
3번째 내용은 내가 트러블 슈팅으로 제시했던 내용이었는데 피드백 해주셔서 정말 감사했다. 리덕스에서는 비즈니스 로직에 관련한 것들만 활용하면 좋다는 것과 자식이 부모의 데이터를 바꿔라라는 이벤트를 나타낼 수도 있다는 꿀팁도 알게되서 좋았다. 피드백 받은부분 적용해서 수정하고, 리팩토링도 하고, 시간이 된다면 새로운 기능도 추가해봐서 실전프로젝트를 더욱 보완 해야겠다..!🤗
