Показать сообщение отдельно
  #2 (permalink)  
Старый 11.10.2020, 13:33
Профессор
Отправить личное сообщение для Nexus Посмотреть профиль Найти все сообщения от Nexus
 
Регистрация: 04.12.2012
Сообщений: 3,748

Мне решение не понравилось
- сайд-эффекты;
- вы не определились ставить точку с запятой или нет (+/-);
- странное описание представления элемента .task (+/-);
- игнорируете атрибут [name] полей формы используя вместо этого их индекс;
- используете parentElement вместо parentNode (посмотрите на поддержку браузерами);
- используете метод remove вместо removeChild (+/-);
- двигаетесь по дереву элементов практически всегда игнорируете методы поиска элементов по селектору, предпочитая этому прямое обращение к родителю и обращение к конкретному ребенку по его индексу ( что будет, если немного изменить верстку? );
- вместо запуска в конце документа определенной функции лучше использовать событие документа DOMContentLoaded, тогда скрипт и в head можно поместить.
Ответить с цитированием