isset($_GET) problemer.

I tænker allesammen forkert ... bortset fra Nikolaj. Der har valgt at lave noget helt anden kode.

 if (isset($_GET['side']) != 'test'): // slettes
 if (isset($_GET['side']) && $_GET['side'] == 'test'): // indsættes i stedet
 
 
 if (isset($_GET['side']) != 'kontakt'): // slettes
 if (isset($_GET['side']) && $_GET['side'] == 'kontakt'): // indsættes i stedet

Du har bare lavet den fejl at test på om isset() er en string og isset() er altid en boolean ... Du skal huske at teste på om variablen er sat OG om den er lig en ønsket værdi.

Som Daniel Mautone fint pointere, er der problemer med dine logiske operatorer.

!= betyder IKKE lig med.
== betyder lig med.

Af samme årsag oplever du fejl i dit script, da du har bedt dit script udføre noget andet, end hvad du havde i tankerne.

Hvis du beder du script om, at tilføje klassen "current" på de sider, der ikke er lig dine strenge i dine kontrol-statements (if, elseif, else), er det klart, at de får klassen "current", når ikke du er på den side, som du har angivet i din streng.

Som dit script er nu, sker følgende:

Hvis $_GET key'en "side" ikke er sat, skal "forside" være current.
Hvis $_GET key'en "side" IKKE er lig "test", skal "test" have klassen current.
Hvis $_GET key'en "side" IKKE er lig "kontakt", skal "kontakt" have klassen current.

Hvad du vil ende op med, vil være omvendt - ihvertfald for de to punkter, der kræver en hvis værdi:

Hvis $_GET key'en "side" ER LIG "test", skal "test" have klassen current.
Eller hvis $_GET key'en "side" ER LIG "kontakt", skal "kontakt" have klassen current.

Du kunne også gøre brug af et switch statement.
Du kan eventuelt tage udgangspunkt i følgende eksempel:

 <?php
 $current = '';
 switch($_GET['side']) 
 {
     case 'test' :
         $current = 'test';
         break;
     case 'kontakt' :
         $current = 'kontakt';
         break;
     default:
         $current = 'forside';
         break;
 }
 echo ($current == 'test') ? 'class="current"' : '';
 ?>
 
 <li>
   <a href="/" <?php echo ($current == 'forside') ? 'class="current"' : ''; ?>>Forside</a>
 </li>
 <li>
   <a href="?side=test" <?php echo ($current == 'test') ? 'class="current"' : ''; ?>>Test</a>
 </li>
 <li>
   <a href="?side=kontakt" <?php echo ($current == 'kontakt') ? 'class="current"' : ''; ?>>Kontakt</a>
 </li>

Hvis du nu lige retter dine if sætninger så der stå følgende:

 if(isset($_GET['side']) == 'kontakt')

Lige så gælder det for test.

Den du har nu siger at hvis det ikke er lig med kontakt eller test så gør dette.
Den jeg har smidt siger hvis lig med kontakt så gør dette

PHP er ikke så meget efter om man benytter isset i $_GET ... PHP er efter om man prøver, at referere et ikke defineret index.

f.eks. $_GET['hejmeddig'], hvis 'hejmeddig' ikke er et index i $_GET vil du få en fejl. Derfor skal du først teste om 'hejmeddig' er sat: isset($_GET['hejmeddig']) og hvis den er sat kan du så bruge den.

Hvad fejl sker der?

jeg har nævner ikke noget om, at du skal sammensætte noget. Du skal bare erstatte 2 linier i dit originale kode.

At test og kontakt bliver sat med current når man ikke er på forsiden.

Altså test og kontakt samtidig.

Hvordan vil du så sammensætte de to du snakker om dan ?



Det virker også fint nok med:

 <?php
     $current = '';
     switch($_GET['side'])
     {
     case 'test' :
     $current = 'test';
     break;
     case 'kontakt' :
     $current = 'kontakt';
     break;
     default:
     $current = 'forside';
     break;
     }
     ?>

Dog forlanger den Undefined altså isset, og så virker det ikke med kontakt, for så vil kontakt ikke få current

Argh, ja det virker.
Og mange tak for jeres hjælp.

Kan i forklare mig hvorfor php er så meget efter om man benytter isset i $_GET ?

Tak