er dette scriptet greit?

[ragnic]

Har laget en webside med 3 kolonner i bredden, menyer i kolonnene til venstre og høyre, når bruker klikker på en av linkene i menyene, lastes den aktuelle siden i midtkolonnen. Dette har jeg gjort ved hjelp av divs som er posisjonert med CSS, selve kodingen for å få innholdet i rett div er gjort med php. Er ganske fersk på dette så jeg lurte om noen kloke hoder kunne se på det jeg har gjort, det virker på maskinen min(har satt den opp med apache), men er det noe her som kunne vært gjort bedre/annerledes før jeg laster den opp.:

har altså en liste med linker:

<li><a href="index.php?id=1">LINK1</a></li>

<li><a href="index.php?id=2">LINK2</a></li>

<li><a href="index.php?id=3">LINK3</a></li>

 

Skriptet som sørger for at innholdet kommer på rett plass:

(ligger i div'en som utgjør midtkolonnen)

<?php

$test=$_GET['id'];

if(empty($test)){

include('start.html');}

switch($test){

case '1':

include('side1.html');

break;

O.S.v...

?>

lurer spesielt på:

index.php som linkene peker til er navnet på hovedsiden med alle div's, de peker altså til seg selv(for å unngå å få opp et nytt vindu), er dette greit, eller er det noen potensielle feller her?

for å få variabelen når bruker trykker en link bruker jeg GET, er dette ok?

TAKK

[kondyj]

<li><a href="index.php?id=1">LINK1</a></li>

<li><a href="index.php?id=2">LINK2</a></li>

<li><a href="index.php?id=3">LINK3</a></li>

 

Skriptet som sørger for at innholdet kommer på rett plass:

(ligger i div'en som utgjør midtkolonnen)

<?php

$test=$_GET['id'];

if(empty($test)){

include('start.html');}

switch($test){

case '1':

include('side1.html');

break;

O.S.v...

?>

 

6842992[/snapback]

 

Kan nok ikke svare på alle spørsmålene dine, men jeg ville gjort om scriptet til å laste start.html hvis id er tom _og_ når det kommer en variabel som du ikke har en side for. Hva skjer hvis index.php?id=heioghopp blir lastet ?

 

Ellers så ville jeg bruke readfile() og ikke include() på statisk innhold.

[Martin A.]

<?php
$fil = $_GET['id'];

switch($fil) {
 case 'nyheter':
   $ink = 'nyheter.php';
   break;
 
 case 'galleri':
   $ink = 'galleri.php';
   break;
 
 default:
   $ink = 'start.php';
}

if(file_exists($ink) {
 include($ink);
}else {
 include('404.php');
}

 

Istede for å sjekke om $id er tom, og inkluderer noe, bruker jeg "default:" i switchen istede.

Endret 12. september 2006 av M4rTiN

[ragnic]

takker for innspill