Problema codice php

[nb_staff]

/* Recupero i dati */
$nome = $_POST['nome'];
$pass = $_POST['pass'];
$mail = $_POST['mail'];
$villo = $_POST['villaggio'];
$sex = $_POST['genere'];



/* 1] Verifico che tutti i campi SONO compilati */
if (isset($nome) && isset($pass) && isset($mail) && isset($villo) && isset($sex)) 
{
    /* 2] Verificio che l'e-mail sia valida */
        $pos = strpos($mail, "@");
        $pos2 = strpos($mail, " ");
            if ($pos === false || $pos2 != false) 
            {
                echo "La tua mail non è valida";
            }    
    /* 2] Verificio che la password sia min 4 caratteri e max 8 */
        $lungpass = strlen($pass);
            if ($lungpass < 4 || $lungpass >8 )
            {
                echo "</br>" . "La tua password non è valida";
            }
}

/* 1] Se tutti i campi NON SONO compilati */
else
{
    echo " Devi inserire tutti i campi per poter completare la registrazione ";
} 

Come mai , il codice php sovrastante mi da il seguente errore :
Se non inserisco la mail o la password , invece di dirmi = Devi inserire tutti i campi ecc ,
mi dice la tua mail non è valida , la tua password non è valida.

Dove sbaglio ?

[Laxus]

Attenzione, isset($var) non controlla se $var è vuota o meno (quello è empty) ma controlla se la variabile esiste o meno! In questo caso isset($pass) e simili restituiranno true in ogni caso

[nb_staff]

Grazie mille , con emprty funziona bene .
Scusa una cosa , ma se io volessi creare una variabile di avanzamento come potrei fare?

Mi spiego meglio , invece di annidare if , dentro if , dentro if , dentro else ecc ( anche se sto cercando di mantenermi il più ordinato possibile ) non sarebbe possibile fare una cosa del genere ?

<?php 

/* Recupero i dati */
$nome = $_POST['nome'];
$pass = $_POST['pass'];
$mail = $_POST['mail'];
$villo = $_POST['villaggio'];
$sex = $_POST['genere'];

/* Assegno variabile di avanzamento */
$ok = false;

/* 1] Verifico che tutti i campi SONO compilati */
if (empty($nome) && empty($pass) && empty($mail) && empty($villo) && empty($sex)) 
{
    $ok = true;
}

/* 1] Se tutti i campi NON SONO compilati */
else
{
    echo " Devi inserire tutti i campi per poter completare la registrazione ";
    $ok = false;
}

    if ( $ok === true ) 
    {
        /* 2] Verificio che l'e-mail sia valida */
        $pos = strpos($mail, "@");
        $pos2 = strpos($mail, " ");
            if ($pos === false || $pos2 != false) 
            {
                echo "La tua mail non è valida";
            }    
        /* 2] Verificio che la password sia min 4 caratteri e max 8 */
        $lungpass = strlen($pass);
            if ($lungpass < 4 || $lungpass >8 )
            {
                echo "</br>" . "La tua password non è valida";
            }
    }
    
    


?>

eppure non funziona.

[Luke70]

certo che non funziona!
empty fa il contrario di quello che ti aspetti: restituisce vero se il campo è vuoto!

if (empty($nome) || empty($pass) || empty($mail) || empty($villo) || empty($sex)) 
{
    echo " Devi inserire tutti i campi per poter completare la registrazione ";
    $ok = false;
} 

[jlocke2010]

Dovresti cambiare la condizione di && con || perchè in quel modo tu gli dici se tutti i campi sono vuoti allora vai avanti invece devi dirgli se anche una delle seguenti variabili è vuota blocca tutto altrimenti prosegui pure

<?php 

/* Recupero i dati */
$nome = $_POST['nome'];
$pass = $_POST['pass'];
$mail = $_POST['mail'];
$villo = $_POST['villaggio'];
$sex = $_POST['genere'];

/* Assegno variabile di avanzamento */
$ok = false;

/* 1] Verifico che tutti i campi SONO compilati */
if (empty($nome) || empty($pass) || empty($mail) || empty($villo) || empty($sex)) 
{
    echo " Devi inserire tutti i campi per poter completare la registrazione ";
    $ok = false;
}

/* 1] Se tutti i campi NON SONO compilati */
else
{
    $ok = true;
}

    if ( $ok === true ) 
    {
        /* 2] Verificio che l'e-mail sia valida */
        $pos = strpos($mail, "@");
        $pos2 = strpos($mail, " ");
            if ($pos === false || $pos2 != false) 
            {
                echo "La tua mail non è valida";
            }    
        /* 2] Verificio che la password sia min 4 caratteri e max 8 */
        $lungpass = strlen($pass);
            if ($lungpass < 4 || $lungpass >8 )
            {
                echo "</br>" . "La tua password non è valida";
            }
    }
    
    


?>

[Bukowski]

Non funziona perchè devi usare l'OR ovvero || e non l'and.

esempio

empty($nome) || empty($pass) || empty($mail) || empty($villo) || empty($sex)

[nb_staff]

Grazie mille , ho capito , io praticamente gli dicevo che se tutti sono vuoti andava bene xD
Grazie ancora per la velocità.

Scusate una delucidazione
all'interno dell'if mettendo $ok = true; o false , funziona alla grande
ma non ho ben capito - a questo punto - a cosa serve RETURN
infatti se metto return $ok= false o true , il codice non funge.


Potreste spiegarmi sto benedetto return a che serve u.u
Grazie per la pazienza ^^

[Laxus]

ma non ho ben capito - a questo punto - a cosa serve RETURN

mi sà che sono ciecato, ma non vedo nessun return nel codice che hai postato
return serve ad una funzione per restituire un valore; al di fuori di una funzione termina l'esecuzione della pagina

[nb_staff]

Nono , non ho inserito nessun return
ma , mettiamo il caso che nel mio codice metta

if (empty($nome) || empty($pass) || empty($mail) || empty($villo) || empty($sex)) 
{
    echo " Devi inserire tutti i campi per poter completare la registrazione ";
    return $ok = false;
}

/* 1] Se tutti i campi NON SONO compilati */
else
{
    return $ok = true;
} 

come mai non funziona più nulla u.u ?

[Luke70]

return interrompe l'esecuzione di una funzione e restiuisce un valore.
Messo così non serve a nulla.

Credo debba restituirti errore (ma forse non li visualizzi)