31Vector31 / react-training

Hooks, Axios, HOC, Redux, Context, Reselect, Router.
0 stars 0 forks source link

review GuestListHooks #5

Closed shevchuknine closed 2 years ago

shevchuknine commented 2 years ago

https://github.com/31Vector31/react-training/blob/7ab7bc4de5cdd3ed371b9eac5178bfab4242edfb/src/GuestListHooks/GuestList.js#L11 если в функцию изменения состояния передавать колбек, можно избавиться от зависимости guests используй для контатенации массивов распыление [...oldArray, newElem] https://github.com/31Vector31/react-training/blob/7ab7bc4de5cdd3ed371b9eac5178bfab4242edfb/src/GuestListHooks/Form.js#L23 колбеки в форме делают одинаковые действия с разными установщиками состояния. можно придумать как красиво обернуть _ https://github.com/31Vector31/react-training/blob/7ab7bc4de5cdd3ed371b9eac5178bfab4242edfb/src/GuestListHooks/Table.js#L4 https://github.com/31Vector31/react-training/blob/7ab7bc4de5cdd3ed371b9eac5178bfab4242edfb/src/GuestListHooks/Table.js#L6 https://github.com/31Vector31/react-training/blob/7ab7bc4de5cdd3ed371b9eac5178bfab4242edfb/src/GuestListHooks/Form.js#L15 не забывай про деструктцризацию, в плане потери контекста тут терять нечего)