Javascript если / еще вопрос

Question

Я хочу открыть div и закрыть его с помощью этой функции, но она не работает. Где неправильная часть, я не могу это исправить, кто-нибудь может мне помочь?

function Element(id)
{    
    if(document.getElementById(id).style.display = 'block')
    {
        document.getElementById(id).style.display = 'block';
    }
    else
    {
        document.getElementById(id).style.display = 'none';
    }
}

Answer 1

Я бы посоветовал вам переписать его с помощью троичного оператора, это гораздо более читабельно и легко поддерживается:

Answer 2

Ваш оператор if назначает свойство вместо его сравнения.
Измените = на ===.

Answer 3

Спасибо! Этот работал. но я изменил код внутри if. Потому что там написано, что вы нашли блок, которого не должно быть.

function Element(id) { 
    if( document.getElementById(id).style.display == 'none') {
      document.getElementById(id).style.display = 'block';
    } else { 
      document.getElementById(id).style.display = 'none'; 
    } 
}

Answer 4

function Element(id) { 
        if( document.getElementById(id).style.display == 'block') {
             document.getElementById(id).style.display = 'block';
       } else { document.getElementById(id).style.display = 'none'; }
} 

Answer 5

Вам не нужен else или троичный оператор, так как ваше первое условие не действует. Так что этого будет достаточно ...

if(document.getElementById(id).style.display != 'block')
{
    document.getElementById(id).style.display = 'none';
};