Привет ребят) Выбрал сегодня время, написал в песочнице мини туду,

Question

Привет ребят) Выбрал сегодня время, написал в песочнице мини туду,

чтобы опробовать базовые знания реакта. Я сейчас работаю 6 дней в неделю и время не так много, но продолжаю потихоньку углубляться в реакт.

Если есть возможность у кого помочь мне, посмотреть мой код в песочнице и сказать где есть косяки если вообще есть. Там всего пару экшенов, один селектор и несколько базовых компонентов. Кода не много, но думаю есть места где можно указать на косяки.

вот песочница: https://codesandbox.io/s/charming-cloud-tlhiy?file=/src/components/containers/TodoContainer/index.jsx

Подсобите пожалуйста ребят)

#frontend #javascript #programming #react.js #russian

0

04.05.2020

3 ответов

12 просмотров

artalar

Для начала вполне хорошо. Только функциия getNextId очень плохо написана - будет бага при сортировке или удалении последнего элемента

0

04.05.2020

Sergey Sova

1. убрать strictmode, он меняет работу реакта, появляются расхождения в исполнениях хуков (надо проверять, могли пофиксить. С этим @zerobias много раз сталкивался) 2. выкинуть export default для компонентов, если к этому не принуждает фреймворк 3. я именую все файлы в param-case. не надо выдумывать новый стиль для каждого типа файлов 4. нет смысла смешивать css и styled. Выбери что-то одно В остальном всё ок. Мои личные советы: - Не разделять на контейнер и компонент там, где нет отдельного переиспользования компонента и/или контейнера. Ты разделишь код, который в этом не нуждается. Логика будет размазана по двум файлам, хотя изменения ты будешь почти всегда вносить в оба файла, что собственно и говорит о том, что код должен лежать в одном месте. - checkbox можно отверстать 1 инпутом и 1 дивом, тебе не нужен отдельный checker. Просто стилизуй input. - погрузись в @AtomicDesign, это методология разделения визуальных компонентов на слои, по сложности и зависимости друг от друга. Сильно упрощает разработку любого проекта. А сразу за этим следует, что нужно разделять компоненты чисто UI, то есть без логики, от компонентов с логикой. Ну и это родило методологию FeatureSlices. Но это позже, скорее всего сам дойдешь. Ну или как будешь готов, пингуй

0

04.05.2020

Вячеслав · Accepted Answer

Вячеслав

- почему не TS? - export default... Накидают тебе сейчас кирпичей - react-addons-update попробуй для работы с immutable data - не ради холивара, но я бы не использовал css-in-js (styled) - для первой тудушки - норм

0

04.05.2020

169 похожих чатов

Привет ребят) Выбрал сегодня время, написал в песочнице мини туду,

3 ответов

Похожие вопросы