Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [JavaScript] setTimeOut, Funkcja wykonuje się co chwile
Lirdoner
post
Post #1





Grupa: Zarejestrowani
Postów: 500
Pomógł: 1
Dołączył: 29.09.2009

Ostrzeżenie: (0%)
-----

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ć?
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi (1 - 9)
nospor
post
Post #2





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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...
Go to the top of the page
+Quote Post
PrinceOfPersia
post
Post #3





Grupa: Zarejestrowani
Postów: 717
Pomógł: 120
Dołączył: 18.04.2009

Ostrzeżenie: (0%)
-----

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.
Go to the top of the page
+Quote Post
timon27
post
Post #4





Grupa: Zarejestrowani
Postów: 578
Pomógł: 69
Dołączył: 15.04.2007
Skąd: Wrocław

Ostrzeżenie: (0%)
-----

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.
Go to the top of the page
+Quote Post
PrinceOfPersia
post
Post #5





Grupa: Zarejestrowani
Postów: 717
Pomógł: 120
Dołączył: 18.04.2009

Ostrzeżenie: (0%)
-----

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.

Ten post edytował PrinceOfPersia 18.04.2013, 09:11:26
Go to the top of the page
+Quote Post
timon27
post
Post #6





Grupa: Zarejestrowani
Postów: 578
Pomógł: 69
Dołączył: 15.04.2007
Skąd: Wrocław

Ostrzeżenie: (0%)
-----

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
Go to the top of the page
+Quote Post
b4x
post
Post #7





Grupa: Zarejestrowani
Postów: 658
Pomógł: 95
Dołączył: 20.12.2005
Skąd: N54,35° E18,63° (Gdańsk)

Ostrzeżenie: (0%)
-----

Pozwoliłem sobie (IMG:style_emoticons/default/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
Go to the top of the page
+Quote Post
PrinceOfPersia
post
Post #8





Grupa: Zarejestrowani
Postów: 717
Pomógł: 120
Dołączył: 18.04.2009

Ostrzeżenie: (0%)
-----

@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 (IMG:style_emoticons/default/wink.gif)

Ten post edytował PrinceOfPersia 23.04.2013, 09:37:53
Go to the top of the page
+Quote Post
b4x
post
Post #9





Grupa: Zarejestrowani
Postów: 658
Pomógł: 95
Dołączył: 20.12.2005
Skąd: N54,35° E18,63° (Gdańsk)

Ostrzeżenie: (0%)
-----

@PricneOfPersia - tak testowałem (IMG:style_emoticons/default/smile.gif)

http://jsfiddle.net/UHbT8/
Go to the top of the page
+Quote Post
PrinceOfPersia
post
Post #10





Grupa: Zarejestrowani
Postów: 717
Pomógł: 120
Dołączył: 18.04.2009

Ostrzeżenie: (0%)
-----

no tak, robisz tam Slider.current, czyli powinno działać (IMG:style_emoticons/default/smile.gif)
(z this.current by raczej nie zadziało (IMG:style_emoticons/default/wink.gif)
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Przełącz na: Standardowy · Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 22.08.2025 - 19:32
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn