Fehler in for Schleife

[db1]

Hallo,
Ich versuche in eine For Schleife Elemente auszublenden was auch funktioniert,
jedoch gibt es den Fehler das:
null is not an object (evaluating 'x.style') obwohl das x vorhanden ist.


for (let i = 0; i < len; i++) {

var weg = "weg_" + i;
var weg = Math.round(Math.random() * (max - min)) + min;


setTimeout(() => {

let x = document.getElementById("img_teil" + i)


x.style.transition = "2s"
x.style.timing = "ease-in-out";
x.style.opacity = "0"
visibility_check = "0";


}, weg);

wenn ich:
x.style.transition = "2s" raus nehme ist der Fehler weg
das opacity = 0 funktioniert auch aber ohne den ease-in-out Effekt.
warum hat x.style.transition = "2s"; null is not an object (evaluating 'x.style') ??


wo hab ich da den Fehler ?

lg
Guenther


//end for
}

 

[M.L.]

Das ist wohl JavaScript-Code (und (lt. KI) auch verbesserungswürdig)

 

[grrr]

var weg = "weg_" + i;
var weg = Math.round(Math.random() * (max - min)) + min;

Doppeldeklaration von weg und du willst doch nur einen timestamp haben, aber keinen string

 

[KonradN]

Der Code ist so schwer lesbar. Das wäre mit sinnvoller Formatierung vermutlich einfacher. Dazu bitte Code auch wirklich als Code posten. Dazu dient der Knopf </> links oberhalb des Eingabebereiches. Dann kommt ein Popup, bei dem Du die Sprache (JavaScript) auswählen und den Code dann posten kannst.

Dann wäre es evtl. sinnvoll, konsequent ; nach Anweisungen zu setzen, denn ich vermute, dass Du keinerlei Tools zur Absicherung einsetzt. Da ist es durchaus sinnvoller, diese altmodisch beizubehalten. Aber das ist nur eine reine Empfehlung.

du willst doch nur einen timestamp haben, aber keinen string

Er will eine Zahl haben - da kommt kein timestamp sondern der timeout in ms. Und das hat er durch die zweite Deklaration, die die erste verdeckt.

Aber zu dem eigentlichen Problem
Ich vermute, dass es Zufall war, dass die Herausnahme dieser einen Zeile dazu führte, dass es plötzlich ging.
getElementbyId hat null geliefert und jede der Zugriffe wäre fehlgeschlagen, so da ein Element nicht gefunden würde.

Generell hast Du ja einen Timeout. Probleme können also sein, dass Deine Grenzen entweder von Anfang an falsch sind unter gewissen umständen oder Du hast das Problem, dass in der Zeit des Timeouts der DOM Tree noch geändert wird.

Aber über die exakte Ursache zu spekulieren bringt hier auch nichts. Die wichtige Lösung ist einfach:

if (!x) return;
Wenn getElementById das Element nicht findet, dann solltest Du diesen Fall berücksichtigen. In diesem Fall ist dann halt nichts zu deaktivieren.

 

[grrr]

Er will eine Zahl haben - da kommt kein timestamp

Für mich sind delays und periods in Form von "Zahlen" auch timestamps.