Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [JavaScript] setTimeOut
Forum PHP.pl > Forum > Przedszkole
Lirdoner
17.04.2013, 17:48:02
Witam, mam problem z funkcją setTimeOut. Przy takim kodzie
[JAVASCRIPT] pobierz, plaintext 
  1. function change(slider) {
  2.     alert(slider);
  3.     $('.sliderOption').removeClass('active');
  4.     $('#slide' + slider).addClass('active');
  5.     slider = parseFloat(slider) + 1;
  6.     if(slider == 6) { slider = 1; }
  7.     setTimeout(change(slider), 3000);
  8. }
  9. change(2);
[JAVASCRIPT] pobierz, plaintext

Dostaję alerty co chwilę mimo że powinny pojawiać się co 3 sekundy. O co może chodzić?
nospor
17.04.2013, 17:49:32
setTimeout(change(slider), 3000);
Zobacz w manualu js jak sie wywołuje setTimeout bo to co ty teraz robisz to zwykłe odpalenie funkcj z paramettrem
change(slider)
więc nic dziwnego ze odpala ci się od razu...
PrinceOfPersia
17.04.2013, 18:42:33
musisz podać funkcję jako parametr, bez jej wywołania, o tak:
Kod
setTimeout(change, 3000); // bez nawiasów


tylko w ten sposób Ci się "gubi" informacja o sliderze. Więc musisz użyć sztuczki, np. stworzyć drugą funkcję, która będzie wywoływała tę twoją:
Cytat
setTimeout(function() { change(slider); }, 3000);

function() { .......KOD.........} tworzy funkcję anonimową

jest jeszcze metoda bind i mógłbyś tak napisać:
Kod
setTimeout(change.bind(this, slider), 3000);

ale to nie będzie Ci działać we wszystkich przeglądarkach.
timon27
17.04.2013, 22:48:29
Cytat(PrinceOfPersia @ 17.04.2013, 19:42:33 ) *
Więc musisz użyć sztuczki


Ja po prostu piszę:
[HTML] pobierz, plaintext 
  1. setTimeout("change(slider)", 3000);
[HTML] pobierz, plaintext

i działa.
Trzeba oczywiscie pamiętać że slider bezie pobrany również za 3s a nie natychmiast,
lecz w podanym przykładzie nie ma z tym problemu.
PrinceOfPersia
18.04.2013, 09:10:56
Cytat
Ja po prostu piszę:
[HTML] pobierz, plaintext
setTimeout("change(slider)", 3000);
[HTML] pobierz, plaintext
i działa.

ależ to prowizorka, czasem działa, czasem nie. (I w tym wypadku nie działa z tego co sprawdzałem). Chodzi o to, że jak podajesz kod w stringu, to żadne zmienne lokalne (zadeklarowane na poziomie funkcji) nie działają. Ani te, które są zadeklarowane po var, ani this, ani argumenty funkcji.
timon27
22.04.2013, 22:14:23
Cytat(PrinceOfPersia @ 18.04.2013, 10:10:56 ) *
ależ to prowizorka, czasem działa, czasem nie.

Zawsze działa, tylko tak jak mówiłem trzeba zwracać uwagę na zmiennne.

Cytat(PrinceOfPersia @ 18.04.2013, 10:10:56 ) *
żadne zmienne lokalne (zadeklarowane na poziomie funkcji) nie działają

Więc użyj zmiennych globalnych
b4x
22.04.2013, 23:07:45
Pozwoliłem sobie tongue.gif

Chyba chodzi Tobie o coś takiego:

[HTML] pobierz, plaintext 
  1. <html>
  2. 	<head>
  3. 		<style type="text/css">
  4. 		.active{display:block !important;}
  5. 		</style>
  6. 		<script src="//ajax.googleapis.com/ajax/libs/jquery/1.9.1/jquery.min.js"></script>
  7. 		<script type="text/javascript">
  8. 		var Slider =
  9. 		{
  10. 			current: 1,
  11. 			change: function()
  12. 			{		
  13. 				 $('.sliderOption').removeClass('active');
  14. 				 $('#slide' + Slider.current).addClass('active');
  15.  
  16. 				Slider.current += 1;
  17.  
  18. 				if(Slider.current >= 6)
  19. 					Slider.current = 1;
  20. 			}
  21. 		}
  22.  
  23. 		$(function(){
  24. 			Slider.change();
  25. 			setInterval(Slider.change, 3000);
  26. 		});
  27. 		</script>
  28. 	</head>
  29. 	<body>
  30. 		<div id="slide1" class="sliderOption" style="display:none;">
  31. 			1
  32. 		</div>
  33. 		<div id="slide2" class="sliderOption" style="display:none;">
  34. 			2
  35. 		</div>
  36. 		<div id="slide3" class="sliderOption" style="display:none;">
  37. 			3
  38. 		</div>
  39. 		<div id="slide4" class="sliderOption" style="display:none;">
  40. 			4
  41. 		</div>
  42. 		<div id="slide5" class="sliderOption" style="display:none;">
  43. 			5
  44. 		</div>
  45. 		<div id="slide6" class="sliderOption" style="display:none;">
  46. 			6
  47. 		</div>
  48. 	</body>
  49. </html>
[HTML] pobierz, plaintext
PrinceOfPersia
23.04.2013, 09:36:22
@bx4, testowałeś to? Bo tak na oko, to też gubi się kontekst this:
[JAVASCRIPT] pobierz, plaintext 
  1. setInterval(Slider.change, 3000);
[JAVASCRIPT] pobierz, plaintext


Cytat
Więc użyj zmiennych globalnych

Nie zawsze się da.

Cytat
Zawsze działa, tylko tak jak mówiłem trzeba zwracać uwagę na zmiennne.

Wolę robić normalnie i nie musieć na nic zwracać uwagę. Poza tym jak robisz normalnie, to masz kolorowanie składni i inne bajery. I masz wszystkie zmienne w JS, możesz też łatwo dać do setTimeouta jakąś większą funkcję. Łatwiej wykrywać błędy. Generalnie same zalety nad hardkodowaniem stringa.
Poza tym hardkodowanie stringa to rozwiązanie z eval, a eval is evil wink.gif
b4x
23.04.2013, 09:46:53
@PricneOfPersia - tak testowałem smile.gif

http://jsfiddle.net/UHbT8/
PrinceOfPersia
23.04.2013, 09:58:50
no tak, robisz tam Slider.current, czyli powinno działać smile.gif
(z this.current by raczej nie zadziało wink.gif
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.