[Vitin]

Ostatnie już pytanie dotyczące wisielca (css mnie nie lubi więc głównie skupcie się na kodzie xd)

1. Dlaczego clearRect nie działa?

2. Gdy spróbujecie zagrać drugi raz, funkcja rysowania (i poprzednie które ją wywołują w takim razie też) są wywoływane 2x, przez co narazie jedyną możliwością restartu jest "Press f5 to try again" ( ͡° ʖ̯ ͡°). Jak to naprawić?

http://jsfiddle.net/h6v2gdad/1/

#webdev #naukaprogramowania #javascript #html5 $jquery

[pilkarskiefakty_com]

@Vitin:

var restart = function(){ //... }

- widzę deklarację funkcji. A gdzie jest jej wywołanie?

[Vitin]

@pilkarskiefakty_com: 17 linia

[pilkarskiefakty_com]

@Vitin: Dobra, nieważne - widzę wszystko. Jakiego spodziewasz się efektu? Dla mnie to działa. Skonczę grę, wysyłam z formularza nowe słowo, i mam wszystko na nowo.

[Vitin]

@pilkarskiefakty_com Okej, za pierwszym razem nie traf kilka razy, za drugim razem wpisz inne słowo i znów nie traf - rozpoczynasz od stanu poprzedniej gry, do tego funkcja drawHangMan() jest wywoływana 2x za każdym razem, czyli za każde nietrafienie dorysowywane są 2 elementy...

[Vitin]

@pilkarskiefakty_com : Idąc dalej, grając drugi raz i typując poprawną literę dostajesz karny element, tak jakby funkcja pierw sprawdzała z aktualnym słowem, a następnie ze słowem z poprzedniej gry...

[dieow]

@Vitin: Jeżeli coś przy kliknięciu dzieje się dwa lub więcej razy to możliwe, że za każdym razem przypisujesz na nowo funkcje do kliknięcia.

I tak właśnie robisz :P

keys.on('click'

za każdym razem dodajesz gdy zaczynasz grę.

[pilkarskiefakty_com]

@Vitin: @dieow: Dokładnie tak, każde wysłanie formularza powoduje kolejny raz zapięcie eventu.

[Vitin]

@dieow: Okej, wiem już gdzie niepotrzebnie to zapętlałem, teraz pytanie w jaki sposób przesłać nowy argument do iClickYou(), jeżeli nie mogę wywołać jej ponownie ;/

edit: Bardzo bym chciał uniknąć przechowywania słowa w globalnej zmiennej, taki mały challenge ;d

[dieow]

@Vitin: Możesz zmienną

word

trzymać globalnie.

I odkryłem dlaczego

clearRect

nie działa. Otóz w funkcji rysującej wisielca

ctx.beginPath();

robisz tylko w przypadku głowy. A powinieneś za każdym razem. (Czyli dać ją przed switchem)

https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D.beginPath

starts a new path by emptying the list of sub-paths.

[Vitin]

@dieow: Właśnie nie chce jej trzymać globalnie ( ͡° ʖ̯ ͡°)

[dieow]

@Vitin: Możesz przy restarcie usunąć 'event handlera' z liter. I za każdym razem na nowo przypisywać :P

[Vitin]

@dieow: Haha, no tak, to też jakieś rozwiązanie :D A nie da się wysłać parametru do funkcji bez jej wywoływania? :(

[dieow]

@Vitin: W sumie to one nie są globalne bo wywołujesz wszystko wewnątrz

$(window).load()

i tam pozostają.

[Vitin]

@dieow: ale można łatwo zwrócić ich wartość w konsoli :p

[dieow]

@Vitin: Próbowałeś? Bo chyba właśnie nie bardzo :P

[Vitin]

@dieow: nie wiem jak na jsfiddle (juz na tel jestem) ale na localhoscie bezproblemowo wpisując w konsole nazwę zmiennej dostaje jej wartość ;)