Gra w wisielca, czyli najnowszy materiał Pana Mirosława.

Question

Kłaniam się z góry jak rzadko który, witam i o zdrowie pytam.

W najnowszym kursie Pana Mirosława dotyczącego programowania w JavaScript (https://www.youtube.com/watch?v=9FVtiJHFCSU), postanowiłem dodać parę linijek własnego kodu.

Chciałem aby w przypadku wygranej, skrypt odtworzył fanfary, ale nie wiem dlaczego dźwięk odtwarzany jest po kliknięciu obojętniej jakiej literki, błędnej, czy poprawnej.

//wygrana
if (haslo == haslo1)
document.getElementById("alfabet").innerHTML  = "Tak jest! Podano prawidłowe hasło: "+haslo+'<br /><br /><span class="reset" onclick="location.reload()">JESZCZE RAZ?</span>';
wygrana.play();
	

Dlaczego w ogóle dźwięk pada za każdym kliknięciem, skoro warunek jest prosty. Tylko w przypadku gdy  haslo jest równe haslo1. Przecież warunek się spełnia dopiero gdy odgadniemy wszystkie literki!

Próbowałem także dodać dźwięk w przypadku przegranej, ale skrypt po kliknięciu tylko jednej literki (obojętnie jakiej), wywalał komuikat o przegramej grze...

 

//przegrana
if (ile_skuch>=9)
document.getElementById("alfabet").innerHTML  = "Przegrana! Prawidłowe hasło: "+haslo+'<br /><br /><span class="reset" onclick="location.reload()">JESZCZE RAZ?</span>';
przegrana.play();

Ktoś wie o w czym tkwi problem?

Answer 1

Hahaha. Zanim mnie wyśmiejecie ... hahahaha. Boże, ale ja jestem głupi :D
Wpadłem na to :D Wpadłem na to sam :D hahaha.

Wybaczcie, ale moja wiedza na temat JS jest na takim samym poziomie jak Wasza na temat kultury Chińskiej :D

Zamknąłem te dwa zdarzenia w nawias i już działa :D
Aż mi wstyd :D

Zamiast takiego kodu:

	//wygrana
	if (haslo == haslo1)
	document.getElementById("alfabet").innerHTML  = "Tak jest! Podano prawidłowe hasło: "+haslo+'<br /><br /><span class="reset" onclick="location.reload()">JESZCZE RAZ?</span>';
	wygrana.play();		

wystarczyły dwa głupie nawiasy :D

	//wygrana
	if (haslo == haslo1){
	document.getElementById("alfabet").innerHTML  = "Tak jest! Podano prawidłowe hasło: "+haslo+'<br /><br /><span class="reset" onclick="location.reload()">JESZCZE RAZ?</span>';
	wygrana.play();		
	}

i działa jak powinno. Sorki za problem :-D

Comments

Widać masz bardzo rozległą wiedzę na temat js xD, jeśli chodzi o kulturę chińską

---

To teraz będziesz to dobrze pamiętać ;)

---

Ale czy rozumiesz chociaż dlaczego wcześniej nie było tam nawiasów klamrowych, a po twojej modyfikacji powinny być?

Mam nadzieję, że tak.

---

Zgaduje, że więcej niż jedno polecenie powinno zapisywać się w nawiasach...
Jeśli źle zgaduje, to mnie popraw. Każda informacja jest dla mnie cenna.

---

Oczywiście, zgadza się