search

lafest / react-diary

1 stars 0 forks source link

[리뷰#2] ContentComponent.js > diaryList.filter() #2

Open kmsbernard opened 4 years ago

kmsbernard commented 4 years ago

https://github.com/lafest/react-diary/blob/da5b21376bd5e07cf98d4d204e51a1fd291914b9/src/components/Content/ContentComponent.js#L109

<DiaryItem>을 렌더하기 위한 validator를 구현하신 것으로 보입니다. validator 구현의 필요성을 느끼신 이유/과정을 설명해주실 수 있나요?

lafest commented 4 years ago

필터 기능을 이용해 언제부터/언제까지/특정태그를 담고있는 DiaryItem만 렌더링하기 위해 사용했습니다. 세 가지 조건을 모두 체크해야하는데 어떻게 하는 게 효율적일까 생각하다가, isValidate라는 변수를 만든 후, element가 각각의 조건을 만족하는지 체크하면서 조건만족여부를 해당 변수에 저장해둔 후 리턴하면 되겠다고 생각했습니다. validator를 기본적으로 true로 지정한 다음, 세 가지 조건에 대해 충족여부를 따진 후 만족하지 않을 시 false를 리턴해 렌더링할 목록에서 빼는 방식으로 구현했습니다. 만약 포함해야하는 조건이 아니라면 해당 값은 null이므로, 우선 조건값이 null인지 확인한 후에 값이 존재한다면 DiaryItem이 들고있는 값과 비교해서 유효한 아이템인지 여부를 판단했습니다.