Co robię źle?

Question

Chcę sprawdzić czy przy danym id jest dana klasa i jeżeli tak to chciałbym tą klasę usunąć. Co robię źle?

for(let i=1;i<=4;i++){
	if(document.querySelector("#remake-" + i).classList.contains("declined") == true){
		document.getElementById("remake-" + i).classList.remove("declined");
	}
}

 

Comments

Ale na czym dokładnie polega problem?

---

Pokaż jeszcze kod HTML. Może źle odnosisz się do elementów? Czy ten skrypt odpala się, gdy DOM jest już gotowy? Czy w konsoli są jakieś błędy?

Pytanie poboczne - dlaczego pobierasz te elementy dwukrotnie, na dodatek dwiema różnymi metodami?

Poza tym, indeksowanie lepiej zaczynać od 0.

---

<tr class="row">
		<td id="fly-1">Moon</td>
		<td id="nr-1"></td>
		<td id="gate-1"></td>
		<td id="remark-1"></td>
		<td id="time-1"></td>
</tr>
<tr class="row">
		<td id="fly-2">Mars</td>
		<td id="nr-2"></td>
		<td id="gate-2"></td>
		<td id="remark-2"></td>
		<td id="time-2"></td>
</tr>

Tak to wygląda w html. 

A odpowiadając na pytanie poboczne to sam nie wiem. Uczę się dopiero jsa i próbuję używać jak największej liczby różnych metod mimo, że czasami jest to to niepotrzebne.

---

Nie odpowiedziałeś na pytania:

Czy ten skrypt odpala się, gdy DOM jest już gotowy? Czy w konsoli są jakieś błędy?

Widzę natomiast, że najprawdopodobniej masz literówki:

<td id="remark-1"></td>

oraz

<td id="remark-2"></td>

, a w JavaScript próbujesz pobrać te elementy szukając po id "remake-" + i. Tu zapewne jest problem. :)

---

Faktycznie.. Dzięki wielkie, nie zauważyłem