[240116] 캡슐화
코드 리뷰 받고 배운 것
캡슐화에 대한 고려
배운 것
- 컴포넌트 역할이 1개 이상 된다고 느껴지면 한번 더 캡슐화에 대한 고민을 하자
상황
-
상위 컴포넌트에서는 배치 및 ON/OFF의 책임이 있었음
-
특정 함수에서 DropDown만 map 돌며 그려주고 있었는데, 조건에 따라 textField 그릴지 말지 추가적인 조건이 생겼었음
-
나의 경우, textField를 배치만 한다고 생각했기 때문에 캡슐화하지 않아도 된다고 생각함 / 조건문 추가 + textField를 넣어줌
-
캡슐화하면 좋을 것 같다는 피드백 받음 : DropDown을 그리냐 textField 그리냐 “어떻게 그려야 될지” 가 들어갔기 때문이다. 역할이 추가 됐기 때문에 해당하는 컴포넌트로 분리되면 좋을 것 같다고 하심
-
조건, 타입에 따라 DropDown / textField 그려주는 하위 컴포넌트 구성, 훨씬 역할이 분리됨을 느꼈다.
Leave a comment