Сделайте, пожалуйста, код-ревью:))
Приветствую!
Хотелось бы попросить провести код-ревью, кому не сложно:) Там совсем немного:yes: Github |
Мне решение не понравилось :(
- сайд-эффекты; - вы не определились ставить точку с запятой или нет (+/-); - странное описание представления элемента .task (+/-); - игнорируете атрибут [name] полей формы используя вместо этого их индекс; - используете parentElement вместо parentNode (посмотрите на поддержку браузерами); - используете метод remove вместо removeChild (+/-); - двигаетесь по дереву элементов практически всегда игнорируете методы поиска элементов по селектору, предпочитая этому прямое обращение к родителю и обращение к конкретному ребенку по его индексу ( что будет, если немного изменить верстку? ); - вместо запуска в конце документа определенной функции лучше использовать событие документа DOMContentLoaded, тогда скрипт и в head можно поместить. |
Спасибо большое:)
|
Цитата:
Извини, а в чём отличие parentElement от parentNode? Только поддержкой? |
|
Часовой пояс GMT +3, время: 18:43. |