Empresas
Empregos
  • Sobre nós
  • Soluções
    • Publicação de vagas
      Publique sua vaga e receba candidatos qualificados em 48h.
    • Avaliações de candidatos
      Mais de 500 testes técnicos e psicológicos, mais anti-fraude.
    • Headhunting
      Busca executiva personalizada do início ao fim.
    • Folha de Pagamento + EOR
      Dispersão de folha e EOR em mais de 15 países da LATAM.
  • Preços
  • Empregos

0

166
Visualizações
check for duplicate before appendChild
const pickNewUl = document.getElementById("ullist1");     
var createLi = document.createElement("li");
                            createLi.id = listItems[i].id;
                            createLi.classList.add(pickNewUlsl);
                            createLi.innerHTML = listItems[i].textContent; 
                            createLi.innerHTML += "<a onclick='remove(this)' class='removebtn'>X</a>";
pickNewUl.appendChild(createLi);

What I need to check in above code is: I want to check if there are any same id LI exists or not, if not then only it should append, otherwise it will not append.

pickNewUl is a UL list

about 4 years ago · Santiago Trujillo
2 Respostas
Responde à pergunta

0

You can find for any element inside element with .querySelectorAll as below.

if (pickNewUl.querySelectorAll('#' + listItems[i].id).length === 0) {
    // Add element    
}

Reference : https://developer.mozilla.org/en-US/docs/Web/API/Document/querySelectorAll

Your complete code should be like below.

const pickNewUl = document.getElementById("ullist1"); 
if (pickNewUl.querySelectorAll('#' + listItems[i].id).length === 0) {   
    var createLi = document.createElement("li");
    createLi.id = listItems[i].id;
    createLi.classList.add(pickNewUlsl);
    createLi.innerHTML = listItems[i].textContent; 
    createLi.innerHTML += "<a onclick='remove(this)' class='removebtn'>X</a>";
    pickNewUl.appendChild(createLi);
}
about 4 years ago · Santiago Trujillo Relatório

0

You can just wrap you code inside an if:

// the same as before but using pickNewUl instead of document
if (!pickNewUl.getElementById(listItems[i].id)) {
  const createLi = document.createElement("li");
  ...
  pickNewUl.appendChild(createLi);
}

Btw, I suggest to use a different approach excluding duplicated id:

// The string into querySelector method is a template string.
if (!pickNewUl.querySelector(`[data-item-id="${listItems[i].id}"]`)) {
  const createLi = document.createElement("li");
  createLi.dataset.itemId=listItems[i].id;
  ...
  pickNewUl.appendChild(createLi);
}
about 4 years ago · Santiago Trujillo Relatório
Responde à pergunta
Encontrar trabalhos remotos

Descubra a nova forma de encontrar um emprego!

melhores empregos
Principais categorias de trabalho
Empresas
Postar vaga Preços Comercial
Jurídico
Termos e Condições Política de privacidade
© 2026 PeakU Inc. All Rights Reserved.
Andres GPT
Recomende algumas ofertas para mim
Preciso de ajuda