if $_GET - ikke er

$cat = isset($_GET['cat']) ? (int)$_GET['cat'] : 0;

if(!($cat >= 1 && $cat <= 8)){
  header("Location...");
  exit();   
}

Sådan?

$cat = $_GET['cat'];  // Udelukkende for at skrive kortere kode om lidt

if ($cat != 1 && $cat != 2 && $cat != 3 && $cat != 4 && $cat != 5 && $cat != 6 && $cat != 7 && $cat != 8)
{
    header("Location.............

Det er ikke 100 % klar om $cat altid er et heltal.

Tak skal i have.. ligger i svar?

Nielle: hmm, skal det så om noget ikke være "OR" istedet for "&&" ?

... noget ikke være "OR" istedet for "&&" ?>

Nej.

$cat skal være forskellig fra 1 OG den skal være forskellig fra 2 OG forskellig fra 3 ...

Hvis du bruger OR ("||") så vil din if altid blive opfattet som sand, da alting opfylder at det enten er forskellig fra 1 eller forskellig fra 2 (og så behøver vi sådan set slet ikke kigge på 2, 3, ...).

$cat = !empty( $_GET['cat'] ) : $_GET['cat'] : 0;
if ( !in_array( $cat, range(1, 8) ) )
{
  header("Location...");
  exit();   
}

? :)

$cat = (int) $_GET['cat'];

if( !$cat OR $cat > 8 )
    header( 'Location...' ) or die();

Den løsning fejler hvis $cat er negativ, eller ikke er et heltal.

Plus der kommer en notice hvis $_GET['cat'] ikke er sat ;P

$cat = ( $_GET['cat'] ) ? (int) $_GET['cat'] : 0;

if( !$cat OR $cat > 8 )
    header( 'Location...' ) or die();

Nej, denne kode virker fint:

$cat = (int) $_GET['cat'];

if( !$cat OR $cat > 8 )
    header( 'Location...' ) or die();

Den vil stadig generere en notice ;P

Jeg tror bare at smitten1 og nielle skal smide et svar ;)

Der blev spurgt specifikt til tallene 1, 2, 3, 4, 5, 6, 7 og 8. Derfor er det i princippet forkert hvis 5,4 afrundes og dermed behandles som om at der havde stået 5.

Desuden vil en negativ (heltal eller ej) værdi stadig blive behandlet forkert.

Et svar fra mig

Man er stadig nødt til at bruge isset eller empty for ikke at generere et notice:

$cat = !empty( $_GET['cat'] ) : $_GET['cat'] : 0;

F.eks. - Ingen point til mig ;)

Takker ;)