Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP] Błąd w kodzie
Forum PHP.pl > Forum > Przedszkole
Neon12345
21.05.2010, 22:46:47
Witam , otóż mam skrypt sklep i próbuje zrobić zabezpieczenie w którym nie można kupić dwóch tych samych typów rzeczy , lecz nie działa poprawnie. Gdy kupię koszulkę , a potem chcę kupić fortkę wyskakuje że nie można kupić 2 tych samych koszulek. Gdzie mam błąd?

[PHP] pobierz, plaintext 
  1. if($_GET['buy']){
  2.  
  3. $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='".$user."'"));
  4. $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id=".$_GET['buy']." "));
  5. $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'"));
  6. $b = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Buty' AND user='".$user."'"));
  7. $f = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Frotka' AND user='".$user."'"));
  8. $k = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Koszulka' AND user='".$user."'"));
  9. $s = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Spodnie' AND user='".$user."'"));
  10.  
  11. if($query['kasa'] < $c['cena']){
  12. echo "
  13. <script type='text/javascript'>window.alert('Nie masz tyle pieniędzy!');document.location.href = 'shop.php';</script>
  14. "; die();
  15. }
  16. if($query['umiejetnosci'] < $c['skills'])
  17. {
  18. echo "
  19. <script type='text/javascript'>window.alert('Nie masz tyle umiejętności!');document.location.href = 'shop.php';</script>
  20. "; die();
  21. }
  22. if($k['stan_koszulka'] == 1)
  23. {
  24. echo "
  25. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch koszulek!');document.location.href = 'shop.php';</script>
  26. "; die();
  27. }
  28. if($b['stan_buty'] == 1)
  29. {
  30. echo "
  31. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par butów!');document.location.href = 'shop.php';</script>
  32. "; die();
  33. }
  34. if($f['stan_frotka'] == 1)
  35. {
  36. echo "
  37. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch Frotek!');document.location.href = 'shop.php';</script>
  38. "; die();
  39. }
  40. if($s['stan_spodnie'] == 1)
  41. {
  42. echo "
  43. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par Spodenek!');document.location.href = 'shop.php';</script>
  44. "; die();
  45. }
  46. else{
  47. $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='$user'"));
  48. $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]'"));
  49. $kk = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Koszulka'"));
  50. $ss = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Spodnie'"));
  51. $ff = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Frotka' "));
  52. $bb = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Buty' "));
  53. $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'"));
  54. $skills = $c['atak'] + $c['obrona'] + $c['strzaly'] + $c['rozgrywanie'] + $c['skutecznosc'] + $c['refleks'] + $c['podania'] + $c['kondycja'];
  55. mysql_query("INSERT INTO `shop_save` ( `nazwa`, `atak`, `kondycja`, `rozgrywanie`, `skutecznosc`, `refleks`, `podania`, `strzaly`, `obrona`, `user`, `typ`, `warty`, `skills`) values ('".$c['nazwa']."', ".$c['atak'].", ".$c['kondycja'].", ".$c['rozgrywanie'].", ".$c['skutecznosc'].", ".$c['refleks'].", ".$c['podania'].", ".$c['strzaly'].", ".$c['obrona'].", '".$query['user']."', '".$c['typ']."', ".$c['warty'].", ".$skills.") ") or die (mysql_error());
  56. mysql_query("UPDATE users SET kasa=kasa-'$c[cena]' WHERE user='$user' ");
  57. if ($ff['typ'] == 'Frotka') {
  58. mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Frotka' AND user='$user' ") or die (mysql_error());
  59. }  
  60. if ($kk['typ'] == 'Koszulka') {
  61. mysql_query("UPDATE shop_save SET stan_koszulka='1' WHERE typ='Koszulka' AND user='$user' ") or die (mysql_error());
  62. }
  63. if ($ss['typ'] == 'Spodnie') {
  64. mysql_query("UPDATE shop_save SET stan_spodnie='1' WHERE typ='Spodnie' AND user='$user' ") or die (mysql_error());
  65. }  
  66. if ($bb['typ'] == 'Buty') {
  67. mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Buty' AND user='$user' ") or die (mysql_error());
  68. } 
  69. echo "
  70. <script type='text/javascript'>window.alert('Wyposażyłeś się w ".$c['nazwa']." !');document.location.href = 'items.php';</script>
  71. ";
  72. }
  73.  
  74. }
[PHP] pobierz, plaintext


Przerobiłem na coś takiego ale dalej nic

[PHP] pobierz, plaintext 
  1. if($_GET['buy']){
  2.  
  3. $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='".$user."'"));
  4. $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id=".$_GET['buy']." "));
  5. $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'"));
  6. $b = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Buty' AND user='".$user."'"));
  7. $f = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Frotka' AND user='".$user."'"));
  8. $k = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Koszulka' AND user='".$user."'"));
  9. $s = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Spodnie' AND user='".$user."'"));
  10.  
  11. if($query['kasa'] < $c['cena']){
  12. echo "
  13. <script type='text/javascript'>window.alert('Nie masz tyle pieniędzy!');document.location.href = 'shop.php';</script>
  14. "; die();
  15. }
  16. if($query['umiejetnosci'] < $c['skills'])
  17. {
  18. echo "
  19. <script type='text/javascript'>window.alert('Nie masz tyle umiejętności!');document.location.href = 'shop.php';</script>
  20. "; die();
  21. }
  22.  
  23.  
  24.  
  25.  
  26. else{
  27. $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='$user'"));
  28. $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]'"));
  29. $kk = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Koszulka'"));
  30. $ss = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Spodnie'"));
  31. $ff = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Frotka' "));
  32. $bb = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Buty' "));
  33. $id = rand(10000000,90000000);
  34. $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'"));
  35. $skills = $c['atak'] + $c['obrona'] + $c['strzaly'] + $c['rozgrywanie'] + $c['skutecznosc'] + $c['refleks'] + $c['podania'] + $c['kondycja'];
  36. mysql_query("INSERT INTO `shop_save` ( `id`, `nazwa`, `atak`, `kondycja`, `rozgrywanie`, `skutecznosc`, `refleks`, `podania`, `strzaly`, `obrona`, `user`, `typ`, `warty`, `skills`) values ('".$id."', '".$c['nazwa']."', ".$c['atak'].", ".$c['kondycja'].", ".$c['rozgrywanie'].", ".$c['skutecznosc'].", ".$c['refleks'].", ".$c['podania'].", ".$c['strzaly'].", ".$c['obrona'].", '".$query['user']."', '".$c['typ']."', ".$c['warty'].", ".$skills.") ") or die (mysql_error());
  37. mysql_query("UPDATE users SET kasa=kasa-'$c[cena]' WHERE user='$user' ");
  38. if($ff['stan_frotka'] == 1){
  39. if ($f['typ'] == 'Frotka') {
  40. echo "
  41. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch Frotek!')ocument.location.href = 'shop.php';</script>
  42. "; die();
  43. }
  44. mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Frotka' AND user='$user' ") or die (mysql_error());
  45. }
  46. if ($kk['typ'] == 'Koszulka') {
  47. if($k['stan_koszulka'] == 1)
  48. {
  49. echo "
  50. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch koszulek!');document.location.href = 'shop.php';</script>
  51. "; die();
  52. }
  53. mysql_query("UPDATE shop_save SET stan_koszulka='1' WHERE typ='Koszulka' AND user='$user' ") or die (mysql_error());
  54. }
  55. if ($ss['typ'] == 'Spodnie') {
  56. if($s['stan_spodnie'] == 1)
  57. {
  58. echo "
  59. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par Spodenek!');document.location.href = 'shop.php';</script>
  60. "; die();
  61. }
  62. mysql_query("UPDATE shop_save SET stan_spodnie='1' WHERE typ='Spodnie' AND user='$user' ") or die (mysql_error());
  63. }  
  64. if ($bb['typ'] == 'Buty') {
  65. if($b['stan_buty'] == 1)
  66. {
  67. echo "
  68. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par butów!');document.location.href = 'shop.php';</script>
  69. "; die();
  70. }
  71. mysql_query("UPDATE shop_save SET stan_buty='1' WHERE typ='Buty' AND user='$user' ") or die (mysql_error());
  72. } 
  73. echo "
  74. <script type='text/javascript'>window.alert('Wyposażyłeś się w ".$c['nazwa']." !');document.location.href = 'items.php';</script>
  75. ";
  76. }
  77.  
  78. }
[PHP] pobierz, plaintext
tehaha
21.05.2010, 22:59:29
ale rzeźnia....5-6 połączeń żeby wyciągnąć dane z jednej tabeli? a jak będziesz miał 50 typów to zrobisz 50 połączeń, nie zrozum mnie źle ale nie ma co szukać tu błędu bo ten cały kod jest błędem, lepiej zacznij od początku
Neon12345
22.05.2010, 00:39:53
Lepiej?:

[PHP] pobierz, plaintext 
  1. if($_GET['buy']){
  2.  
  3. $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='".$user."'"));
  4. $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id=".$_GET['buy']." "));
  5. $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'"));
  6.  
  7.  
  8. if($query['kasa'] < $c['cena']){
  9. echo "
  10. <script type='text/javascript'>window.alert('Nie masz tyle pieniędzy!');document.location.href = 'shop.php';</script>
  11. "; die();
  12. }
  13. if($query['umiejetnosci'] < $c['skills'])
  14. {
  15. echo "
  16. <script type='text/javascript'>window.alert('Nie masz tyle umiejętności!');document.location.href = 'shop.php';</script>
  17. "; die();
  18. }
  19. if($cc['stan_koszulka'] == 1)
  20. {
  21. echo "
  22. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch koszulek!');document.location.href = 'shop.php';</script>
  23. "; die();
  24. }
  25. if($cc['stan_buty'] == 1)
  26. {
  27. echo "
  28. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par butów!');document.location.href = 'shop.php';</script>
  29. "; die();
  30. }
  31. if($cc['stan_frotka'] == 1)
  32. {
  33. echo "
  34. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch Frotek!');document.location.href = 'shop.php';</script>
  35. "; die();
  36. }
  37. if($cc['stan_spodnie'] == 1)
  38. {
  39. echo "
  40. <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par Spodenek!');document.location.href = 'shop.php';</script>
  41. "; die();
  42. }
  43. else{
  44. $skills = $c['atak'] + $c['obrona'] + $c['strzaly'] + $c['rozgrywanie'] + $c['skutecznosc'] + $c['refleks'] + $c['podania'] + $c['kondycja'];
  45. mysql_query("INSERT INTO `shop_save` ( `nazwa`, `atak`, `kondycja`, `rozgrywanie`, `skutecznosc`, `refleks`, `podania`, `strzaly`, `obrona`, `user`, `typ`, `warty`, `skills`) values ('".$c['nazwa']."', ".$c['atak'].", ".$c['kondycja'].", ".$c['rozgrywanie'].", ".$c['skutecznosc'].", ".$c['refleks'].", ".$c['podania'].", ".$c['strzaly'].", ".$c['obrona'].", '".$query['user']."', '".$c['typ']."', ".$c['warty'].", ".$skills.") ") or die (mysql_error());
  46. mysql_query("UPDATE users SET kasa=kasa-'$c[cena]' WHERE user='$user' ");
  47.  
  48. if ($cc['typ'] == 'Frotka') {
  49. mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Frotka' AND user='$user' ") or die (mysql_error());
  50. }  
  51. if ($cc['typ'] == 'Koszulka') {
  52. mysql_query("UPDATE shop_save SET stan_koszulka='1' WHERE typ='Koszulka' AND user='$user' ") or die (mysql_error());
  53. }
  54. if ($cc['typ'] == 'Spodnie') {
  55. mysql_query("UPDATE shop_save SET stan_spodnie='1' WHERE typ='Spodnie' AND user='$user' ") or die (mysql_error());
  56. }  
  57. if ($cc['typ'] == 'Buty') {
  58. mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Buty' AND user='$user' ") or die (mysql_error());
  59. } 
  60. echo "
  61. <script type='text/javascript'>window.alert('Wyposażyłeś się w ".$c['nazwa']." !');document.location.href = 'items.php';</script>
  62. ";
  63. }
  64.  
  65. }  
[PHP] pobierz, plaintext
tehaha
22.05.2010, 10:39:31
no lepiej ale i tak słabo, rzecz w tym, że wielokrotnie powtarzasz prawie identyczny kod, zamiast np używać pętli lub innych funkcji, przykładowo linijki 48-58 mógłbyś zastąpić:
[PHP] pobierz, plaintext 
  1. $types = array('Frotka', 'Koszulka', 'Spodnie', 'Buty');
  2. if(in_array($cc['typ'], $types))
  3. {
  4. 	mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='".$cc['typ']."' AND user='$user' ") or die (mysql_error());
  5. }
[PHP] pobierz, plaintext

analogicznie mógłbyś wykonać między linijkami 19-42

Ponadto parząc na to, że danych nie pobierasz w pętli tylko tak jednorazowo to wygląda na to, że dla każdego produktu utworzyłeś oddzielną kolumnę w tabeli, a to też jest nie prawidłowy sposób, więc dobrze było by się cofnąć do zaprojektowania odpowiednio bazy danych,
np.
TABELA PRODUKTY
product_id | product_label | product_desc | product_count

TABELA STAN PRODUKTU
status_id | product_id | status_value

i przykładowe dane:

1 | stan_koszulka | 1
2 | stan_buty | 0
3 | stan_frotka | 1
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.