Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [JavaScript]Tablica - działa tylko ostatni element
Forum PHP.pl > Forum > Przedszkole
Glarden
8.05.2008, 14:19:03
Joł, joł. Robię w JS grę w pokera. Skrypt wygląda na razie tak:
[HTML] pobierz, plaintext 
  1. <script>
  2. 	karty = new Array();
  3. 	karty[0] = 'as_pik';
  4. 	karty[1] = 'as_trefl';
  5. 	karty[2] = 'as_karo';
  6. 	karty[3] = 'as_kier';
  7.  
  8. 	karty[4] = 'krol_pik';
  9. 	karty[5] = 'krol_trefl';
  10. 	karty[6] = 'krol_karo';
  11. 	karty[7] = 'krol_kier';
  12.  
  13. 	karty[8] = 'dama_pik';
  14. 	karty[9] = 'dama_trefl';
  15. 	karty[10] = 'dama_karo';
  16. 	karty[11] = 'dama_kier';
  17.  
  18. 	karty[12] = 'jop_pik';
  19. 	karty[13] = 'jop_trefl';
  20. 	karty[14] = 'jop_karo';
  21. 	karty[15] = 'jop_kier';
  22.  
  23. 	karty[16] = '10_pik';
  24. 	karty[17] = '10_trefl';
  25. 	karty[18] = '10_karo';
  26. 	karty[19] = '10_kier';
  27.  
  28. 	karty[20] = '9_pik';
  29. 	karty[21] = '9_trefl';
  30. 	karty[22] = '9_karo';
  31. 	karty[23] = '9_kier';
  32.  
  33. 	function rozdaj() {
  34. 		for (i=0;i<5;i++) {
  35. 			karta = Math.floor(Math.random() * 23) +1;
  36. 			karty_gracz = new Array();
  37. 			karty_gracz[i] = karty[karta];
  38. 			karty.splice(karta, 1);
  39. 		}
  40. 		pokaz_karty();
  41. 	}
  42. 	function pokaz_karty() {
  43. 		for (i=0;i<5;i++) {
  44. 			document.getElementById("karty").innerHTML += "<br>"+karty_gracz[i];
  45. 		}
  46. 	}
  47.  
  48. </script>
  49. <body onLoad="rozdaj();">
  50.  
  51. 	<div id="licznik">
  52. 		Kasa: 1000
  53. 	</div>
  54. 	<a href="javascript:wchodze();">Wchodzę</a><br>
  55.  
  56. 	<div id="karty">
  57. 	</div>
  58.  
  59. </body>
[HTML] pobierz, plaintext


Przy włączeniu strony, uruchamiana jest funkcja rozdająca karty użytkownikowi (rozdaj()winksmiley.jpg. I teraz problem jest taki, że kiedy wyświetla te karty (funkcją pokaz_karty()winksmiley.jpg zamiast pierwszych czterech kart pokazuje "undefined", natomiast piąta działa w 80% (tak na oko) przypadków, w pozostałych też jest "undefined". I zwracam się do Was z pytaniem: co się dzieje się?
lord_t
8.05.2008, 15:11:51
Kod
function rozdaj() {
karty_gracz = new Array();
for (i=0;i<5;i++) {
karta = Math.floor(Math.random() * karty.length-1) +1;alert(karta);
karty_gracz[i] = karty[karta];
karty.splice(karta, 1);
}
pokaz_karty();
}

Za każdym krokiem w pętli tworzyłeś nową tablicę (z jednym elementem) nadpisując poprzednią.

Drugi błąd: cały czas losowałeś z przedziału 0-23 a za każdym krokiem w pętli usuwałeś jeden element.
phpion
8.05.2008, 15:19:14
Moim zdaniem są tutaj 2 problemy.
1) Jak zauważył ~lord_t: za każdym razem zerowałeś tablicę karty_gracz. To jednak nie ma wpływu na Twój problem - wyszłoby później.
2) Musisz zdefiniować tablicę karty_gracz jako globalną czyli poza ciałem funkcji rozdaj. Dlaczego? Aktualnie tablica ta jest tylko lokalną tablicą dla tej funkcji i nie jest widoczna poza nią. Zadeklaruj karty_gracz poza funkcją.

Finalnie powinno to wyglądać tak:
[HTML] pobierz, plaintext 
  1. <script>
  2. 	karty_gracz = new Array();  // <- pojawila sie deklaracja tablicy jako globalnej
  3.  
  4. 	karty = new Array();
  5. 	karty[0] = 'as_pik';
  6. 	karty[1] = 'as_trefl';
  7. 	karty[2] = 'as_karo';
  8. 	karty[3] = 'as_kier';
  9.  
  10. 	karty[4] = 'krol_pik';
  11. 	karty[5] = 'krol_trefl';
  12. 	karty[6] = 'krol_karo';
  13. 	karty[7] = 'krol_kier';
  14.  
  15. 	karty[8] = 'dama_pik';
  16. 	karty[9] = 'dama_trefl';
  17. 	karty[10] = 'dama_karo';
  18. 	karty[11] = 'dama_kier';
  19.  
  20. 	karty[12] = 'jop_pik';
  21. 	karty[13] = 'jop_trefl';
  22. 	karty[14] = 'jop_karo';
  23. 	karty[15] = 'jop_kier';
  24.  
  25. 	karty[16] = '10_pik';
  26. 	karty[17] = '10_trefl';
  27. 	karty[18] = '10_karo';
  28. 	karty[19] = '10_kier';
  29.  
  30. 	karty[20] = '9_pik';
  31. 	karty[21] = '9_trefl';
  32. 	karty[22] = '9_karo';
  33. 	karty[23] = '9_kier';
  34.  
  35. 	function rozdaj() {
  36. 		karty_gracz = new Array(); // <- pojawilo sie zerowanie tablicy
  37.  
  38. 		for (i=0;i<5;i++) {
  39. 			karta = Math.floor(Math.random() * 23) +1;
  40. 			// karty_gracz = new Array(); <- tego tu nie ma
  41. 		 karty_gracz[i] = karty[karta];
  42. 			karty.splice(karta, 1);
  43. 		}
  44. 		pokaz_karty();
  45. 	}
  46. 	function pokaz_karty() {
  47. 		for (i=0;i<5;i++) {
  48. 			document.getElementById("karty").innerHTML += "<br>"+karty_gracz[i];
  49. 		}
  50. 	}
  51.  
  52. </script>
  53. <body onLoad="rozdaj();">
  54.  
  55. 	<div id="licznik">
  56. 		Kasa: 1000
  57. 	</div>
  58. 	<a href="javascript:wchodze();">Wchodzę</a><br>
  59.  
  60. 	<div id="karty">
  61. 	</div>
  62.  
  63. </body>
[HTML] pobierz, plaintext


PS: nie testowałem ale wydaje mi się, że powinno pomóc.
Glarden
8.05.2008, 16:06:49
Prawie działa, jeszcze tylko jeden problem. Wspomniałem wcześniej, że raz na jakiś czas nawet ta ostatnia karta, działająca już w moim skrypcie, czasem jest "undefined". Teraz jest tak z wszystkimi kartami - w większości przypadków działa fajnie, ale czasem jedna z kart jest niezdefiniowana...
lord_t
8.05.2008, 16:58:14
To przeczytaj mojego posta wyżej. :|
Glarden
8.05.2008, 21:17:52
O kurde, ale ja głupi jestem. Przepraszam i dziękuję za pomoc!
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.