Sitat av
Alex!
Trådstarter, vær så snill, slå på feilmeldinger og varsler når du prøver å lage noe, for det du har skrevet så langt vil komme med varsler i hytt og pine.
I tillegg har vi følgjande juvel:
Kode
$skoleIsSet = $_GET['skoleId'];
$eventIsSet = $_GET['event'];
if($skoleIsSet && !$eventIsSet) {
Ettersom det er snakk om ID kan vi vel gjette på int e.l.
PHP.net skriv følgjande om kva som vil verte evaluert som false når ein caster til bool:
the boolean FALSE itself
the integer 0 (zero)
the float 0.0 (zero)
the empty string, and the string "0"
an array with zero elements
the special type NULL (including unset variables)
SimpleXML objects created from empty tags
Det er ikkje det trådstarter vil. Det trådstarter vil er å sjekke om variabelen $_GET['skoleId'] er satt. Uttrykket bør skrivast om til if (isset($_GET[...] && isset($_GET[...])
Viare så ser det ut til at data vert henta frå database - også filtrert i PHP! Kvifor ikkje bruke ei spørring som filtrerer (SELECT `foo`,`bar`... WHERE `skoleid` = $id? Det er antakeleg kjappere enn å loope gjennom alt. Uviktig om ein har ti ting i databasen, essensielt når ein har ein halv million.