Empresas
Empleos
  • Sobre nosotros
  • Soluciones
    • Publicación de vacantes
      Publica tu vacante y recibe candidatos calificados en 48h.
    • Evaluación de candidatos
      500+ pruebas técnicas y psicológicas, más anti-fraude.
    • Headhunting
      Búsqueda ejecutiva a la medida de principio a fin.
    • Nómina + EOR
      Dispersión de nómina y EOR en más de 15 países de LATAM.
  • Precios
  • Empleos

0

171
Vistas
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 Respuestas
Responde la pregunta

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 Denunciar

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 Denunciar
Responde la pregunta
Encuentra empleos remotos

¡Descubre la nueva forma de encontrar empleo!

Top de empleos
Top categorías de empleo
Empresas
Publicar vacante Precios Comercial
Legal
Términos y condiciones Política de privacidad
© 2026 PeakU Inc. All Rights Reserved.
Andres GPT
Recomiéndame algunas ofertas
Necesito ayuda