Forum PHP.pl

Forum PHP.pl > Forum > Przedszkole

Reply to this topic

Start new topic

[PHP] Błąd w kodzie

Neon12345 Zobacz profil	21.05.2010, 22:46:47 Post #1
Grupa: Zarejestrowani Postów: 74 Pomógł: 0 Dołączył: 2.04.2010 Skąd: Wrocław Ostrzeżenie: (0%)	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 if($_GET['buy']){ $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='".$user."'")); $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id=".$_GET['buy']." ")); $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'")); $b = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Buty' AND user='".$user."'")); $f = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Frotka' AND user='".$user."'")); $k = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Koszulka' AND user='".$user."'")); $s = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Spodnie' AND user='".$user."'")); if($query['kasa'] < $c['cena']){ echo " <script type='text/javascript'>window.alert('Nie masz tyle pieniędzy!');document.location.href = 'shop.php';</script> "; die(); } if($query['umiejetnosci'] < $c['skills']) { echo " <script type='text/javascript'>window.alert('Nie masz tyle umiejętności!');document.location.href = 'shop.php';</script> "; die(); } if($k['stan_koszulka'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch koszulek!');document.location.href = 'shop.php';</script> "; die(); } if($b['stan_buty'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par butów!');document.location.href = 'shop.php';</script> "; die(); } if($f['stan_frotka'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch Frotek!');document.location.href = 'shop.php';</script> "; die(); } if($s['stan_spodnie'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par Spodenek!');document.location.href = 'shop.php';</script> "; die(); } else{ $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='$user'")); $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]'")); $kk = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Koszulka'")); $ss = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Spodnie'")); $ff = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Frotka' ")); $bb = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Buty' ")); $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'")); $skills = $c['atak'] + $c['obrona'] + $c['strzaly'] + $c['rozgrywanie'] + $c['skutecznosc'] + $c['refleks'] + $c['podania'] + $c['kondycja']; 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()); mysql_query("UPDATE users SET kasa=kasa-'$c[cena]' WHERE user='$user' "); if ($ff['typ'] == 'Frotka') { mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Frotka' AND user='$user' ") or die (mysql_error()); } if ($kk['typ'] == 'Koszulka') { mysql_query("UPDATE shop_save SET stan_koszulka='1' WHERE typ='Koszulka' AND user='$user' ") or die (mysql_error()); } if ($ss['typ'] == 'Spodnie') { mysql_query("UPDATE shop_save SET stan_spodnie='1' WHERE typ='Spodnie' AND user='$user' ") or die (mysql_error()); } if ($bb['typ'] == 'Buty') { mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Buty' AND user='$user' ") or die (mysql_error()); } echo " <script type='text/javascript'>window.alert('Wyposażyłeś się w ".$c['nazwa']." !');document.location.href = 'items.php';</script> "; } } [PHP] pobierz, plaintext Przerobiłem na coś takiego ale dalej nic [PHP] pobierz, plaintext if($_GET['buy']){ $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='".$user."'")); $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id=".$_GET['buy']." ")); $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'")); $b = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Buty' AND user='".$user."'")); $f = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Frotka' AND user='".$user."'")); $k = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Koszulka' AND user='".$user."'")); $s = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE typ='Spodnie' AND user='".$user."'")); if($query['kasa'] < $c['cena']){ echo " <script type='text/javascript'>window.alert('Nie masz tyle pieniędzy!');document.location.href = 'shop.php';</script> "; die(); } if($query['umiejetnosci'] < $c['skills']) { echo " <script type='text/javascript'>window.alert('Nie masz tyle umiejętności!');document.location.href = 'shop.php';</script> "; die(); } else{ $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='$user'")); $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]'")); $kk = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Koszulka'")); $ss = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Spodnie'")); $ff = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Frotka' ")); $bb = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id='$_GET[buy]' AND typ='Buty' ")); $id = rand(10000000,90000000); $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'")); $skills = $c['atak'] + $c['obrona'] + $c['strzaly'] + $c['rozgrywanie'] + $c['skutecznosc'] + $c['refleks'] + $c['podania'] + $c['kondycja']; 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()); mysql_query("UPDATE users SET kasa=kasa-'$c[cena]' WHERE user='$user' "); if($ff['stan_frotka'] == 1){ if ($f['typ'] == 'Frotka') { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch Frotek!')ocument.location.href = 'shop.php';</script> "; die(); } mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Frotka' AND user='$user' ") or die (mysql_error()); } if ($kk['typ'] == 'Koszulka') { if($k['stan_koszulka'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch koszulek!');document.location.href = 'shop.php';</script> "; die(); } mysql_query("UPDATE shop_save SET stan_koszulka='1' WHERE typ='Koszulka' AND user='$user' ") or die (mysql_error()); } if ($ss['typ'] == 'Spodnie') { if($s['stan_spodnie'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par Spodenek!');document.location.href = 'shop.php';</script> "; die(); } mysql_query("UPDATE shop_save SET stan_spodnie='1' WHERE typ='Spodnie' AND user='$user' ") or die (mysql_error()); } if ($bb['typ'] == 'Buty') { if($b['stan_buty'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par butów!');document.location.href = 'shop.php';</script> "; die(); } mysql_query("UPDATE shop_save SET stan_buty='1' WHERE typ='Buty' AND user='$user' ") or die (mysql_error()); } echo " <script type='text/javascript'>window.alert('Wyposażyłeś się w ".$c['nazwa']." !');document.location.href = 'items.php';</script> "; } } [PHP] pobierz, plaintext Ten post edytował Neon12345 22.05.2010, 22:22:26

tehaha Zobacz profil	21.05.2010, 22:59:29 Post #2
Grupa: Zarejestrowani Postów: 1 748 Pomógł: 388 Dołączył: 21.08.2009 Skąd: Gdynia Ostrzeżenie: (0%)	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 Zobacz profil	22.05.2010, 00:39:53 Post #3
Grupa: Zarejestrowani Postów: 74 Pomógł: 0 Dołączył: 2.04.2010 Skąd: Wrocław Ostrzeżenie: (0%)	Lepiej?: [PHP] pobierz, plaintext if($_GET['buy']){ $cc = mysql_fetch_array(mysql_query("SELECT * FROM shop_save WHERE user='".$user."'")); $c = mysql_fetch_array(mysql_query("SELECT * FROM shop WHERE id=".$_GET['buy']." ")); $query = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE user='$user'")); if($query['kasa'] < $c['cena']){ echo " <script type='text/javascript'>window.alert('Nie masz tyle pieniędzy!');document.location.href = 'shop.php';</script> "; die(); } if($query['umiejetnosci'] < $c['skills']) { echo " <script type='text/javascript'>window.alert('Nie masz tyle umiejętności!');document.location.href = 'shop.php';</script> "; die(); } if($cc['stan_koszulka'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch koszulek!');document.location.href = 'shop.php';</script> "; die(); } if($cc['stan_buty'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par butów!');document.location.href = 'shop.php';</script> "; die(); } if($cc['stan_frotka'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch Frotek!');document.location.href = 'shop.php';</script> "; die(); } if($cc['stan_spodnie'] == 1) { echo " <script type='text/javascript'>window.alert('Nie możesz kupić dwóch par Spodenek!');document.location.href = 'shop.php';</script> "; die(); } else{ $skills = $c['atak'] + $c['obrona'] + $c['strzaly'] + $c['rozgrywanie'] + $c['skutecznosc'] + $c['refleks'] + $c['podania'] + $c['kondycja']; 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()); mysql_query("UPDATE users SET kasa=kasa-'$c[cena]' WHERE user='$user' "); if ($cc['typ'] == 'Frotka') { mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Frotka' AND user='$user' ") or die (mysql_error()); } if ($cc['typ'] == 'Koszulka') { mysql_query("UPDATE shop_save SET stan_koszulka='1' WHERE typ='Koszulka' AND user='$user' ") or die (mysql_error()); } if ($cc['typ'] == 'Spodnie') { mysql_query("UPDATE shop_save SET stan_spodnie='1' WHERE typ='Spodnie' AND user='$user' ") or die (mysql_error()); } if ($cc['typ'] == 'Buty') { mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='Buty' AND user='$user' ") or die (mysql_error()); } echo " <script type='text/javascript'>window.alert('Wyposażyłeś się w ".$c['nazwa']." !');document.location.href = 'items.php';</script> "; } } [PHP] pobierz, plaintext

tehaha Zobacz profil	22.05.2010, 10:39:31 Post #4
Grupa: Zarejestrowani Postów: 1 748 Pomógł: 388 Dołączył: 21.08.2009 Skąd: Gdynia Ostrzeżenie: (0%)	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 $types = array('Frotka', 'Koszulka', 'Spodnie', 'Buty'); if(in_array($cc['typ'], $types)) { mysql_query("UPDATE shop_save SET stan_frotka='1' WHERE typ='".$cc['typ']."' AND user='$user' ") or die (mysql_error()); } [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

« Następny starszy · Przedszkole · Następny nowszy »

Reply to this topic

Start new topic

1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)

0 Zarejestrowanych:

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

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

Wersja Lo-Fi

Aktualny czas: 12.06.2025 - 19:35

Powered By IP.Board © 2025 IPS, Inc.
All changes by PHP.pl Administrators

Hosting zapewnia

NQ.pl hosting, trac, svn