0

Основная задумка в том, чтобы при нажатии на какую-то кнопку кнопка нажатая до этого 'потухала'. Но почему-то в итоге я просто кликаю по кнопкам и они поочередно активируются, при этом другие также активны. Я не могу понять, что в коде не так.

JS:

function colorChanger(obj){ 
            let elem = document.getElementsByClassName('btn')
            for (let i; i < elem.length; i++) {
                if (elem[i].classList.contains('active')) {
                    elem[i].remove('active')
                }
            }
            obj.classList.add('active')
        }

HTML:

<body>
    <button onclick="colorChanger(this)" class="btn">Button 1</button>
    <button onclick="colorChanger(this)" class="btn">Button 2</button>
    <button onclick="colorChanger(this)" class="btn">Button 3</button>
    <script src="script.js"></script>
</body>
1
  • В obj что лежит? Html покажите. А вообще такие штуки надо учится делать самому - навешайте console.log или через debugger в chrome тормозите код, смотрите что у вас в переменных на каждом шаге. Где-то что-то не находит. 9 мар 2021 в 14:21

2 ответа 2

1

Ошибка была в цикле, я не задал начальное значение i...

for (let i = 0; i < elem.length; i++) {...

А так, все работает как и задумано

0

Пропробуйте так

function colorChanger(obj){ 
            let elem = [...document.getElementsByClassName('btn')]
            for (let i; i < elem.length; i++) {
                if (elem[i].classList.contains('active')) {
                    elem[i].remove('active')
                }
            }
            obj.classList.add('active')
        }

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.