#3 JS22(jsx1 и jsx2): 8

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

http://gitlab.a-level.com.ua/yankevich0210/homework/src/master/react/hw2/src/App.js#L62 - через && это все проще;

http://gitlab.a-level.com.ua/yankevich0210/homework/src/master/react/hw2/src/App.js#L150-L154 - нет смысла заводить state, который зависит от другого state, дописывать useEffect и т.п.

http://gitlab.a-level.com.ua/yankevich0210/homework/src/master/react/hw2/src/App.js#L223-L240 - в целом верно, но:

  • число таймера надо вычислять по разнице между текущим моментом времени и начальным, иначе будет "плыть" точность;
  • TimerComponent получает время в секундах, а не часы/минуты/секунды;

http://gitlab.a-level.com.ua/yankevich0210/homework/src/master/react/hw2/src/App.js#L315-L318 - не надо так. Просто TimerControl должен отрисовать в себе TimerContainer или же через onStart отдать наверх время, которое будет передано в TimerContainer

http://gitlab.a-level.com.ua/yankevich0210/homework/src/master/react/hw2/src/App.js#L62 - через `&&` это все проще; http://gitlab.a-level.com.ua/yankevich0210/homework/src/master/react/hw2/src/App.js#L150-L154 - нет смысла заводить `state`, который зависит от другого `state`, дописывать `useEffect` и т.п. http://gitlab.a-level.com.ua/yankevich0210/homework/src/master/react/hw2/src/App.js#L223-L240 - в целом верно, но: - число таймера надо вычислять по разнице между текущим моментом времени и начальным, иначе будет "плыть" точность; - TimerComponent получает время в секундах, а не часы/минуты/секунды; http://gitlab.a-level.com.ua/yankevich0210/homework/src/master/react/hw2/src/App.js#L315-L318 - не надо так. Просто `TimerControl` должен отрисовать в себе `TimerContainer` или же через onStart отдать наверх время, которое будет передано в `TimerContainer`
Sign in to join this conversation.
No Label
No Milestone
No assignee
1 Participants
Loading...
Cancel
Save
There is no content yet.