Изменить свойства одного элемента по клику

@ne_Proger · Регистрация: 01.11.2021

Студворк — интернет-сервис помощи студентам

Здравствуйте!

На страницу в цикле выводится список из некоторого количества элементов. При нажатии на любой из элемент списка необходимо изменить стили именно нажатого элемента.

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
let ul = document.createElement('ul');
  ul.className = "list";
  body.append(ul);
 
  for (let i = 0; i < countCard; i++) {
    let li = document.createElement('li');
    li.className = "list__card";
    ul.append(li);
 
    let h2 = document.createElement('h2');
    h2.className = "list__card__header ";
    h2.textContent = arr[i];
    li.append(h2);
 
    const clickCard = document.querySelector("li");
    clickCard.addEventListener("click", (event) => {
      clickCard.className = "list__card--rotate";
      h2.className = "list__card__header--rotate";
    });
  };

Это позволяет изменить стили, но только у самого первого элемента. И нажимается соответственно тоже только первый элемент. Что я должен изменить, чтобы можно было кликнуть любой из элементов?

@Zubbol · 17.07.2024, 13:11

Добрый день. Дело в том, что document.querySelector берёт первый попавшийся элемент, если у вас имеется несколько подходящих под условие элементов на странице, в данном случае элементов с тегом li. И так как вы делаете это в цикле, то подозреваю, что на первый элемент вашего списка было навешено несколько обработчиков клика. Можете проверить мою догадку через console.log на 17 строке.

Вы могли бы использовать document.querySelectorAll, который соберёт коллекцию имеющихся элементов li на странице, по которой можно будет пройти циклом и повесить обработчики на каждый элемент, но:
1. у вас получится цикл в цикле, а это плохая практика;
2. не решена проблема закрепления нескольких обработчиков за одним и тем же элементом, только теперь на каждый элемент будет вешаться по несколько обработчиков

Вместо этого предлагаю совсем отказаться от каких-либо способов поиска элементов li на странице, ведь вы уже сохранили искомый элемент li в let li(советую изменить на const, а имя поменять на ту же clickCard), а значит по этому же имени можно обращаться напрямую к элементу. Если не заморачиваться на нейминге и зря использованной переменной, то достаточно стереть всё на 15 строке, а вместо clickCard ниже по коду использовать li - этим вы решите все проблемы)

Ещё немного побубню: советую коллбэк, передаваемый вторым аргументом в addEventListener, сохранить куда-то отдельно в константу - вдруг вам понадобится потом снять обработчик. С анонимной функцией это будет затруднительно. Но для этого придётся немного унифицировать сам коллбэк)

@voraa · 17.07.2024, 13:35

Делегировать событие клика

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
let ul = document.createElement('ul');
  ul.className = "list";
  body.append(ul);
 
  for (let i = 0; i < countCard; i++) {
    let li = document.createElement('li');
    li.className = "list__card";
    ul.append(li);
 
    let h2 = document.createElement('h2');
    h2.className = "list__card__header ";
    h2.textContent = arr[i];
    li.append(h2);
  };
  
  ul.addEventListener("click", (event) => {
    const li = event.target.closest('li');
    if (li) {
        const h2 = li.querySelector('h2');
        li.className = "list__card--rotate";
        h2.className = "list__card__header--rotate";
    }
  });

@ne_Proger · 17.07.2024, 14:17 **[ТС]**

voraa, спасибо. Получилось.
И еще спрошу. Как можно посчитать количество кликов? Кликнули один элемент, затем второй и т.д. Если кликов задангое количество, то возвращаем классы по умолчанию

@Zubbol · 17.07.2024, 14:19

Понял, вам не разобраться нужно, а готовое)

@ne_Proger · 18.07.2024, 08:32 **[ТС]**

Zubbol, нет, не готовое.
Нужна только помощь.

На данный момент вот что получилось у меня

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 let a = 0; // счетчик кликов
  ul.addEventListener("click", (event) => {
    const li = event.target.closest('li');
    if (li) {
      a = a + 1;
      console.log(a);
      const h2 = li.querySelector('h2');
      if (a < 3) {
        li.className = "list__card__rotate";
        h2.className = "list__card__header__rotate";
      } else {
        const allCard = document.querySelectorAll('li');
        const allCardHeader = document.querySelectorAll('h2');
        allCard.forEach(list__card__rotate => {
          list__card__rotate.className = 'list__card';
        });
        allCardHeader.forEach(list__card__header__rotate => {
          list__card__header__rotate.className = 'list__card__header';
        });
        a = 0;
      }
    }
  });

Правда счетчик срабатывает и при повторном нажатии на элемент. А лучше бы исключить это

krvsa · 18.07.2024, 08:56

Сообщение от ne_Proger

Правда счетчик срабатывает и при повторном нажатии на элемент. А лучше бы исключить это

Так ты проверяй выбран элемент или нет... Если выбран - ничего не делай.

@voraa · 18.07.2024, 09:18

Что такое можно

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
const maxSelCard = 3;
ul.addEventListener("click", (event) => {
    const li = event.target.closest('li');
    if (li) {
        const selCard = document.querySelectorAll('ul>li.list__card__rotate');
        console.log(selCard.length);
        if (selCard.length < maxSelCard) {
            li.className = "list__card__rotate";
            const h2 = li.querySelector('h2');
            h2.className = "list__card__header__rotate";
        } else {
            selCard.forEach(card => {
                card.className = 'list__card';
                card.querySelector('h2').className = 'list__card__header';
            });
        }
        }
});

Как то с классами странно. Зачем надо классы list__card и list__card__rotate. Проще ведь использовать list__card, а для выбранных добавлять класс rotate и убирать его когда не нужен, чем переустанавливать все классы.

@ne_Proger · 18.07.2024, 09:48 **[ТС]**

Сообщение от voraa

Как то с классами странно. Зачем надо классы list__card и list__card__rotate. Проще ведь использовать list__card, а для выбранных добавлять класс rotate и убирать его когда не нужен, чем переустанавливать все классы.

Я художник, я так вижу )))
Для меня JS остается не очень то и понятным. Потому пока делаю так, как лично для меня понятнее. Позже я уже понял, что можно просто пару свойств в классе менять, чем переопределять весь класс. Как добью этот кусок, так и поправлю уже эту мелочь

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .

@Zubbol 1 / 1 / 0 Регистрация: 07.01.2015 Сообщений: 92
	17.07.2024, 13:11
	Добрый день. Дело в том, что document.querySelector берёт первый попавшийся элемент, если у вас имеется несколько подходящих под условие элементов на странице, в данном случае элементов с тегом li. И так как вы делаете это в цикле, то подозреваю, что на первый элемент вашего списка было навешено несколько обработчиков клика. Можете проверить мою догадку через console.log на 17 строке. Вы могли бы использовать document.querySelectorAll, который соберёт коллекцию имеющихся элементов li на странице, по которой можно будет пройти циклом и повесить обработчики на каждый элемент, но: 1. у вас получится цикл в цикле, а это плохая практика; 2. не решена проблема закрепления нескольких обработчиков за одним и тем же элементом, только теперь на каждый элемент будет вешаться по несколько обработчиков Вместо этого предлагаю совсем отказаться от каких-либо способов поиска элементов li на странице, ведь вы уже сохранили искомый элемент li в let li(советую изменить на const, а имя поменять на ту же clickCard), а значит по этому же имени можно обращаться напрямую к элементу. Если не заморачиваться на нейминге и зря использованной переменной, то достаточно стереть всё на 15 строке, а вместо clickCard ниже по коду использовать li - этим вы решите все проблемы) Ещё немного побубню: советую коллбэк, передаваемый вторым аргументом в addEventListener, сохранить куда-то отдельно в константу - вдруг вам понадобится потом снять обработчик. С анонимной функцией это будет затруднительно. Но для этого придётся немного унифицировать сам коллбэк) 0

@ne_Proger 4 / 4 / 5 Регистрация: 01.11.2021 Сообщений: 211
	17.07.2024, 14:17 [ТС]
	voraa, спасибо. Получилось. И еще спрошу. Как можно посчитать количество кликов? Кликнули один элемент, затем второй и т.д. Если кликов задангое количество, то возвращаем классы по умолчанию 0

@Zubbol 1 / 1 / 0 Регистрация: 07.01.2015 Сообщений: 92
	17.07.2024, 14:19
	Понял, вам не разобраться нужно, а готовое) 0

Изменить свойства одного элемента по клику

Решение