if elseif else koder

//kode 3
if (isset($_GET["side"]) && $_GET['side'] == '1') {
  // noget
}

//kode 4
if(isset($_GET["side"]) && $_GET['side'] == "hej") {
  echo "Dette er side 1";
}

(jeg har flere koder oppe i ærmet)

I PHP5 vil der kræves at du benytter isset, udover din select. Det er også den bedste måde.

Du bør gøre det sådan her:

<?php

if(isset($_GET["side"]) && $_GET["side"] == "hej")
{
  echo "Dette er side 1";
}
elseif(isset($_GET["side"]) && $_GET["side"] == "do")
{
  echo "Dette er side 2";
}
else
{
  echo "Dette er forsiden";
}

?>

Men en switch er ligeså god

<?php
if(isset($_GET['side']))
{
  $side = $_GET['side'];
  switch($side)
  {
    case "hej":
    echo "Dette er side 1";
    break;

    case "do":
    echo "Dette er side 2";
    break;
  }
}
else
{
  echo "Dette er forsiden";
}
?>

claus_joergensen, din switch kan laves smartere:

<?php
switch($_GET['side'])
      {
    case 'hej':
    echo 'Dette er side 1';
    break;

      case 'do':
      echo 'Dette er side 2';
      break;
       
    default:
    echo 'Dette er forsiden';       
    }
?>

wicez

Der skal alligevel en isset uden om, + at så vil "Dette er forsiden" ikke komme medmindre ?side faktisk ER sat :)

Her er den kommenteret:

<?php
switch($_GET['side'])
    {
    //i stedet for if($_GET['side'] == 'hej')
    case 'hej':
    echo 'Dette er side 1';
    break;
    //I stedet for elseif($_GET['side'] == 'do')
    case 'do':
    echo 'Dette er side 2';
    break;
    //I stedet for else       
    default:
    echo 'Dette er forsiden';       
    }
?>

Hvis side ikke er sat, eller er lig andet end "hej" og "do", vil "Dette er forsiden" blive udskrevet, hvilket vel også er hensigten.

Hvis vi skal være "smarte", så kan vi jo godt:

<?php
$side=isset($_GET['side'])?$_GET['side']:"";
switch($side) {
    case 'hej':
      echo 'Dette er side 1';
      break;

    case 'do':
      echo 'Dette er side 2';
      break;
     
    default:
      echo 'Dette er forsiden';     
    }
?>

wizez: du kommer ikke udenom at teste med isset

hmm, ja det kan du godt have ret i.

PHP er også underligt at det kan køre switch på Strings, Java kan kun på Integers.

1. Jeg tror snart jeg skifter brugernavn, da ingen staver det rigtigt ;'(

2. erikjacobsen er du sikker på at den ikke vil tage "Default", hvis $_GET['side'] ikke er sat?

2. Det er inderligt ligegyldigt. Du skal teste med isset, da du ellers vil få en advarsel fra PHP, hvis den ellers er konfigureret som den bør.

Det skulle måske nævnes at $side=isset($_GET['side'])?$_GET['side']:""; er ternary operatoren, som jeg aldrig kan huske syntaksen til :D

http://se.php.net/manual/en/language.operators.php

oki. Så tror jeg, at jeg vil holde min mund i resten af denne diskussion :o)

http://se2.php.net/manual/en/language.operators.comparison.php#language.operators.comparison.ternary

Var det rigtige link.

<ole>

erikjacobsen >> Kunne du ikke beskrive din special-opsætning af PHP? Den fejl har jeg endnu til gode - og jeg kan ikke fremprovokere den i min opsætning af min lokale Apache/PHP med denne kode:

#$bla = "gylp";
switch ($bla) {
    case "gylp":
        print "Bræk";
    break;
   
    default:
        print "Nøh ...";
}

/mvh
</bole>

claus_joergensen: Det kan jeg heller ikke, derfor holder jeg mig fra dem.
Måske var det på tide at få sat sig ind i disse :o)

olebole

Der menes at et switch direkte på $_GET['side'] vil retunere i en fejl ("Undefined index  'side' in filnavn.php") , og der derfor skal testes med isset først.

PHP4 giver sjældent denne fejl, hvor PHP5 er mere striks.

Så er det derfor jeg ikke helt kunne forstå det. Jeg kører med PHP4 :o)

- klart ... min lokale er også en 4'er  ;o)

Det er ikke en specialopsætning. Det er den rigtige opsætning. Det er nu rart at
få at vide, når der er noget galt. Visse mennesker, vi nævner ingen navne, vel Ole,
vil hellere gemme hovedet under busken, end at få at vide de har lavet en fejl. ik' ;)

Får man fejlrapporter som i "rigtige" sprog som Java, C# etc, så kan man altså
nemmere lave korrekte programmer.

Tror du på autoriterne, Ole, så prøv med en
    error_reporting(E_ALL);
øverst i dit script. Er du en kylling så
    error_reporting(0);

Ikke, at jeg er i tvivl om, man altid bør checke, om variabler eksisterer - og at de indeholder noget forventet - før man bruger dem.
Jeg har bare aldrig set en fejl af lige den årsag ... og kunne af gode grunde ikke fremkalde den, da jeg testede  :)

Og autoriteterne kan ikke engang stave til "autoriteterne".... suk

Skal vi være hel hip, så kunne vi jo smide empty() www.php.net/empty på istedet for isset()

Den fungere på sammen måde, men checker også om ?side=  er blank, eller indeholder en værdi. Det kan tit spare en for et ekstra check.

Er error_reporting sat til 'E_ALL' i default-opsætningen af PHP?

jeg kunne ikke være mere enig claus_joergensen. Jeg har aldrig været ude for, at man ikke har kunnet bruge isset i stedet for empty, mens jeg ofte er ude for det modsatte.

Det er ikke nær det samme at bruge empty i stedet for isset. Pas på med det. empty er et misfoster, der stort set aldrig har en korrekt anvendelse.

Men hvorfor bruge en midlertidig variabel? Hvorfor ikke bare:

<?php
switch(isset($_GET['side']) ? $_GET['side'] : "") {
    case 'hej':
      echo 'Dette er side 1';
      break;

    case 'do':
      echo 'Dette er side 2';
      break;
   
    default:
      echo 'Dette er forsiden';   
    }
?>

Ingen grund til at lave flere variabler end højst nødvendigt. :)

--
Morten Barklund - der også stemmer for error_reporting(E_ALL)

http://www.blueshoes.org/en/developer/php_cheat_sheet/ er intersassant.

barklund

Pænere kode ?-)
Og hvis man , som jeg i nogle tilfælde, skal bruge værdien af variablen længere nede i scriptet :)

Ellers har du skam ret.

default-opsætning? Tjah, hvem ved. Men det er igen ligegyldigt. Vil du lave PHP-scripts, der skal kunne køre på flere installationer skal du regne med den er sat "hårdest muligt".

Men hvis man bare leger med lidt scripting er alt det vi diskuterer bare pjat. Så skal man have det sjovt og nemt. Spørgeren ville blot have den bedste måde....

Jeg tror, du har kikket for dybt i hvidvinen - og blander spørgsmålet sammen med noget, du tror, du har læst. Spørgeren spurgte om brugen af isset. Det er der vist ingen tvivl om. Jeg spurgte om noget helt andet  :)

Og jeg svarer bare som vinden blæser ;) Jeg har nu aldrig forstået hvorfor man skulle kikke på hvidvin?

Nå.
claus_joergensen fik svaret på fejlen i kode 1.
Men hvilken en af de 2 jeg skrev er den bedste / mest sikker at bruge?

Altså, om det er mest sikkert at bruge tal eller navne til at identificere siderne? Det er fuldstændig ligegyldigt. Med mindre du selvfølgelig ikke ønsker, at man skal kunne gætte sig til andre sidenavne - for så er tal nemmere at gætte på end længere strenge.

Det var ikke det jeg tænkte på.
Men dette:
//kode1
if(isset($_GET["side"]) && $_GET["side"] == "1") {
echo "Dette er side 1";
//kode2
if ($_GET['side'] == '1') {
echo "Dette er side 1";

Om du bruger isset eller ej? Det har intet med sikkerhed at gøre, blot "kode-renlighed". At bruge isset er det rigtigste, og gør man ikke, vil man med strikse indstillinger få en fejl - men det er nok ikke tilfældet for dig.

Jeg vil råde dig til at bruge isset, det vil i hvert fald ikke give problemer på nogen måde - det er blot pænere og rigtigere end at udelade det.

claus_joergensen vil du osse ha din del af points?

Tark for points...