Week 4 코드 리뷰 질문 작성

[step3-kakao-tech-campus]

Week 4 코드 리뷰 질문 작성

[JeonDoGyun]

• VLoginForm.tsx, VSignupInputForm.tsx를 보시면 boolean 값으로 랜더링을 할지말지 정해서 안내 텍스트가 나오도록 구현하였습니다. 저희 팀이 VAC 패턴을 써서 UI 부분과 비즈니스 로직 부분을 나눠서 짜고 있는데,
  • 해당 부분도 UI가 아닌 비즈니스 로직 부분으로 쳐서 뺀 다음 가져오는 것이 좋을지,
  • 아니면 랜더링만 관여하니까, View 안에 넣어도 상관없을지 궁금합니다.
• Tailwindcss 라이브러리를 써서 스타일을 주고 있습니다. flex, flex-col과 같은 부분은 사용되는 부분이 많아서 리팩토링을 한다면 해당 부분을 변수로 따로 지정하는 방법이 괜찮을까요?
• Atomic 디자인 패턴에 따라 atoms 레벨의 컴포넌트를 만들어 재사용하고 있습니다. Banner.tsx, Container.tsx 같은 경우, 컴포넌트를 따로 만들어서 부르지 않아도 코드에 큰 차이가 없을 것이라 생각되는데, 디렉토리의 복잡성을 염두에 두고 없애는 것이 나을까요?

[LimSumi]

• profileListHome에서 api 요청 하나로 페이지 전체에 대한 정보를 받도록 하려는데 혹시 긴급과 신규 부분에 대한 api를 나누어 한 요소에 대한 정보가 먼저 뜨도록 하는 게 좋을까요? 아니면 추후 스켈레톤을 구현하고 그냥 하나의 api로 구현하는 것이 더 효율적일까요?

[hjiwon]

• 각 이슈마다 ‘#’뒤의 숫자를 기반으로 브랜치를 생성하고 있는데,(현재 이슈로 이름을 짓는다면 feat/#25) 더 나은 브랜치 작명법이 있을까요?
• VAC 패턴으로 구현하면서 파일 구조가 장황해지는 경향이 있습니다. 오각형 차트를 구현한 src/components/atoms/RadarChart.tsx 파일을 보시면 코드가 길고 복잡한데, 사용되는 스타일은 width, height 뿐입니다. 너비와 높이만을 위해서 V컴포넌트를 추가적으로 생성하며 props 등을 정의하는 것이 꽤 수고스럽다는 생각이 들었습니다.
  그래서 ‘V컴포넌트가 필요해 보이는 것’만 V컴포넌트를 생성하기로 팀원들과 약속했습니다. 그랬더니 파일들이 뒤섞이며, 어떤 컴포넌트는 V컴포넌트가 있고, 어떤 건 없었습니다. 결과적으로 V컴포넌트를 생성하는 기준에 주관이 크게 작용한다고 느껴집니다.
  디렉토리 구조를 어떻게 짜는 것이 좋을까요? 기본 컴포넌트와 V컴포넌트가 짝지어지는 만큼 ATOM 폴더 하위에 컴포넌트별로 폴더를 하나씩 생성하는 것이 좋을까요?