Antwort an „Gunnar Bittersmann“ verfassen

@@Linuchs

var bilder = document.querySelectorAll( "img" );

Nö, das ist Quatsch. document.images existiert bereits.


for ( let i=0; i<bilder.length; i++ ) {

Die Tage der guten alten FOR-Schleife sind aber nun auch schon eine ganze Weile gezählt. Es gibt sicher noch Anwendungsfälle dafür, aber alle Elemente einer Sammlung zu durchlaufen, zählt nicht dazu; dafür gibt es forof.

Dann musst du nicht mit Indizes rummachen und der Code wird lesbarer:

for (const image of document.images) {
	console.log(image);
}

Für etliche Objekttypen gibt es auch die forEach-Methode; nicht aber für HTMLCollection, was document.images ist. Kann man aber fix in ein Array umwandeln:

[...document.images].forEach(image => {
	console.log(image);
});

  thumbverzeichnis  += "<div class='li bo'>\n";

Das ist unlesbarer Code. Wofür steht 'li'? Wofür steht 'bo'?

Und wozu sollen die div-Container überhaupt gut sein? Nur die img-Elemente sollten doch reichen?

Und die Zeilenumbrüche interessieren den HTML-Parser nicht die Bohne, die \n können weg.

🖖 Live long and prosper

--
“Space: the final frontier. Above us. Around us. Within us.
We have always looked to the stars to discover who we are.”

— Michael Burnham in Star Trek: Discovery S2:E1
freiwillig, öffentlich sichtbar
freiwillig, öffentlich sichtbar
freiwillig, öffentlich sichtbar

Ihre Identität in einem Cookie zu speichern erlaubt es Ihnen, Ihre Beiträge zu editieren. Außerdem müssen Sie dann bei neuen Beiträgen nicht mehr die Felder Name, E-Mail und Homepage ausfüllen.

abbrechen