[Bezkrytyczny]

Mireczki z #javascript #webdev, mam problem z grą. Tworzę sobie w javascript Simon game i napotkałem problem, którego nie umiem rozwiązać. Po drugim kroku gra się wysypuje, a ja nie wiem czemu. Pomoże ktoś?
https://jsfiddle.net/abellay/mhkv0y74/
#kiciochpyta

[N.....y]

@Bezkrytyczny: bo sie wykonuje location.reload(); po kliknięciu kwadratu. Sprawdź co masz w zmiennych bo ewidentnie leci do else

[Bezkrytyczny]

@Niestabilny: Ale czemu za pierwszym razem się nie wykonuje? Dodatkowo, jak się wylosuje 2 razy pod rząd to samo, to po kliknięciu przechodzi do 3 rundy. Przechodziłem kilka razy całym algorytmem palcem i nie widzę błędu (╯︵╰,)

[Bezkrytyczny]

@Niestabilny: Poumieszczałem jeszcze alerty w przyciskach i wychodzi na to, że jak pętla idzie drugi raz to pracuje cały czas pierwszy wciśnięty przycisk, tak jakbym go trzymał cały czas. Da się to naprawić jakimś breakiem, albo czymś w tym stylu?

[N.....y]

@Bezkrytyczny: dej chwile badam sprawę

[Bezkrytyczny]

@Niestabilny: Wrzuciłem $("#red").one("click",checkred); zamiast $("#red").on("click",checkred); i działa jedną rundę dłużej, ale potem znowu wariuje tak jakby przycisk był trzymany

[N.....y]

@Bezkrytyczny: czym jest j oraz i? Wygląda tak jakby po nowej rundzie nie resetował się licznik ilości kliknięć w danej rundzie

[Bezkrytyczny]

@Niestabilny: i używam jak w pętli for w funkcji myLoop to trzęsienia kaflami. J w funkcjach check porównuje wciśnięty kafel z odpowiednim miejscem w tabeli losowań (if(tablica[j]==red/green itd))

[Bezkrytyczny]

@Niestabilny: Udało mi się wreszcie ogarnąć. Eventy się nie wyłączały po wciśnięciu, tylko tak jakby ciągle trwały. Znalazłem metodę off, wrzuciłem

$("#red").off();
$("#green").off();
$("#blue").off();
$("#yellow").off();

przed nasłuchiwaniem przycisków i wszystko działa. Dzięki za pomoc Mireczku (｡◕‿‿◕｡)

[N.....y]

@Bezkrytyczny: nie pomogłem za specjalnie, ale jak działa to najważniejsze :D

[N.....y]

@Bezkrytyczny: uprość sobie kod, bo mógłbys go zredukować o połowę ze spokojem. Robisz funkcje które w zasadzie sa takie same i wystarczyło by ino parametry zmieniac dla tych funkcji. Czym mniej kodu tym łatwiej potem to ogarnąć