- Notifications
You must be signed in to change notification settings - Fork173
상태를 변경하는 컴포넌트라면 반드시 ...#177
-
안녕하세요! 예를 들어, 저는 아래처럼 작성하는 걸 선호해요. // ✅ 선호하는 방식typeProps={value:Array<Term|TermGroup>;onChange?:(value:Array<Term|TermGroup>)=>void;}// ❌ 지양하는 방식typeProps={terms:Array<Term|TermGroup>;onTermsChange?:(value:Array<Term|TermGroup>)=>void;}// ❌ 지양하는 방식ContextAPI 제가 value / onChange 를 선호하는 이유는 다음과 같아요.
저는 요구 사항을 받으면 항상 아래처럼 구성하려고 해요.
혹시 다른 기준이나 의견이 있으신지 공유해주시면 감사하겠습니다! |
상태를 변경하는 컴포넌트라면 `value` 랑 `onChange` 를 사용하는게 권장된다 31% 상관없다 68% 32 votes· |
BetaWas this translation helpful?Give feedback.
All reactions
Replies: 1 comment
-
작성해주신 예시를 보면 해당 props가 들어가는 컴포넌트가 왠지 terms를 조작하는 컴포넌트일 것 같아요... 🤔 그렇다고 하면 한 발 더 나아가서.. 저는 컴포넌트 props뿐만 아니라 다른 곳에서도 비슷한 논리로 이름을 짓는 편이에요. 물론 domain specific한 용어들은 적용하면 안되겠고, '일반적인' 속성일 경우는 이렇게 이름 짓는 게 좋은 것 같아요. 그런 '일반적인' 속성명이 중복되는 상황이 발생하면, 객체의 계층 구조가 올바르지 않다는 걸 알려주기도 해요. // ✅typePost={id:string// ...author:Author}typeAuthor={id:string// ...}// ❌typePost={postId:string// ...authorId:string// ...} |
BetaWas this translation helpful?Give feedback.
All reactions
👍 3