#9 JS19(module): 8

Open
opened 1 year ago by gitgod · 0 comments
gitgod commented 1 year ago

http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L91 - тут желательно использовать инфрмацию, сохраненную благодаря localStoredReducer;

http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L121 - запись в localStorage лишняя/не обязательная в силу наличия localStoredReducer

http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L147 - тут нужно учитывать предыдущее значение, которое было ранее. ADD значит добавить, добавить к тому, что там уже было;

http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L168 - мутировать state плохая идея;

http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L531-L533 - в action не должно быть работы с DOM

http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L549-L551 - аналогично)

http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L91 - тут желательно использовать инфрмацию, сохраненную благодаря `localStoredReducer`; http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L121 - запись в `localStorage` лишняя/не обязательная в силу наличия `localStoredReducer` http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L147 - тут нужно учитывать предыдущее значение, которое было ранее. `ADD` значит _добавить_, добавить к тому, что там _уже_ было; http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L168 - мутировать `state` плохая идея; http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L531-L533 - в action не должно быть работы с DOM http://gitlab.a-level.com.ua/BohdanBaberia/Js.Homework/src/main/moduleRedux/main.js#L549-L551 - аналогично)
Sign in to join this conversation.
No Label
No Milestone
No assignee
1 Participants
Loading...
Cancel
Save
There is no content yet.