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.