Forum PHP.pl

Czemu ten skrypt nie działa ?

Zamiast id=1 i id=2 mam podstawione $row['id'];

____________________________________________

[PHP] pobierz, plaintext 
<? include 'cfg.php';
 
$sql = mysql_query("SELECT * FROM forum_temat");
 
$show = (int)$_GET['show'];
$id = $_GET['id'];
 
switch($show):
case 1:
for($i=0; $i<$row=mysql_fetch_assoc($sql); $i++){
 
		if($row['dzial'] == pp){
				echo '<a href="index.php?id='.$row[id].'">'.$row['temat'].'</a>';
				echo "</br>";
		}
}
break;
case 2:
for($i=0; $i<$row=mysql_fetch_assoc($sql); $i++){
 
		if($row['dzial'] == dorg){
				echo '<a href="index.php?id='.$row[id].'">'.$row['temat'].'</a>';
				echo "</br>";
		}
}
break;
default:
 
echo '<a href="index.php?show=1">Pierwsza Pomoc</a>';
echo "</br>";
echo '<a href="index.php?show=2">Dyskusje o rozwoju Gry</a>';
 
break;
endswitch;
 
if($id == 1){
	echo 'test';
}
 
if($id == 2){
	echo 'test2';
}
?>
[PHP] pobierz, plaintext 

W instrukcji for($i=0; $i<$row=mysql_fetch_assoc($sql); $i++) porównujesz jabłka i krowy. Serio.
$i jest zmienną całkowitą, zmienna $row to jest wiersz w tablicy bazy danych przekazany jako tablica.

Porównujesz tablicę (array) z liczbą całkowitą.

Poczytaj (na sieci - tutoriali jest mnóstwo) o tym, co i w jakiej formie zwraca mysql_fetch_assoc, jesli nie będziesz wiedział, wypluj sobie to na ekran (pomocne funkcje: print_r, var_dump) i zobaczysz, o co kaman. To nawet nie przedszkole - to elementarz.

P.S.
skrypt php warto zaczynać od "<?php" - "<?" to skrócona postać, nie wszędzie obsługiwana.

abort

to działa bez zarzutu pobiera wszystkie zmienne jak while.

a dokładnie chodzi mi o to że nie pokazuje mi :

index.php?id=1 // index.php?id=2

a dokładnie zawartości tego.

to co napisałeś nie jest do końca prawdą. ale nie będę się sprzeczał.
Jak chcesz to zobacz :

http://www.piszemystrony.pl/1/forum/


Jbc.

TEST i TESTT

są rekordami z bazy danych

Ten post edytował sonquer 25.08.2012, 01:14:57

Z tego co widzę, w switchu przy każdym case odczytujesz z bazy do PHP całą tabelę po to, by znaleźć jeden wiersz. Zmień zapytanie do bazy na przykład na

[SQL] pobierz, plaintext 
SELECT * FROM forum_temat WHERE dzial = 'pp'
[SQL] pobierz, plaintext 

Jest wydajniej - w sumie po co obrabiać w PHP, skoro po coś wymyślono bazy danych?

Poza tym, masz mnóstwo błędów składni PHP - aż cud, że kod działa. Włącz sobie wyświetlanie tych błędów, np. dopisz na początku pliku:

[PHP] pobierz, plaintext 
error_reporting(E_ALL);
[PHP] pobierz, plaintext 

Jeśli już się uczysz pisać, to niech to będzie poprawne składniowo - jak się uczyć, to porządnie. Oprócz rzeczy, o których wcześniej wspomniałem, warto zqauwazyć:
1. if($row['dzial'] == pp) - co to jest to pp? Na zmienną mi nie wygląda (zmienne zaczynamy od "$"), na ciąg znaków też nie - te "owijamy" w apostrofy lub cudzysłowy
2. echo '<a href="index.php?id='.$row[id].'">'.$row['temat'].'</a>'; -- masz $row[id] i $row['temat'] - ciutke konsekwencji pliz

A co do Twojego pytania i problemu: a jak wygląda baza, co w niej masz?

if($row['dzial'] == pp)

: Jeżeli dzial jest równy pp pokaż zawartość.

tak wygląda db :

[SQL] pobierz, plaintext 
CREATE TABLE `forum_temat` (
`id` INT(11) NOT NULL AUTO_INCREMENT ,
`dzial` VARCHAR(32) NOT NULL ,
`temat` VARCHAR(32) NOT NULL ,
`autor` VARCHAR(32) NOT NULL ,
`tresc` VARCHAR(10000) NOT NULL ,
PRIMARY KEY (`id`)
);
[SQL] pobierz, plaintext 

Ten post edytował sonquer 25.08.2012, 01:31:34

Już pisałem. Taka konstrukcja powoduje, że PHP nie wie, czy porównujesz do zmiennej, czy do ciagu znaków. W związku z tym nie pisz:

[PHP] pobierz, plaintext 
if($row['dzial'] == pp)
[PHP] pobierz, plaintext 

bo PHP nie wie, jak to zinterpretować. Pomóż mu i zastosuj jedną z dwóch możliwości:

[PHP] pobierz, plaintext 
// jeśli porównujesz z zawartością zmiennej
if($row['dzial'] == $pp)
 
// jeśli porównujesz z ciągiem znaków określonym w kodzie.
// możesz zastosować apostrofy zamiast cudzysłowów, zadziała identycznie
if($row['dzial'] == "pp")
[PHP] pobierz, plaintext 

Poza tym dodaj sobie na początku kodu

[PHP] pobierz, plaintext 
    error_reporting(E_ALL);
    ini_set('display_errors','On');    // tego ostatnio zapomniałem
[PHP] pobierz, plaintext 

I pousuwaj wszelkie błędy, które Ci się wyświetlą.

Ten post edytował abort 26.08.2012, 22:23:41