jQuery Event wywołuje się po załadowaniu strony

Question

WItam

Odwzorowuję grę za czasów PRLu - Mini Master Mind. Używam jQuery i gdy dodaję event do przycisku, to po załadowaniu strony event się sam wykonuje.

elmnt.on("click", add_color(elmnt, colors[i])); // Wycinek z kodu więc proszę o nie zwracanie uwagi na zmienne i własne funkcje.

Z góry dziękuję za odpowiedź.

Answer 1

Bo do event listenera trzeba przekazać funkcję, Ty przekazujesz wynik wywołania funkcji. Jeśli po nazwie funkcji następuje (), to zawsze się wywoła od razu.

Najwygodniej byłoby tutaj użyć funkcji strzałkowej:

elmnt.on("click", () => {
	add_color(elmnt, colors[i]);
});

 

Comments

Ja to się nie znam dobrze na tym a nie zadziałałoby elmnt.on("click", function(){add_color(elmnt, colors[i])});   ??? ewentualnie  elmnt.on("click",add_colors);?

---

To pierwsze jest praktycznie tożsame z funkcją strzałkową, więc można stosować albo to, albo to. Aczkolwiek dla mnie funkcje strzałkowe są wygodniejsze w takim zastosowaniu.

To drugie nie zadziała, bo nie przekazujemy poprawnych parametrów.

---

AI mi marudzi że strzałkowe lepsze bo wywołują z właściwego zasięgu zmiennych. A ja wolę bindować. To drugie to tak strzeliłem bo tak się setTimeout wywołuje prawda? JS to chyba najpotężniejszy język obecnie.

---

AI mi marudzi że strzałkowe lepsze bo wywołują z właściwego zasięgu zmiennych. A ja wolę bindować.

Akurat w tym kodzie to nie ma znaczenia, bo kontekst this nie jest wgl używany.

 To drugie to tak strzeliłem bo tak się setTimeout wywołuje prawda?

W setTimeout() się przekazuje funkcję – a więc działają wszystkie te same sposoby zapisu co tu.