Skip to main content
adicionou 1157 caracteres ao conteúdo
Fonte Link
Syzoth
  • 100,5mil
  • 23
  • 169
  • 361

Primeiro teu HTML esta errado ou teu JS esta errado, aqui contêm um ID id="minhaDiv":

<body class="curso">
<div id="minhaDiv" class="col-sm-12">
<h1>TESTE</h1>
</div>
</body>

Mas no teu JS esta usando div.minhaDiv que não é pra ID e sim para CLASSES.

Vale lembrar que IDs não podem se repetir. O querySelectorAll é para pegar vários elementos e não um só, se é para pegar o ID use o querySelector (sem All) ou getElementById, deve ficar assim:

var corpo = document.querySelectorAll("body.cursos").length;

var some = document.getElementById("minhaDiv");

if (corpo >= 1) {
    some.style.display = 'block';
} else {
    some.style.display = 'none';
}
<body class="curso">
<div id="minhaDiv" class="col-sm-12">
<h1>TESTE</h1>
</div>
</body>

Veja que ao carregar o elemento já é ocultado.

O document.getElementById("minhaDiv") pega UM unico elemento que tenha a ID, você pode até usar

var some = document.querySelector("#minhaDiv");

Mas querySelector seria mais indicado se precisar de um seletor mais complexo e não é o caso.

Nota: Não precisa checar se é "zero" no else, o primeiro IF já contem >=1, a exceção então só pode ser zero mesmo, afinal o .length só retorna 0 ou maior.


Resposta ANTES da edição da pergunta

O document.querySelectorAll("div.minhaDiv") pega vários elementos e não um só, tem que aplicar a um for acaso queira aplicar o display para todos elementos com a mesma classe, assim:

Não precisa checar se é "zero" no else, o primeiro IF já contem >=1, a exceção então só pode ser zero mesmo, afinal o .length só retorna 0 ou maior.

O document.querySelectorAll("div.minhaDiv") pega vários elementos e não um só, tem que aplicar a um for acaso queira aplicar o display para todos elementos com a mesma classe, assim:

Não precisa checar se é "zero" no else, o primeiro IF já contem >=1, a exceção então só pode ser zero mesmo, afinal o .length só retorna 0 ou maior.

Primeiro teu HTML esta errado ou teu JS esta errado, aqui contêm um ID id="minhaDiv":

<body class="curso">
<div id="minhaDiv" class="col-sm-12">
<h1>TESTE</h1>
</div>
</body>

Mas no teu JS esta usando div.minhaDiv que não é pra ID e sim para CLASSES.

Vale lembrar que IDs não podem se repetir. O querySelectorAll é para pegar vários elementos e não um só, se é para pegar o ID use o querySelector (sem All) ou getElementById, deve ficar assim:

var corpo = document.querySelectorAll("body.cursos").length;

var some = document.getElementById("minhaDiv");

if (corpo >= 1) {
    some.style.display = 'block';
} else {
    some.style.display = 'none';
}
<body class="curso">
<div id="minhaDiv" class="col-sm-12">
<h1>TESTE</h1>
</div>
</body>

Veja que ao carregar o elemento já é ocultado.

O document.getElementById("minhaDiv") pega UM unico elemento que tenha a ID, você pode até usar

var some = document.querySelector("#minhaDiv");

Mas querySelector seria mais indicado se precisar de um seletor mais complexo e não é o caso.

Nota: Não precisa checar se é "zero" no else, o primeiro IF já contem >=1, a exceção então só pode ser zero mesmo, afinal o .length só retorna 0 ou maior.


Resposta ANTES da edição da pergunta

O document.querySelectorAll("div.minhaDiv") pega vários elementos e não um só, tem que aplicar a um for acaso queira aplicar o display para todos elementos com a mesma classe, assim:

Fonte Link
Syzoth
  • 100,5mil
  • 23
  • 169
  • 361

O document.querySelectorAll("div.minhaDiv") pega vários elementos e não um só, tem que aplicar a um for acaso queira aplicar o display para todos elementos com a mesma classe, assim:

var corpo = document.querySelectorAll("body.category-cursos").length;
var some = document.querySelectorAll("div.minhaDiv");

if (corpo >= 1) {
    for (var i = 0, j = some.length; i < j; i++) {
        some[i].style.display = 'block';
    }
} else {
    for (var i = 0, j = some.length; i < j; i++) {
        some[i].style.display = 'none';
    }
}

Não precisa checar se é "zero" no else, o primeiro IF já contem >=1, a exceção então só pode ser zero mesmo, afinal o .length só retorna 0 ou maior.


Usando for, você também poderia usar para iterar:

[].forEach.call(some, function (elemento) {
     elemento.style.display = 'block';
});

E para navegador modernos (com suporte):

some.forEach(function (elemento) {
     elemento.style.display = 'block';
});

Recomendo que leia: