Problema con in_array()

[Simo990]

Ciao a tutti.

Sto cercando di implementare un sito dinamico, ma non avendolo mai fatto prima d'ora mi trovo un po' in difficoltà.

Allora, io ho 4 pagine:

• index.php
• chisiamo.html
• dovesiamo.html
• contatti.html

index.php è la struttura, mentre tutte le altre vengono incorporate nella struttura tramite include_once().

Lo script che uso è il seguente:

<?php
                $pages = array("chisiamo","dovesiamo","contatti");
                $pag = $_GET['p'];
            if( isset($pag) )
        {
            if( in_array($pag, $pages) )
            {
                include_once "$pag.html";
                        }
            else
            {
                show_news();
                   }
              }
?>

Funziona, ma il problema è che la funzione show_news() non viene mai interpretata... o meglio, solo nel caso in cui provo a modificare manualmente l'url... scrivendo tipo:

http: //www. miosito.it/?p=boh

..in tutti gli altri casi il contenuto della pagina rimane vuoto, cioè non viene incluso niente e non viene eseguita nessuna funzione.

Ho controllato la documentazione e ho letto che "in_array — Checks if a value exists in an array". Ma se l'array è vuoto, perché non passa oltre ed interpreta quindi la funzione show_news()?

[Simo990]

Ho capito dov'era il problema... ho sbagliato a posizionare l'else! Scusate per il disturbo

[oly1982]

il problema è un rigo prima

 $pag = $_GET['p']; 
            if( isset($pag) ) 
//etc etc 

Tu valorizzi/crei la variabile $pag (non importa con cosa... e che eventualmente $_GET['p'] non esite...)

Da quel momento in poi $pag esiste... ergo:
isset($pag) è una condizione sempre vera!

Fai così

$pages = array("chisiamo","dovesiamo","contatti");               
$pag = isset($_GET['p']) ? trim($_GET['p']) : '';
if( !empty($pag) AND in_array($pag, $pages))
    {
    include_once "$pag.html"; 
    }
else
    {
    show_news(); 
    } 

ho editato il post... avevo omesso una cosa...

[Simo990]

grazie preferisco usare la tua versione che è senz'altro più corretta rispetto a questa:

$pages = array("chisiamo","dovesiamo","contatti");
$pag = $_GET['p'];
if(isset($pag))
{
   if(in_array($pag, $pages))
   {
      include_once "$pag.html";
   }
}
else
{
   show_news();
} 

ciao!