-
Notifications
You must be signed in to change notification settings - Fork 5
Week5_day04 스크럼
Eunsol Lee edited this page Dec 18, 2020
·
1 revision
-
박은식
- 동작할 때 나타나는 에러들을 고쳤습니다
- 색상 초기화 시 init으로 문구가 나타남
- 경관 > 테마 > undo시에 테마가 남아있음
- 어떤 이슈가 추가적으로 발생할지는 모르겠습니다
- 동작할 때 나타나는 에러들을 고쳤습니다
-
이연정
- useEffect 관련해서 찾아보았지만 원하는 예시가 나오지 않았습니다.
-
위정훈
- 발표 시나리오, 예상 질문 써보고 있습니다
- (https://hackmd.io/vm27uI10Q5O1U6rgQFVKnw?edit)
- 코멘트 해주시면 감사하겠습니다, 크게 달라진건 없긴하지만..
- 결론...
- 발표 시나리오, 예상 질문 써보고 있습니다
-
이준희
- 잠들어버렸습니다... 죄송합니다.. 요즘 몸이 안 좋아서...ㅜㅠ
-
이은솔
- 내보내기/가져오기 동작이 일치해야할 것 같아서, 마커를 내보내기에 추가하려고 시도했습니다.
- useMarkerFeature의 registerMarker 함수의 재사용이 필요한데, useEffect로 인해서 영향을 다시 받아서 한번 더 분리가 필요할 것 같습니다.
- custom Hook 재사용을 위해서는 최대한 역할을 작게 갖고 있어야 하는구나.. 싶었습니다.
- all 초기 색상 처리
- marker 내보내기 가져오기 useMarkerFeature에서 useEffect 분리 필요
- 멘토님 요청드리기
언제 컨테이너를 도입해야하나요? 우선 앱을 만들때 프레젠테이션 컴포넌트를 먼저 만드세요. 그러면 너무 많은 props 를 중간 컴포넌트로 보내야 한다는 것을 깨닫게 될것입니다. 전달받은 props 를 사용하지 않고 아래로 전달하기만 하는 컴포넌트나 자식 컴포넌트가 더 많은 데이터를 필요로 할때 모든 중간 컴포넌트를 재구성해야하는 컴포넌트들이 있다는 것을 알게 될것입니다. 바로 이 때 컨테이너 컴포넌트를 도입해야합니다. 데이터나 아무 상관없는 중간 컴포넌트에 대해 걱정이 없는 leaf 컴포넌트의 행위가 담긴 props 를 얻을 수 있는 방법이 될 것입니다.
presentational 컴포넌트의 깊이가 깊어질수록 계속 최상위에 있는 container로부터 받은 props를 전달해주어야 하는 점 때문에 컴포넌트가 받아야 하는 props가 많아져서 지저분해진다는 것과 로직이 복잡해지다 보니 container가 점점 비대해진다는 것이 예상되었다.
- 왜 우리는 Custom Hook을 사용하기 시작했을까?
- 복잡하게 만들었다.
- 무엇이?
-
폴더 구조와 네이밍? Container, Presentational이 하나의 쌍처럼 사용함
-
컴포넌트 이름이 우선 너무 길어졌고, 이렇게 사용해도 되나요??하는 혼란을 주었음
-
개발을 위한 패턴이 아닌 패턴을 위한 패턴적용..?
-
컴포넌트 depth가 깊어지면서,
-