Leezero3 / todo-list

https://todo-list-leezero3.vercel.app
0 stars 0 forks source link

TodoList의 box 컴포넌트 중첩 #1

Open Leezero3 opened 1 year ago

Leezero3 commented 1 year ago

App.js 에서 TodoList 가 나열되는 코드가 중첩되고 있었습니다. 해결을 위해 TodoBox로 컴포넌트 분리를 하였고 해당 컴포넌트들로 아래와 같은 props들을 넘겨주었습니다.

todos={todos.filter((todo) => todo.isDone)}
RemoveButtonHandler={RemoveButtonHandler}
ClearButtonHandler={ClearButtonHandler} 

그 결과 이전과 같이 동작하고 있으며 각 코드의 길이 또한 줄어든 것을 확인하였습니다.

endol007 commented 1 year ago

lv1에 대한 코멘트 남깁니다! 같은 컴포넌트를 재사용하고 로직들을 상위 컴포넌트에 모아놓은것도 깔끔하고 좋은 것 같네요

보완할 점으로는 html태그를

태그 이외에도 다른 태그들을 잘 활용해보면 좋겠습니다