PHP interesse |
|
normaal zou ik dit zo programmeren:
<?
if(isset($_GET["action"])) {
// eerst es reguliere expressie op de waarde toepassen zodat er geen "rare" tekens in je script kunnen sluipen(lees: hacks) -> heb ik er nu niet bijgestoken, anders lees je de tut over reg expressie maar eens
switch($_GET["action"]) {
case "reg":
// ier komt je code te staan wat er nu moet gebeuren als action=reg
break;
case "active":
// ier komt je code te staan wat er nu moet gebeuren als action=active
break;
// enzoverder
} // einde switch
} else {
echo "Geen $_GET['action']";
}
?>
<? if(isset($_GET["action"])) { // eerst es reguliere expressie op de waarde toepassen zodat er geen "rare" tekens in je script kunnen sluipen(lees: hacks) -> heb ik er nu niet bijgestoken, anders lees je de tut over reg expressie maar eens switch($_GET["action"]) { case "reg": // ier komt je code te staan wat er nu moet gebeuren als action=reg break; case "active": // ier komt je code te staan wat er nu moet gebeuren als action=active break; // enzoverder } // einde switch } else { echo "Geen $_GET['action']"; } ?>
Probeer nooit niet teveel if-elseif enzo na elkaar te gebruiken. Een switch is gewoon stuk sneller en overzichtelijker.
Sneller doordat als je waarde gelijk is aan case "1", de andere code(binnen de swicth) niet meer gelezen wordt.
Overzichtelijker omdat je weet dat je de waarde van dezelfde variable controleert. Bij if-elseif is dit niet noodzakelijk.
Jouw fout is dus eigelijk: Je wil een waarde gebruiken die niet bestaat. En da gaat niet é
Dus altijd eerst controleren of die wel bestaat...
Greetz,
Thomas |