Bildauswahl Reset

[Felix Riesterer]

Lieber Rolf,

seit Jahren bin ich der Überzeugung, dass eine for-Schleife besser durch eine forEach-Funktion ersetzt werden sollte, weil der neue Scope in der Callback-Funktion die Möglichkeit bietet, mit Closures zu arbeiten. Das mag für Anfänger zwar zu viel auf einmal sein, weil Closures kein Anfänger-Thema sind, aber anstelle einer Schleife forEach zu benutzen, sollte ohne großen Aufwand zu verstehen sein.

   let galleries = document.querySelectorAll(".gallery");
   for (let gallery of galleries) {
      let banner = galerie.querySelector(".banner");
      let thumb1 = gallery.querySelector("button img");

      // Banner aus Thumb-Informationen befüllen
      banner.src = thumb.dataset.pic;
      banner.alt = thumb.alt;
   }
});

Warum nicht so?

document.querySelectorAll(".gallery").forEach(gallery => {
  const
    banner = gallery.querySelector(".banner"),
    thumb = gallery.querySelector("button img");

  // Banner aus Thumb-Informationen befüllen
  if (banner && thumb) {
    banner.src = thumb.dataset.pic;
    banner.alt = thumb.alt;
  }
});

Liebe Grüße

Felix Riesterer