Skip to main content
2 de 3
se añadieron 507 caracteres en el cuerpo
Pablo Lozano
  • 49.9k
  • 8
  • 55
  • 92

El problema es el siguiente:

let childs = document.getElementById("allstars").childNodes;

Aparte de que el plural de child es children, tenemos que tiene no 5 elementos sino 10.

Usando

const children = document.querySelectorAll('.allstars .star');

obtendrás realmente los elementos que necesitas.

Además, podrías simplificar el código haciendo algo como:

const rating = 0.5;


const children = document.querySelectorAll('.star');
children.forEach(el => el.src = '../assets/empty_star.svg');
const fullStars = Math.floor(rating);
const hasHalfStar = fullStars !== rating;

for (let i=0; i < fullStars; i++) {
    children[i].src = "../assets/full_star.svg";
}
if (hasHalfStar) {
    children[fullStars].src = "../assets/half_star.svg";
}
Pablo Lozano
  • 49.9k
  • 8
  • 55
  • 92