Kai345: Neues Problem nach aufdröseln

Beitrag lesen

[latex]Mae  govannen![/latex]

if (isset ($_GET['Hauptmenue']))
   {
      if ($_GET['Hauptmenue'] !== null && strlen ($_GET['Hauptmenue'] > 0))
      {
         if (preg_match ("/^d{1,2}$/", $_GET['Hauptmenue']))
            $Hauptmenue = $_GET['Hauptmenue'];
         else
            $Hauptmenue = 0;
      }
      else
         $Hauptmenue = 0;
   }
   else
      $Hauptmenue = 0;

  
Der Vergleich ($\_GET['Hauptmenue'] !== null...) ist überflüssig, da dies bereits durch isset() getestet wird (Determine if a variable is set and is not NULL). Gleiches gilt für strlen, da ein leerer Wert so oder so spätestens am nächsten Schritt (preg\_match) scheitern wird.  
  
Ansonsten ist die gesamte Konstruktion extrem umständlich.  
  
~~~php
$Hauptmenue = 0;  
if (isset ($_GET['Hauptmenue']) && preg_match ("/^d{1,2}$/", $_GET['Hauptmenue']))  
{  
    $Hauptmenue = $_GET['Hauptmenue'];  
}

reicht vollkommen aus, wenn keine weiteren alternativen Zweige vorhanden sind (und ich jetzt keinen Denkfehler drin habe)

Stur lächeln und winken, Männer!
Kai

--
var jQuery = $(hit);
„Die Borg würden nicht mal Spaß verstehen, wenn sie einen Vergnügungspark assimiliert hätten!” (B'Elanna Torres)
SelfHTML-Forum-Stylesheet