Forum PHP.pl

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 
<?php
session_start();
require_once "funkcje.php";
pol_mysql();
echo '<center>';
 
if ($_SESSION['user']) {
 
include "tabelaleft.php";
 
echo '<td width="550" height="700"><center><b>';
 
$pokazl = "SELECT bron, tarcza, czar, lvl, nbron, ntarcza, nczar, zloto, profesja, xy 
FROM users WHERE login = '". $_SESSION['user'] ."'";
$wykonaj = mysql_query($pokazl);
while($wiersz = mysql_fetch_array($wykonaj))
{
$bron = $wiersz['bron'];
$tarcza = $wiersz['tarcza'];
$czar = $wiersz['czar'];
$nbron = $wiersz['nbron'];
$ntarcza = $wiersz['ntarcza'];
$nczar = $wiersz['nczar'];
$zloto = $wiersz['zloto'];
$profesja = $wiersz['profesja'];
$lvl = $wiersz['lvl'];
$xy = $wiersz['xy'];
}
 
if ($profesja != 'Mag') { echo '<b><center>Tylko dla magów</b></center>'; } else {
 
echo 'Kupując broń bez sprzedania starej, stracisz takową nie dostając za nią nic!<br>_____________________________________<br><br>';
 
 
if ($xy == 1 || $xy == 3) {
 
if ($nbron != 'Drewniana różdżka') {
 
if (!empty($_POST['drewnianarozdzka'])) {
if ($zloto >= 200) {
mysql_query("UPDATE users SET bron = 5 WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET nbron = 'Drewniana różdżka' WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET zloto = $zloto-200 WHERE login = '". $_SESSION['user'] ."'");
echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
} else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
} else {
echo '
<form action="wkostury.php" method="POST">
<INPUT TYPE="hidden" name="drewnianarozdzka" value="1">
DREWNIANA RÓŻDŻKA<br>200zł, ATAK: 5 <input type="image" src="buttons/kup.png" border="0">
</form>';
}
}
}
 
if ($nbron != 'Różdżka') {
if (!empty($_POST['rozdzka'])) {
if ($zloto >= 500) {
mysql_query("UPDATE users SET bron = 13 WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET nbron = 'Różdżka' WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET zloto = $zloto-500 WHERE login = '". $_SESSION['user'] ."'");
echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
} else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
} else {
echo '
<form action="wkostury.php" method="POST">
<INPUT TYPE="hidden" name="rozdzka" value="1">
RÓŻDŻKA<br>500zł, ATAK: 13 <input type="image" src="buttons/kup.png" border="0">
</form>';
}
}
}
 
if ($nbron != 'Kościana różdżka') {
 
if ($lvl >= 4) {
 
if (!empty($_POST['koscianarozdzka'])) {
if ($zloto >= 2000) {
mysql_query("UPDATE users SET bron = 30 WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET nbron = 'Kościana różdżka' WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET zloto = $zloto-2000 WHERE login = '". $_SESSION['user'] ."'");
echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
} else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
} else {
echo '
<form action="wkostury.php" method="POST">
<INPUT TYPE="hidden" name="koscianarozdzka" value="1">
KOŚCIANA RÓŻDŻKA<br>2000zł, ATAK: 30 <input type="image" src="buttons/kup.png" border="0">
</form>';
}
}
}
 
 
if ($nbron != 'Różdżka Ildara') {
if ($lvl >= 7) {
 
if (!empty($_POST['rozdzkaildara'])) {
if ($zloto >= 4500) {
mysql_query("UPDATE users SET bron = 80 WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET nbron = 'Różdżka Ildara' WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET zloto = $zloto-4500 WHERE login = '". $_SESSION['user'] ."'");
echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
} else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
} else {
echo '
<form action="wkostury.php" method="POST">
<INPUT TYPE="hidden" name="rozdzkaildara" value="1">
RÓŻDŻKA ILDARA<br>4500zł, ATAK: 80 <input type="image" src="buttons/kup.png" border="0">
</form>';
}
}
}
 
if ($nbron != 'Eukanos') {
if ($lvl >= 10) {
 
if (!empty($_POST['eukanos'])) {
if ($zloto >= 7000) {
mysql_query("UPDATE users SET bron = 150 WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET nbron = 'Eukanos' WHERE login = '". $_SESSION['user'] ."'");
mysql_query("UPDATE users SET zloto = $zloto-7000 WHERE login = '". $_SESSION['user'] ."'");
echo 'Zakupiono przedmiot!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">';
} else { echo 'Nie masz pieniędzy na zakup przedmiotu!<META HTTP-EQUIV="Refresh" CONTENT="1;URL=wkostury.php">'; }
} else {
echo '
<form action="wkostury.php" method="POST">
<INPUT TYPE="hidden" name="eukanos" value="1">
EUKANOS<br>7000zł, ATAK: 150 <input type="image" src="buttons/kup.png" border="0">
</form>';
}
}
}
 
 
 
 
 
 
}
 
 
 
echo '<br><br>_____________________________________<br>';
 
} else { echo '<b>BŁĄD!'; }
 
include "tabelaright.php";
 
} else {
echo "Wróć i zaloguj się!";
}
 
?>
[PHP] pobierz, plaintext 

Proponuję zedytować kod, zrobić wcięcia i przestrzegać miłej konwencji nawiasów klamrowych typu

[PHP] pobierz, plaintext 
<?php
if()
{
}
?>
[PHP] pobierz, plaintext 

zamiast

[PHP] pobierz, plaintext 
<?php
if() {
}
?>
[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 
<?php
while ($wiersz = mysql_fetch_array($wykonaj))
{
  extract($wiersz);
}
?>
[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ł ) (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 
<?php
while ($wiersz = mysql_fetch_array($wykonaj))
?>
[PHP] pobierz, plaintext 

wystarczy tylko

[PHP] pobierz, plaintext 
<?php
$wiersz = mysql_fetch_array($wykonaj)
?>
[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 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.

Ten post edytował Hazel 16.08.2007, 18:57:06

od linii 140:

[PHP] pobierz, plaintext 
<?php
} // to zamyka warunek z 7 linii: if ($_SESSION['user']) {
 
 
 
echo '<br><br>_____________________________________<br>';
 
} else { echo '<b>BŁĄD!'; }
 
include "tabelaright.php";
 
} else {
echo "Wróć i zaloguj się!";
}
?>
[PHP] pobierz, plaintext 

Tak jak napisał Hazel powinieneś się zaprzyjaźnić z Tabulatorem
Mnie pomaga też notepad++ który podświetla pary klamr - otwierającą i zamykającą

Wielkie dzięki za odpowiedź, że się tak rozpisaliście

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ć z góry dzięki