1차 프로젝트 Live Code Review 후 메모

hyeonze·2022년 1월 5일
0

1/6 내일은 거의 merge day, 오전에 PR마무리, 차후 전체흐름 파악 및 AWS배포

src>config.js 파일 import를 통해 fetch할 주소와 url등 한 파일에서 관리하기

공통 함수나 변수의 경우 컴포넌트 밖에서 선언하기도 함. 특히 전역변수의 경우(사진크기 등) const 대문자로 밖아버리기. 또한 재사용을 위해 config.js나 utils.js에서 선언하기도 함.

WIP(Working in process) : 작업 중 브랜치 이동 시 커밋메시지

리베이스 : 커밋을 묶는 작업

클래스네임에서 카멜케이스 사용시 장점 : 부모-자식 관계를 명확히 해줌

react에서 공백 사용하고싶을 때 : {' '}

주석 자제하는게 좋지만 빈 css선택자등이 많을 때 ToDo:~ 로 작성해주면 좋음. PR에서도 언급해주면 나도 친절한 개발자가 될 수 있음.

동적 라우팅에서 변수명을 일치시켜줘야함. 프론트끼리 회의 필요.

드롭다운 메뉴 구현 시 상위 메뉴에서 map하면서 하위메뉴가 있을 경우에 하위메뉴를 map하는 로직을 사용할 수 있음.(갬성팀 민욱님 2단 nav참조)

드롭다운 위치지정 시 => 일반적으로 부모태그 하위에 위치시킨 후, 부모에 position: relative;, 자식에 position: absolute;로 처리함. 본 프로젝트에서 react에 익숙지 않아 map으로 리스트를 구현할 것을 대비해 하위 태그에 위치시키지 못해 미흡하게 처리된 부분이 발생한 것 같음.

useEffect의 의존성배열에 함수를 추가하면 무한루프에 빠짐. 이를 방지하기 위해 useCallback 사용.

profile
Advanced thinking should be put into advanced code.

0개의 댓글

관련 채용 정보