//////
Search
💻

1월 2일 호준님 코드리뷰

생성자
회의 유형
코드리뷰
이미지 세개일때 검은색이미지 올렸을때 점이 안보이는 현상 => 점을 이미지 밖에 놓는 방법
리팩토링할때 s. 스타일 컴포넌트 분리 추천한다 ⇒ 코드 수정할때 스크롤을 내려야 하므로
아토믹 패턴 도입 자체는 쉽지 않기 때문에 플러스 요소이다 ⇒ 규모있는 프로젝트에서 3번이상의 릴리즈에서 빛을 발한다 해봤다는것이 경험적 요소
그렇지만 폴더명을 통일성있게 해야한다 (폴더명 컨벤션을 정하던지) ⇒ 2번~3번째 릴리즈 쯤에서는 퇴사자가 발생하는데 아토믹 패턴을 도입하는 이유에 대해 생각해봐요 폴더를 클릭하지 않아도 무엇인지 알 수 있게끔 짜는게 유지보수성이 좋다
env 잘했다
못해도 80점이상의 개발은 해야한다 ⇒ 한 95점정도? 상위15%정도는 되는 것같다
실무에서 본인이 할수 있는 역량 -2일위 기한을 주기 때문에 클린코드 원칙을 모두 지킬 수는 없다

<개인질문>

시현님 디테일 포스트 => 코드 안더러워요 깔끔해요 잘못된 통합보다는 중복이 낫다 유즈 에이피아이 훅 잘 짰는지 => 잘 짰습니다
승지님 더미 데이터 활용 const chatdata= {'name': 리듬이 'preview': 'data' }, { 다른 데이터들도 똑같이 } 값을 props로 밑에서 받게 => 실제로 구현한다면 다시 만들어야되는 코드 마이너스는 아니지만 개선이 필요
민지님 서치 폴더트리가 잘못 되었다 크리티컬 하진 않고 마이너스 1,2~5점 정도? => 스타일 컴포넌트가 다른 폴더에 있다, 공유하고 있는 폴더로 옮겨주는게 맞지 않나 중복되는 스타일 폼은 빼는게 나은가 =>그렇지 않다 중복이 되더라도! 코드 복잡도가 더 높아지고 유지보수성이 X