Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [php][mysql] Błąd w skrypcie
Forum PHP.pl > Forum > Przedszkole
Gość
16.08.2007, 17:53:29
Witam!
Piszę skrypt i mam błąd i kombinuje chyba z 2 godziny i nic nie wymyśliłem, coś jest pewnie z instrukcjami warunkowymi.
Nie mogę doszukać się błędu. Oczywiście wiem, że forum nie słuzy do poprawiania składni ale będę bardzo wdzięczny.
Pozdrawiam



Błąd jest na linii 146 czyli pod koniec, tak jakby brakowało jednego } ale nie wiem. Z góry dziękuję
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3. require_once "funkcje.php";
  4. pol_mysql();
  5. echo '<center>';
  6.  
  7. if ($_SESSION['user']) {
  8.  
  9. include "tabelaleft.php";
  10.  
  11. echo '<td width="550" height="700"><center><b>';
  12.  
  13. $pokazl = "SELECT bron, tarcza, czar, lvl, nbron, ntarcza, nczar, zloto, profesja, xy 
    FROM users WHERE login = '"    . $_SESSION['user'] ."'";
  14. $wykonaj = mysql_query($pokazl);
  15. while($wiersz = mysql_fetch_array($wykonaj))
  16. {
  17. $bron = $wiersz['bron'];
  18. $tarcza = $wiersz['tarcza'];
  19. $czar = $wiersz['czar'];
  20. $nbron = $wiersz['nbron'];
  21. $ntarcza = $wiersz['ntarcza'];
  22. $nczar = $wiersz['nczar'];
  23. $zloto = $wiersz['zloto'];
  24. $profesja = $wiersz['profesja'];
  25. $lvl = $wiersz['lvl'];
  26. $xy = $wiersz['xy'];
  27. }
  28.  
  29. if ($profesja != 'Mag') { echo '<b><center>Tylko dla magów</b></center>'; } else {
  30.  
  31. echo 'Kupując broń bez sprzedania starej, stracisz takową nie dostając za nią nic!<br>_____________________________________<br><br>';
  32.  
  33.  
  34. if ($xy == 1 || $xy == 3) {
  35.  
  36. if ($nbron != 'Drewniana różdżka') {
  37.  
  38. if (!empty($_POST['drewnianarozdzka'])) {
  39. if ($zloto >= 200) {
  40. mysql_query("UPDATE users SET bron = 5 WHERE login = '". $_SESSION['user'] ."'");
  41. mysql_query("UPDATE users SET nbron = 'Drewniana różdżka' WHERE login = '". $_SESSION['user'] ."'");
  42. mysql_query("UPDATE users SET zloto = $zloto-200 WHERE login = '". $_SESSION['user'] ."'");
  43. echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
  44. } else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
  45. } else {
  46. echo '
  47. <form action="wkostury.php" method="POST">
  48. <INPUT TYPE="hidden" name="drewnianarozdzka" value="1">
  49. DREWNIANA RÓŻDŻKA<br>200zł, ATAK: 5 <input type="image" src="buttons/kup.png" border="0">
  50. </form>';
  51. }
  52. }
  53. }
  54.  
  55. if ($nbron != 'Różdżka') {
  56. if (!empty($_POST['rozdzka'])) {
  57. if ($zloto >= 500) {
  58. mysql_query("UPDATE users SET bron = 13 WHERE login = '". $_SESSION['user'] ."'");
  59. mysql_query("UPDATE users SET nbron = 'Różdżka' WHERE login = '". $_SESSION['user'] ."'");
  60. mysql_query("UPDATE users SET zloto = $zloto-500 WHERE login = '". $_SESSION['user'] ."'");
  61. echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
  62. } else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
  63. } else {
  64. echo '
  65. <form action="wkostury.php" method="POST">
  66. <INPUT TYPE="hidden" name="rozdzka" value="1">
  67. RÓŻDŻKA<br>500zł, ATAK: 13 <input type="image" src="buttons/kup.png" border="0">
  68. </form>';
  69. }
  70. }
  71. }
  72.  
  73. if ($nbron != 'Kościana różdżka') {
  74.  
  75. if ($lvl >= 4) {
  76.  
  77. if (!empty($_POST['koscianarozdzka'])) {
  78. if ($zloto >= 2000) {
  79. mysql_query("UPDATE users SET bron = 30 WHERE login = '". $_SESSION['user'] ."'");
  80. mysql_query("UPDATE users SET nbron = 'Kościana różdżka' WHERE login = '". $_SESSION['user'] ."'");
  81. mysql_query("UPDATE users SET zloto = $zloto-2000 WHERE login = '". $_SESSION['user'] ."'");
  82. echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
  83. } else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
  84. } else {
  85. echo '
  86. <form action="wkostury.php" method="POST">
  87. <INPUT TYPE="hidden" name="koscianarozdzka" value="1">
  88. KOŚCIANA RÓŻDŻKA<br>2000zł, ATAK: 30 <input type="image" src="buttons/kup.png" border="0">
  89. </form>';
  90. }
  91. }
  92. }
  93.  
  94.  
  95. if ($nbron != 'Różdżka Ildara') {
  96. if ($lvl >= 7) {
  97.  
  98. if (!empty($_POST['rozdzkaildara'])) {
  99. if ($zloto >= 4500) {
  100. mysql_query("UPDATE users SET bron = 80 WHERE login = '". $_SESSION['user'] ."'");
  101. mysql_query("UPDATE users SET nbron = 'Różdżka Ildara' WHERE login = '". $_SESSION['user'] ."'");
  102. mysql_query("UPDATE users SET zloto = $zloto-4500 WHERE login = '". $_SESSION['user'] ."'");
  103. echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
  104. } else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
  105. } else {
  106. echo '
  107. <form action="wkostury.php" method="POST">
  108. <INPUT TYPE="hidden" name="rozdzkaildara" value="1">
  109. RÓŻDŻKA ILDARA<br>4500zł, ATAK: 80 <input type="image" src="buttons/kup.png" border="0">
  110. </form>';
  111. }
  112. }
  113. }
  114.  
  115. if ($nbron != 'Eukanos') {
  116. if ($lvl >= 10) {
  117.  
  118. if (!empty($_POST['eukanos'])) {
  119. if ($zloto >= 7000) {
  120. mysql_query("UPDATE users SET bron = 150 WHERE login = '". $_SESSION['user'] ."'");
  121. mysql_query("UPDATE users SET nbron = 'Eukanos' WHERE login = '". $_SESSION['user'] ."'");
  122. mysql_query("UPDATE users SET zloto = $zloto-7000 WHERE login = '". $_SESSION['user'] ."'");
  123. echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
  124. } else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
  125. } else {
  126. echo '
  127. <form action="wkostury.php" method="POST">
  128. <INPUT TYPE="hidden" name="eukanos" value="1">
  129. EUKANOS<br>7000zł, ATAK: 150 <input type="image" src="buttons/kup.png" border="0">
  130. </form>';
  131. }
  132. }
  133. }
  134.  
  135.  
  136.  
  137.  
  138.  
  139.  
  140. }
  141.  
  142.  
  143.  
  144. echo '<br><br>_____________________________________<br>';
  145.  
  146. } else { echo '<b>BŁĄD!'; }
  147.  
  148. include "tabelaright.php";
  149.  
  150. } else {
  151. echo "Wróć i zaloguj się!";
  152. }
  153.  
  154. ?>
[PHP] pobierz, plaintext
Hazel
16.08.2007, 18:49:33
Proponuję zedytować kod, zrobić wcięcia i przestrzegać miłej konwencji nawiasów klamrowych typu
[PHP] pobierz, plaintext 
  1. <?php
  2. if()
  3. {
  4. }
  5. ?>
[PHP] pobierz, plaintext


zamiast

[PHP] pobierz, plaintext 
  1. <?php
  2. if() {
  3. }
  4. ?>
[PHP] pobierz, plaintext


Wprawdzie obie konwencje są zalecane, prawidłowe i równie popularne, ale, nie tylko moim zdaniem, pierwsza, choć zajmuje więcej linii, jest znacznie łatwiejsza do odczytania - ja ją stosuję i nie mam problemów z nawiasami klamrowymi, wszystko jest czytelne i przejrzyste, nawet przy bardzo dużych i rozbudowanych w pętle oraz instrukcje warunkowe plikach.

Skopiowałem sobie do edytora ten Twój kod i po usunięciu wierszy dołączających pliki też mi w tej samej linii wyskakuje błąd, dlatego myślę, że to nawiasy klamrowe - nie widzę innego błędu składni czy parsowania w pobliżu tamtego miejsca...

Jeszcze jedna uwaga - pętlę while z linii 15-27 (zajmuje 13 linijek) można zastąpić następującym blokiem:

[PHP] pobierz, plaintext 
  1. <?php
  2. while ($wiersz = mysql_fetch_array($wykonaj))
  3. {
  4.   extract($wiersz);
  5. }
  6. ?>
[PHP] pobierz, plaintext


Nieco łatwiej i krócej, nieprawdaż?
Funkcja extract() tworzy zmienne o wartościach takich jak klucze tablicy. Ma tam parę wyjątków - nie zawsze działa prawidłowo w tej formie, domyślnie nadpisuje istniejące zmienne itp., ale większość jej niedoskonałości można zniwelować przez podanie jej opcjonalnych parametrów (które w tym przypadku nie są ani trochę potrzebne). Chcesz to zajrzyj do dokumentacji, bo nie chce mi się opisywać czegoś, o co nie pytałeś. Tak tylko mówię, żebyś zoptymalizował trochę kod (a raczej go skrócił tongue.gif) (zreszta, można to samo zrobić pętlą foreach lub dowolną inną, tyle że to już jest zaimplementowane... w każdym razie nie należy zajmować 13 linii kodu tam, gdzie potrzeba 4 (w przypadku extract()) lub 5 (w przypadku foreach)). A jak zrobisz coś z tymi nawiasami klamrowymi, to sam będziesz wiedział, gdzie jest błąd (a jak nie to pisz).

edit: A tak w ogóle to prawdopodobnie nie potrzeba tam 
[PHP] pobierz, plaintext 
  1. <?php
  2. while ($wiersz = mysql_fetch_array($wykonaj))
  3. ?>
[PHP] pobierz, plaintext
wystarczy tylko 
[PHP] pobierz, plaintext 
  1. <?php
  2. $wiersz = mysql_fetch_array($wykonaj)
  3. ?>
[PHP] pobierz, plaintext

No chyba że chcesz, żeby wartościami zmiennych były dane pobrane z ostatniego odczytanego rekordu tablicy, a nie z pierwszego...
Bo robiąc pętlę tak jakby chcesz, żeby te zmienne posiadały tyle wartości, ile jest rekordów w bazie danych, a to przecież nie tablice, tylko zwyczajne zmienne skalarne tongue.gif Stąd wnioskuję, że liczba wierszy wyniku MySQL $wykonaj jest równa 1, w przeciwnym przypadku kod działa Ci błędnie. A jeśli wiersz jest jeden, to nie potrzebujesz pętli.
kresh
16.08.2007, 18:59:54
od linii 140:
[PHP] pobierz, plaintext 
  1. <?php
  2. } // to zamyka warunek z 7 linii: if ($_SESSION['user']) {
  3.  
  4.  
  5.  
  6. echo '<br><br>_____________________________________<br>';
  7.  
  8. } else { echo '<b>BŁĄD!'; }
  9.  
  10. include "tabelaright.php";
  11.  
  12. } else {
  13. echo "Wróć i zaloguj się!";
  14. }
  15. ?>
[PHP] pobierz, plaintext


Tak jak napisał Hazel powinieneś się zaprzyjaźnić z Tabulatorem smile.gif
Mnie pomaga też notepad++ który podświetla pary klamr - otwierającą i zamykającą
Gość
16.08.2007, 19:26:46
Wielkie dzięki za odpowiedź, że się tak rozpisaliście biggrin.gif

P.S.

Po poprawieniu błędu przerobie ten kod na bardziej czytelniejszy jak napisaliście.

Kresh, napisałeś:
} // to zamyka warunek z 7 linii: if ($_SESSION['user']) {

o co chodzi?
Po if ($_SESSION['user']) mam jeszcze sprawdzenie profesji i potem XY, więc te 2 instrukcje trzeba chyba zamknąć przed zakonczeniem $_SESSION['user']. Nie jestem jakimś zawodowcem i nie zbyt wiem jak to zrobić winksmiley.jpg z góry dzięki
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.