View - Coding Conventions #14
Replies: 7 comments 34 replies
-
1) 폴더 구조회의를 통해 아래와 같이 의견이 나왔습니다.
관련되어 다른 의견이 있으신 분은 자유롭게 reply로 코멘트 해주세요! |
Beta Was this translation helpful? Give feedback.
-
|
Components 폴더 만들어서 feature단위로 폴더 만들고 그 하위에 feature를 또 구분하는 컴포넌트가 있으면 좋을 것 같아요!!! Components/VerticalClusterList/index.tsx ㅎㅎ,, 다른 의견도 들어보고싶습니다!! |
Beta Was this translation helpful? Give feedback.
-
|
@jin-Pro 님 의견에 동의합니다~ |
Beta Was this translation helpful? Give feedback.
-
|
저도 @jin-Pro 님 의견에 동의합니다! |
Beta Was this translation helpful? Give feedback.
-
2) Lint 설정회의를 통해 아래와 같이 의견이 나왔습니다.
관련되어 다른 의견이 있으신 분은 자유롭게 reply로 코멘트 해주세요! |
Beta Was this translation helpful? Give feedback.
-
|
engine팀이 잘해주셔서 따라가도 좋을 것 같아요! |
Beta Was this translation helpful? Give feedback.
-
|
진행하면서 rule 추가하는 건 자주 하지는 않았으면 좋겠습니다 ! |
Beta Was this translation helpful? Give feedback.
-
3. 컴포넌트 선언 키워드functions vs const |
Beta Was this translation helpful? Give feedback.
-
|
(someone..) 컴포넌트는 주로 const로 util함수는 주로 function 로 사용합니다! |
Beta Was this translation helpful? Give feedback.
-
|
@jejecrunch const의 경우 컴포넌트를 익명함수로 선언되어 발생하는 단점을 아실까요??? 컴포넌트가 호이스팅 이슈로 인해 발생하는 불이익은 없을 것 같아서요! 음!! ㅎㅎ,, 현업에서는 어떻게 사용하시나요??? |
Beta Was this translation helpful? Give feedback.
-
|
저는 실무에서 거의 const로 사용하고 있습니다~! |
Beta Was this translation helpful? Give feedback.
-
|
저는 컴포넌트로 const로 사용하고 혹시 명시적으로 바인딩 해줄 일이 있지 않을까 싶어서 유틸함수는 function으로 사용하는데요. 아직까지 한 번도 명시적으로 바인딩을 해줄 일은 없었습니다..ㅎㅎ |
Beta Was this translation helpful? Give feedback.
-
|
@jin-Pro 님 말씀대로 생각해보면 컴포넌트라 호이스팅 이슈는 딱히 없을 것 같아요 ! |
Beta Was this translation helpful? Give feedback.
-
4. 타입 선언 방법type vs interface |
Beta Was this translation helpful? Give feedback.
-
|
감사합니다 ㅎㅎ 읽어보겠습니다! |
Beta Was this translation helpful? Give feedback.
-
|
공식문서에서 type을 고려하라는것 같아요! |
Beta Was this translation helpful? Give feedback.
-
|
저도 개별 prop은 type으로 작성하더라도 interface로 통일하면 좋을 것 같습니다! |
Beta Was this translation helpful? Give feedback.
-
|
그러면 props들의 경우는 type으로 작성하고 그외의 타입은 interface로 가는 걸까요?! |
Beta Was this translation helpful? Give feedback.
-
|
이부분 type으로 통일하여 작성, 이후 확장 가능성이 있다면 interface로 변경하기로 8/31 오프라인 미팅에서 결정되었습니다 |
Beta Was this translation helpful? Give feedback.
-
5. 머지 기준
|
Beta Was this translation helpful? Give feedback.
-
|
그 외에서 null과 undefined는 의미적 차이가 중요할 것 같아요~ 이후 값이 들어오지만, 그 값이 동일한 타입을 유지한다면 undefined를 사용하는게 좋을 것 같아요! |
Beta Was this translation helpful? Give feedback.
-
|
null과 undefined 관련해 개발자가 명시적으로 입력할 때는 undefined 사용을 지양해야한다고 알고 있습니다.
cf. |
Beta Was this translation helpful? Give feedback.
-
|
음! 납득이 되는거 같아요! ㅎㅎ 감사합니다 |
Beta Was this translation helpful? Give feedback.
-
한슬님이 해주신 말씀도 맞긴 한데 d3의 일부 함수는 값의 타입으로 null을 허용하지 않아 undefined를 사용할 수 밖에 없더라구요! |
Beta Was this translation helpful? Give feedback.
-
|
@pshdev1030 오 몰랐네요 감사합니다..! |
Beta Was this translation helpful? Give feedback.
-
6. Material-UI 사용 |
Beta Was this translation helpful? Give feedback.
-
|
시간적 여유가 없기도 해서 저는 쓰는게 낫다는 생각입니다! |
Beta Was this translation helpful? Give feedback.
-
하지만 반응형을 고려하지 않아도 되면 쓰지 않아도 좋을 것 같습니다/ |
Beta Was this translation helpful? Give feedback.
-
|
추가로 chakra ui가 있고, blueprint가 데스트탑 어플리케이션을 만드는 용도로 반응형을 고려하지 않는걸로 알고있습니다..! |
Beta Was this translation helpful? Give feedback.
-
|
MUI를 써도 러닝커브가 그렇게 크지 않다면 쓰는 것도 괜찮다고 생각합니다 ! @pshdev1030 님 말씀대로 시간적 여유가 없기도 하고 저 같은 경우는 MUI를 써본 경험은 있는데 익숙하게 쓸 수는 없어서 다른 분들은 어떨지 모르겠네요 ... 사용하는 건 그렇게 어렵지 않았는데 타입스크립트에서는 써본 경험이 없어서 ㅜㅜ MUI 아니더라도 Ant Design같은 경우는 경험이 많긴 하지만 타입스크립트에서는 처음이라 애매하네요 ㅜㅜ |
Beta Was this translation helpful? Give feedback.
-
7. 네이밍 컨벤션네이밍 컨벤션도 정하고 가면 좋을 것 같아서 discussion에 추가하겠습니다 ! 맞추고 가면 좋을 것 같은 네이밍 컨벤션 reply 로 달아주시거나 edit으로 편집해주세요 !
|
Beta Was this translation helpful? Give feedback.
-
|
type의 네이밍은 어떻게 하면 좋을까요? 저는 우선 그 객체가 정말 무엇을 나타내는지 명사를 적었는데 다른 분들은 어떤지 궁금하네요. |
Beta Was this translation helpful? Give feedback.
-
|
컴포넌트 머지 후 css 네이밍 컨벤션 관련해 스타일이 중복 적용되는 경우가 발생하는데
|
Beta Was this translation helpful? Give feedback.
-
이렇게 가는 거 좋은 거 같습니다. 생각해보니까 데이터명이 겹치는 곳이 많은데 css 선택자 명을 어떻게 할지 결정하지 않았었네요. |
Beta Was this translation helpful? Give feedback.
-
|
넵! 기한, 영진, 지혜님 회의에서 위 컨벤션에 맞춰가기로 결정하였습니다. |
Beta Was this translation helpful? Give feedback.
-
목표
8/16일 회의를 기반으로, 함께 논의해야하는 컨벤션 항목에 대한 의견 나누기
참고사항
자유롭게 의견을 남겨주세요!
Beta Was this translation helpful? Give feedback.
All reactions