Updated by
Deleted
testwork-marks- Ignore whitespace
-Переходи на использование typescript (в идеале), можно сначала попробовать javascript нового поколения. Назвается es2015. Для того чтобы заставить его работать в браузере надо подключить webpack, babel и кучу сопутствующих технологий. Также есть пакет react-bootstrap,
-который предоставляет набор готовых компонент для использования реакта. Есть ещё тулкит material-ui.com
-index.html: в нём должно быть только подключение скриптов и `<div id="root />`. Остальное всё должно создаваться реактом.
-Table.getInitialState: здесь ты по сути копируешь props в локальный state. Для реакта это в принципе антипаттерн,
-состояние приложения должно быть в единственном экземпляре, то есть храниться в App. Чтобы его менять компонент Table должен
-deleteItem, editItem и прочие. В данном случае ты *мутируешь* состояние компонента. Это можно делать, но не желательно, желательно
-вместо изменения массива создавать на его основе новый. Функции splice, push изменяют объект. Перед измененим массива бери его копию
-ItemForm. Тут всё хуже :( Тебе не надо присваивать в инпуты значения руками. Реакт может это делать за тебя, всё что тебе надо
Added
testwork-marks.md- Ignore whitespace
+Переходи на использование typescript (в идеале), можно сначала попробовать javascript нового поколения. Назвается es2015. Для того чтобы заставить его работать в браузере надо подключить webpack, babel и кучу сопутствующих технологий. Также есть пакет react-bootstrap,
+который предоставляет набор готовых компонент для использования реакта. Есть ещё тулкит material-ui.com
+index.html: в нём должно быть только подключение скриптов и `<div id="root />`. Остальное всё должно создаваться реактом.
+Table.getInitialState: здесь ты по сути копируешь props в локальный state. Для реакта это в принципе антипаттерн,
+состояние приложения должно быть в единственном экземпляре, то есть храниться в App. Чтобы его менять компонент Table должен
+deleteItem, editItem и прочие. В данном случае ты *мутируешь* состояние компонента. Это можно делать, но не желательно, желательно
+вместо изменения массива создавать на его основе новый. Функции splice, push изменяют объект. Перед измененим массива бери его копию
+ItemForm. Тут всё хуже :( Тебе не надо присваивать в инпуты значения руками. Реакт может это делать за тебя, всё что тебе надо