Как передать значения блоков?

Question

[Andrej Sharapov]

Подскажите, как правильно составить скрипт?

Задачи:
1. Вывести в value='' input значение background по клику на блок
2. При этом же клике должен меняться цвет красной линии

Answer 1

[Никита Егоров]

Так?

Comments

[Andrej Sharapov]

да, отлично. Правда я думал, код будет проще)

[Дмитрий Беляев]

когда уже на тостере будет кнопка "не нравится"?..
а еще лучше, когда уже гугл с яндексом будут снижать в выдаче за неоптимизированный js?..

[Andrej Sharapov]

Дмитрий Беляев, а что не так с его кодом? Он выполняет свои функции как надо, немного переделал под себя, но работает.
Спросил с целью расширения познаний

[Никита Егоров]

Andrej Sharapov, вообще конечно лучше использовать делегирования событий, и повесить один обработчик на контейнер, тогда будет оптимизация.

[Дмитрий Беляев]

Andrej Sharapov, трансформация дом коллекции в массив, только ради того чтобы сделать у него map, который ничего не возвращает + создание функции в цикле
как итог - вместо O(n) у нас сложность O(3n), вместо 1 функции и 1 дом коллекции из n элементов у нас в памяти n функций, 1 дом коллекция из n элементов и 2 массива из n элементов - и вот Ваш сайт уже жрет мою оперативку и мои такты проца, а как следствие электричество, за которое я плачу свои кровнозаработанные, и при его посещении я захочу Вас проклясть и побыстрее закрыть Ваш тормозящий сайт

[Дмитрий Беляев]

Никита Егоров, в предложенном Вами варианте сложность выходит O(1) - что еще лучше чем у меня, ибо вместо цикла у Вас просто 1 действие для подписки на события
Ну от поиска в DOM мы тут абстрагируемся, ибо мало на него можем повлиять

[Никита Егоров]

Дмитрий Беляев, ай, комент удалил, хотел подправить.

Примерно то что было в нем ->
Написал что у меня сложность O(n), а у Дмитрия O(n2), и спросил так ли это.

[Дмитрий Беляев]

Никита Егоров, я успел прочесть.
добавлю еще, что я бы избавился от closest если нужна поддержка старья, полифил не очень шустро работает
тем более можно просто заменить на проверку наличия класса grad у event.target

if(!event.target.classList.contains('grad')) { return; }

[Никита Егоров]

Дмитрий Беляев,

тем более можно просто заменить на проверку наличия класса grad у event.target

Да но смысл, ведь это только увеличит код, так как получить grad нам все равно нужно, а раз уж получили, то можно просто проверить, что мы получили.

Answer 2

[Дмитрий Беляев]