[240116] 캡슐화

코드 리뷰 받고 배운 것

캡슐화에 대한 고려

배운 것

  • 컴포넌트 역할이 1개 이상 된다고 느껴지면 한번 더 캡슐화에 대한 고민을 하자

상황

  1. 상위 컴포넌트에서는 배치 및 ON/OFF의 책임이 있었음

  2. 특정 함수에서 DropDown만 map 돌며 그려주고 있었는데, 조건에 따라 textField 그릴지 말지 추가적인 조건이 생겼었음

  3. 나의 경우, textField를 배치만 한다고 생각했기 때문에 캡슐화하지 않아도 된다고 생각함 / 조건문 추가 + textField를 넣어줌

  4. 캡슐화하면 좋을 것 같다는 피드백 받음 : DropDown을 그리냐 textField 그리냐 “어떻게 그려야 될지” 가 들어갔기 때문이다. 역할이 추가 됐기 때문에 해당하는 컴포넌트로 분리되면 좋을 것 같다고 하심

  5. 조건, 타입에 따라 DropDown / textField 그려주는 하위 컴포넌트 구성, 훨씬 역할이 분리됨을 느꼈다.

Categories:

Updated:

Leave a comment