Задать вопрос
@Ivan2507

Ревью кода. Что можно улучшить в этом коде?

Здравствуйте, я изучаю front-end разработку 2 .5 месяца, решил перейти на практику и написал todo list за часов 8
1) Напишите, пожалуйста, свое мнение о коде и как его можно улучшить (желательно по html, css тоже, но в приоритете JS)
2) Хочу начать изучение React, уже можно начинать или лучше ещё попрактиковаться на JS?

Ссылка на код: https://codepen.io/domarchuk77/pen/xxRoMBY
  • Вопрос задан
  • 186 просмотров
Подписаться 1 Простой Комментировать
Ответ пользователя Сергей Соколов К ответам на вопрос (4)
sergiks
@sergiks Куратор тега JavaScript
♬♬
В класс Task впихнули всё. А лучше бы как-то разделить: вот Task, их может быть ни одного или несколько. Вот App – это приложение, форма создания новой задачи; может, обработчик всех событий, которые всплывают из задач, в т.ч.; коллекция созданных задач; их сохранение в LocalStorage; отрисовка части задач в соответствие с фильтром.

Внутри класса у методов общий префикс названия taskЧто-то-там – лишний, имхо.

В коде жосско прописаны названия классов элементов, с которыми работать, где искать, и т.п. Может, лучше делать класс независимым от разметки и передавать в него уже созданные элементы. Если же элементы создаются внутри класса, сохранять референсы к ним.

p.s. React да, можно. Или Vue.
Ответ написан
Комментировать