Почему не работает проверка на наличие класса jquery?

Question

[privat13]

Есть верстка (все упрощаю для облегчения понимания)

<div class="cityWrapper" onclick="dropdown('city__dropdown', 'city__dropdown_active');"></div>
<div class="city__dropdown"></div>

Класс city__dropdown_active добавляет видимость элементу city__dropdown

Есть скрипт

function dropdown(targetElem, activity) {
            let elem = $('.'+targetElem);
            if (elem.hasClass(activity) == true) {
                elem.removeClass(activity);
            } else {
                elem.addClass(activity);
            }         
        }

Логика... по нажатию на элемент cityWrapper, вызывается функция dropdown. По ее выполнению должен появиться элемент city__dropdown.

Внутри функции идет проверка.
Если city__dropdown уже имеет класс city__dropdown_active (то ест он уже видимый), то класс city__dropdown_active удаляется (элемент исчезает).
В противном случае элемент появляется.

Но элемент только появляется и не исчезает. В чем может быть причина?

Comments

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

А Вы дебажить код пробовали? Что приходит функции на вход? Что вычисляется в elem?

Ну и по самому коду:
1. Не нужно в if делать == true
2. Стоит использовать строгие сравнения (===)
3. Посмотрите в сторону toggleClass
4. Не стоит навешивать события в HTML

[privat13]

Дмитрий Беляев, Принято. Спасибо

Answer 1

[Надим]

<div class="cityWrapper" onclick="$('.city__dropdown').toggle();"></div>

jQuery метод .toggle() позволяет отобразить или скрыть выбранные элементы. Если элемент изначально отображается, то он будет скрыт, если элемент скрыт, то он будет отображен.