Ne vidim da ti je bilo gde u kodu deklarisana promenjljiva $id a koristiš je pri upitu u bazu za izlistavanjem podforuma. Ako je promenjljiva neinicirana a tebi iz nekog razloga kao prva vrednost identifikacije podforuma u bazi stoji vrednost nula uvek ćeš kao rezultat upita dobiti vrednost prvo unetog infa o podforumu tj. prvi uneti zapis. Sve ovo naravno zavisi i od strukture tabele u koju se smeštaju podaci.

Kao drugo, imao bih ozbiljniju primedbu na tvoj pristup očitavanju podataka iz baze podataka Pošto očigledno pokušavaš da dizajniraš dobar forumski sistem jedan od koraka bi svakako trebala i da bude što je moguće veći stepen optimizacije saobraćaja tj. pokušaj da smanjiš potrebu za komunikacijom php-a i baze podataka. Pogledaj tvoj kod. Ti prvo učitavaš kompletan spisak kategorija iz baze (što je jedan upit) a zatim za svaku od tih kategorija radiš po jedan upit za iščitavanje podforuma koji pripadaju toj kategoriji. Zamisli da imaš stotinak podforuma. Prvi problem je što će ti kod uraditi proveru za podforumima čak i kada postoji samo kategorija a ne i njemu dodeljeni podforumi a ako drugo za svaku od kategorija ide po jedan upit. Zamisli sada pedesetak korisnika koji zahtevaju tvoju stranicu na jednom recimo osrednje posećenom sajtu. Čini mi se da ti je jasno da bi se veliki teret postavljao pred komunikaciju između MySQL-a i php-a.

Moja preporuka je da kada god možeš koristiš JOIN operacije. Na taj način bi sve potrebne podatke mogao da povučeš jednim jedinim upitom a s obzirom da je svaka obrada na serveru brža od uzastopne obrade na MySQL serveru trud će se isplatiti

Konkretno tvoj dupli upit bi mogao da se odradi na sledeći način pod uslovom da obe tabele (forum_podforumi i forumi_kategorije) imaju zajedničko polje kategorija koja je u tabeli forumi_kategorije primarni ključ a u tabeli forumi_podforumi pravi ili simulirani strani ključ. Tada bi jednim jedinim pozivom sledećeg SQL-a dobio apsolutno sve podatke u bazi koji su ti potrebni za uspešan ispis tvoje tabele. Ne bi ti trebale čak ni uzastopne while petelje

U prethodnom kodu vrednost podforumID je u stvari primarni ključ podforuma (pretpostavio sam da je u pitanju autoincrement polje tipa broj).
Na ovaj način si u jednom upitu dobio sve podatke o podforumima na koje su u odgovarajućim kolonama dodati i svi podaci o kategorijama i to još grupisane po kategoriji i poređane po rednom broju podforuma u rastućem redosledu.

Izvinjavam se ako sam preterao sa detaljima ali učinilo mi se da je ovo dobro mesto za ovakvu napomenu

Odličan post oko optimizacije i sl. Hvala puno.

Kod sam malo preuredio:



Evo kako izgleda struktura same baze:



Ja napisem u tpl: {$sta.forum_kategorije.naziv} i nista

I vidim da mi ima tri kategorije, tj prikazace mi tri kategorije (zelena bgcolor bez nzaiva) mislim da se to ne moze rjesiti na ovaj nacin. Zato sto forum_podforumi.kat = forum_kategorije.id ; Tako da ispade ako ima puno podforuma ima i kategorija. Jedno da se ne ponavljaju kategorije.

{foreach from=$forum item=sta}
<tr>
<td width="43%" height="23" bgcolor="#73B000">{$sta.forum_kategorije.naziv}</td>
</tr> {/foreach}

..... I kako cu citati kada ovako imam join ovaj.

-----------------------------

EDIT:

Kad dodjem ovo:



I vamo:



PRikaze tri kategorije, pa podforuma tri. I to ne u kategorijama, vec poslije. A trebalo bi dvije kategorije.

Ako sam dobro razumeo ono što si napisao tebi treba način da lepo i kulturno uradiš prikaz svih kategorija podforuma zajedno sa kategorijama kojima ti podforumi pripadaju. Drugim rečima način da se podforumi prikazuju na sledeći način:

Kategorija1
+Podforum 1
+Podforum 2
+Podforum 3
Kategorija 2
+Podforum 4
+Podforum 5
...

U tom slučaju prethodni JOIN izraz možeš da modifikuješ tako da se umesto vezivanja kategorija za podforume uradiš obrnutu stvar tj. da se vezuju podforumi za kategorije. Za tu potrebu možeš umesto LEFT JOIN-a da koristiš recimo RIGHT JOIN ali ti ja preporučujem da uradiš LEFT JOIN i to na sledeći način.

Kao što ćeš videti ovim izrazom za svaku kategoriju vezuješ sve njegove podforume a ne obrnuto što je u ovom slučaju logičnije. Jedina stvar koju ti ovde treba da uradiš jeste da uradiš ispisivanje podataka.

Još par stvari u vezi optimizacije

Kao prvo, ako sam dobro ukapirao čemu treba da ti služi polje tema u tom slučaju tvoja baza ne ispunjava uslove normalizovanosti tabela. Drugim rečima, ako ti tema polje služi da u njemu napišeš koja tema pripada datom podforumu (i to po jedna tema za jedan zapis) u tom slučaju, povrh svega, baza pati od anomalije ažuriranja i brisanja. Ako si hteo da napraviš bazu koja će biti sposobna da vodi računa o kategorijama, podforumima i temama trebalo je da polje sa temama koje pripadaju odgovarajućem podforumu odvojiš u posebnu tabelu. Onako laički rečeno problem je u tome što ako recimo obrišeš određeni podforum automatski gubiš info o njemu pripadajućim temama i sl. Isto tako nemoj nikada da u bazu skladištiš podatke do kojih se može doći analizom već postojećih podataka u njoj. Primera radi u tvojoj bazi skladištiš vrednost postova tj. u toj koloni skladištiš ukupan broj postova u nekom podforumu. Taj problem se zove redunsansa podataka tj. skladištenje viška nepotrebnih podataka jer si taj podatak mogao da dobiješ jednostavnim upitom tipa

Ovo je veoma jednostavan primer a kompletna stvar se može dodatano optimizovati kombinovanjem u okviru jednog jedinog složenijeg upita.

Važna napomena: Ova fora sa COUNT-om nije preporučljiva ako su ti tabele u bazi tipa InnoDB a baza povelika jer se brojanje vrši bukvalnim proveravanjem svakog pojedinačnog zapisa što znači da će ti aplikacija pokazivati strašan pad performansi rada

Kako da rešiš problem vezan za tabele? Tako što ćeš kao što rekoh da odvojiš teme u posebnu tabelu i kasnije kad ti zatreba da koristiš JOIN operaciju.

Dakle tabele trebaju da ti izgledaju ovako:

podforumi(id, naziv, opis, kat, slika)

kategorije(id, naziv)

teme(temaID, forumID)

To bi bilo to za danas Puno sreće u radu jer su počeci najuzbudljiviji

Prikaze mi dva podforuma, vjerovatno (sigurno) radi ovoga:

... ON forum_podforumi.kat=forum_kategorije.id ...

A treba tri da prikaze.

Nije problem u uslovu JOIN-a već u pogrešnom tipu JOIN-a za ovu priliku. Nisam uzeo u obzir za trenutak da mogu postojati kategorije bez podforuma. Rešenje bi trebalo da bude korišćenje INNER JOIN-a

Kad upišem:



Opet prikaže isto, kao gore. Dakle, dva podforuma.