setInterval zmiana tła

pytanie zadane 6 grudnia 2016 w JavaScript przez Wilier Bywalec (2,570 p.)

Hej,

mógłby ktoś rzucić okiem na kod? Nie mam pojęcia czemu nie ma reakcji na skrypt, wydaje się logiczny.

http://codepen.io/Wiljer/pen/yVKrMr

1 odpowiedź

odpowiedź 6 grudnia 2016 przez kubaapk Nałogowiec (44,270 p.)
wybrane 6 grudnia 2016 przez Wilier

Najlepsza

http://codepen.io/anon/pen/womLWz

komentarz 6 grudnia 2016 przez Wilier Bywalec (2,570 p.)

dzięki!

mam jeszcze jedno pytanie, dlaczego 3 krotne wywołanie funkcji nie powoduje, że kolor zostanie zmieniony 3x? Ogólnie to chciałem to wsadzić w pętle nieskończoną żeby sobie mrugało no ale to również nie zadziała.

http://codepen.io/Wiljer/pen/yVKrMr

komentarz 6 grudnia 2016 przez xmentor Nałogowiec (49,520 p.)
edycja 6 grudnia 2016 przez xmentor

Ogólnie to chciałem to wsadzić w pętle nieskończoną żeby sobie mrugało no ale to również nie zadziała.

Pętla żadna nie potrzebna, kod, który dał kolega jest prawie okej.

if( box.css('background-color', 'red') )

przy tym kodzie zawsze będzie się wykonywało to:

box.css('background-color', 'green');

czyli to co jest wewnątrz bloku if, nie else - nawet gdy kolor box-u jest zielony.

Powinno być coś ala:

if( box.css('background-color') === 'red' )

Aczkolwiek warto zamienić red na rgb(255, 0, 0), gdyż pobrany kolor będzie zwracany w postaci RGB.

Tym samy, kod powinien wyglądać tak:

if( box.css('background-color') === 'rgb(255, 0, 0)' ) {
  box.css('background-color', 'green'); //lub rgb(0, 128, 0)
} else {
  box.css('background-color', 'red'); //lub rgb(255, 0, 0)
}

komentarz 6 grudnia 2016 przez Wilier Bywalec (2,570 p.)

Bardzo dziękuję!

Mógłbys mi jeszcze tylko powiedzieć czemu dzieje się tak, że warunek jest ciągle sprawdzany? Sądziłem, że interpreter języka przelatuje przez cały kod 1 raz i wykonuje go. Co jeśli chciałbym aby ten kod został wykonany tylko raz? Czyli jednorazowa zmiana z czerwonego na zielony lub z czerwonego na czerwony.