Un if in php crea problemi nella decisione dei numeri

Qualcuno potrebbe dirmi perchè questo codice mi fa diventare page sempre 1 qualsiasi valore io metta?

if(empty($_GET['page']) || $_GET['page'] <= 0 || !is_nan($_GET['page'])) {
    $page = 1;
    } else {
    $page = $_GET['page'];
    } 

[javascripter]

Semplicemente perché all'operatore logico OR ( || ) basta che un singolo controllo sia vero per rendere vero tutto l'if. Per esempio:

if(true || false || false) { echo 'vero'; } 

Il problema principale è la funziona empty, se $_GET['page'] non è vuota l'if è reso vero e quindi il resto del controllo è inutile. Usa isset al suo posto.

Anche se nel tuo caso io avrei fatto:

$page = isset($_GET['page']) && is_numeric($_GET['page']) && $_GET['page'] > 0 ? $_GET['page'] : 1; 

Grazie! Funziona perfettamente!

p.s. devo imparare php più approfondito.... tu con una rifga fai tutto quello che io faccio con un if e un else :O

[javascripter]

Nessuno ti vieta di usare if/else; però è più pratico usare l'operatore ternario per operazioni così semplici.