Javascript-форум (https://javascript.ru/forum/)
-   Общие вопросы Javascript (https://javascript.ru/forum/misc/)
-   -   Сделайте, пожалуйста, код-ревью:)) (https://javascript.ru/forum/misc/81165-sdelajjte-pozhalujjsta-kod-revyu.html)

zzloyshkolnik 11.10.2020 11:09

Сделайте, пожалуйста, код-ревью:))
 
Приветствую!
Хотелось бы попросить провести код-ревью, кому не сложно:)
Там совсем немного:yes:

Github

Nexus 11.10.2020 13:33

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

zzloyshkolnik 11.10.2020 13:47

Спасибо большое:)

zzloyshkolnik 11.10.2020 13:53

Цитата:

Сообщение от Nexus (Сообщение 529679)
- используете parentElement вместо parentNode (посмотрите на поддержку браузерами);


Извини, а в чём отличие parentElement от parentNode? Только поддержкой?

Nexus 11.10.2020 14:00

https://stackoverflow.com/questions/...-parentelement


Часовой пояс GMT +3, время: 18:43.