Не работает валидация во второй раз

Мой код должен переключать цвет на красный и обратно после щелчка, но он работает только один раз!

<div class = "time-block">00:00 - 00:59</div>

Класс "red-my-day" для переключения цвета, добавляя или удаляя его.

function addTimeBlocksHandler() {
    let arrTimeBlocks = document.getElementsByClassName("time-block");

    for (let i = 0; i < arrTimeBlocks.length; i++) {
        if (document.getElementsByClassName("time-block")[i].classList.contains("red-my-day")) {
                arrTimeBlocks[i].addEventListener("click", () => {
                document.getElementsByClassName("time-block")[i].classList.remove("red-my-day");
            });
        } else if (!document.getElementsByClassName("time-block")[i].classList.contains("red-my-day")) {
            arrTimeBlocks[i].addEventListener("click", () => {
                document.getElementsByClassName("time-block")[i].classList.add("red-my-day");
            });
        }
    }


}

addTimeBlocksHandler();

Почему это работает один раз, переключитесь на красный цвет и все?

Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
0
0
16
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Причина, по которой ваш код не работает, заключается в том, что при выполнении javascript он получает класс class = "time-block" для элемента div. Поскольку у этого элемента нет класса red-my-day, он пропустит добавление прослушивателя событий click внутри первого блока if, перейдет к блоку else и назначит прослушиватель для этого события. Поэтому для красного цвета он сработал только один раз.

Чтобы упростить свой код, вы можете просто использовать toggle(), чтобы он переключал класс при каждом щелчке по элементу.

function addTimeBlocksHandler() {
    let arrTimeBlocks = document.getElementsByClassName("time-block");
    for (let i = 0; i < arrTimeBlocks.length; i++) {
      arrTimeBlocks[i].addEventListener("click", (elem) => {
        elem.target.classList.toggle('red-my-day');
      });
    }
}

addTimeBlocksHandler();
.red-my-day{
  color: red;
}
<div class = "time-block">00:00 - 00:59</div>
<div class = "time-block">00:10 - 01:59</div>
<div class = "time-block">00:20 - 02:59</div>

Другие вопросы по теме