Snake w js - nie działające czyszczenie mapy

Question

Witam, pisze sobie snake w JS dla ćwiczeń, jestem początkujący w tym języku i dlaczego nie działa ten fragment usuwanie innych klas, a dodawanie klas all oraz map. Zaznaczyłem to w tym kodzie: http://pastebin.com/KVyQBaH6 nie wyskakuje żaden błąd. Dodam, ze klasa player w css ma tylko kolor tla nadany jako green, a klasa map ma white.

Answer 1

document.querySelectorAll('.all').className

zielony fragment kodu zwróci pseudo-tablicę (node list).

Nie działa, ponieważ tablica ta nie ma właściwości className.

Jeśli chcesz działać na poszczególnych elementach, musisz użyć pętli for lub zamienić node list na pełnoprawną tablicę i użyć metody forEach

Comments

Dzięki ;)

---

Pętle for rozumiem jako 

		var elements = document.querySelectorAll('.all');
		for(var i = 0; i<=elements; i++)
		{
			elements[i].className = "all map";
		}

czy dobrze myśle?

---

Nie działa to co napisałem teraz :/

---

Całkiem nieźle. Nie zapominaj, żeby zawsze spojrzeć do konsoli, zazwyczaj dość szybko można dzięki temu zlokalizować błąd.

i <= elements

Nie możesz sprawdzić czy liczba jest mniejsza od tablicy

elements.length, aby odczytać ilość indeksów

Pamiętaj, że indeksowanie jest od 0 : )

---

Dzięki ponownie ;)

---

btw

setTimeout("game()", 1000);

Nie wywołuj eval, eval to zło. Jeśli nie przekazujesz argumentów do funkcji game, po prostu przekaż game jako argument setTimeout

setTimeout(game, 1000);

---

Oki dzięki serdecznie, czyli mam rozumieć, że mam używać setTimeout("game()", 1000); gdy daje jakieś argumenty, tak? :)

---

Nope, 

Kiedy potrzebujesz wywołać funkcję game z argumentem, zrób to wewnątrz funkcji anonimowej. 

setTimeout(function() {
    game(arg1, arg2);
}, 1000);

---

Oki, dzięki, a jeśli mogę wiedzieć, to dlaczego eval zły? :)

---

Zdanie z eval jest podlinkowane do artykułu, tam znajdziesz odpowiedź