|
Сообщение от dmitry111
|
|
Вот сделал небольшой сприпт с использованием ООП, покритикуйте
|
Мне не понравилось вот это:
add.onclick, item.onmouseover ...
https://developer.mozilla.org/ru/doc...dEventListener
show и hide можно объединить в визибилити:
Unit.prototype.Visibility(item, showHideFlag)
Не понравилось вот это:
items[i] = new Unit(items[i].getAttribute("data-text"), items[i]);
Больше подходит для тестов, чем для нормального кода.
Использование реги считаю неоправданным, покрайней мере можно не выносить её в переменную и вообще разбить по
.split(/[.!?]/)[0]
Правда можно вынести её в метод и назвать его getFirstSentence
Реги нужно отделять от вызова только если они многократно используются иначе их неудобно искать в коде и не ясно, что происходит в этом месте. Но это ИМХО.
В остальном мне такой подход нравиться гораздо больше, чем
DjDiablo, т.к. он правильно допиленный легко расширяем без потери читабельности кода.
Мне важно, чтобы я мог быстро расширить любой код и не ломать голову мотая скролл по текущей функции в поисках подфункции, когда можно её вынести в метод. Каюсь, сам иногда делаю подфункции, но стараюсь делать это редко.