utilizzo "return" e concatenazione cicli if

[sitodue]

Ciao a tutti;
Premesso che pare funzionare allo stesso modo in entrambe le soluzioni, mi chiedevo quale fosse la maniera più corretta di controllare delle condizioni nei cicli ed eventualmente reindirizzare...
Faccio un esempio banale in pseudocodice:

if [$nomeutente != corretto]  {
    echo "nome utente errato";
    header("Refresh -> homepage");
    return;
}

if [$password != corretta]  {
    echo "password errata";
    header("Refresh -> homepage");
    return;
}
...altri if...

[corpo della pagina] 

Oppure

if [$nomeutente  corretto]  {
     if [$password = corretta]   {
           ...altri if...
                 [corpo della pagina]
     }else{
           echo "nome utente errato";
           header("Refresh -> homepage");
     }
}else{
     echo "password errata";
    header("Refresh -> homepage");
} 

C'è un motivo per cui l'uno o l'altro metodo è d evitare?

[badaze]

Il return serve in una funzione di solito con un valore.

Il codice che hai postato non puo' funzionare anche se scritto in pseudo codice in quanto non devono esserci "scritte" (echo ad esempio) prima del comando header.

Per me c'è una terza via; cioè come il tuo primo esempio ma che funziona come il secondo. Lo scopo è di poter aggiungere delle condizioni senza dover modificare il codice già scritto e "fermarsi" al primo errore riscontrato.
Di più, ti consiglio di fattorizzare quanto possibile il codice. Nel tuo esempio se tu dovessi cambiare il re-indirizzamento dello header dovresti fare tante modifiche quante condizioni.

$errorMessage = '';
if [$nomeutente != corretto]  {
   $errorMessage = "utente errato";
}

if ($errorMessage == "") {
 if [$password != corretta]  {
    $errorMessage =  "password errata";
 } 
}

if ($errorMessage == "") {
 if [$email != corretta]  {
    $errorMessage =  "email errata";
 } 
}

if ($errorMessage != "") {
  header("Refresh -> homepage + $errorMessage ");
} 

[sitodue]

Ciao e grazie per la risposta prima di tutto...
La 3a via che hai indicato mi sembra effettivamente la più corretta e pulita...
Però mi viene da chiederti, se avessi ad esempio 100 o più condizioni da verificare prima di poter proseguire, se già la prima non risulta verificata, a me non importa verificare le restanti 99... c'è un modo corretto in quel caso di spezzare il flusso della pagina e reindirizzare?
Per quanto riguarda il discorso dell'echo prima dell'header, se ne era parlato in un'altra discussione qualche tempo fa e ricordo che qualcuno confermò che funziona anche in questo modo perché dipende da una configurazione nel php.ini se non erro... tanto è vero che i moduli tipo accesso.php li ho sempre scritti in questo modo e funzionano al 100% (magari sbagliando concettualmente però!)

[badaze]

Volendo puoi fare cosi (per certi goto is evil).

$errorMessage = '';
if [$nomeutente != corretto]  {
   $errorMessage = "utente errato";
   goto fine_controllo;
}

if [$password != corretta]  {
    $errorMessage =  "password errata";
    goto fine_controllo;
}

if [$email != corretta]  {
    $errorMessage =  "email errata";
    goto fine_controllo;
} 


fine_controllo:
if ($errorMessage != "") {
  header("Refresh -> homepage + $errorMessage ");
} 

[las]

Premesso che penso sia molto una questione di stile di ciascuno (come dire 'de gustibus') comunque volendo 'risparmiare' righe di codice e avendo centinaia di controlli da fare io opterei per una soluzione di questo tipo:

$arrControllare = array("utente"   => array($_POST['user'],'admin'),"password" => array($_POST['pass'],'abcdf') );

function controlla($arrControllare){
    foreach($arrControllare AS $campo => $arrCoppieValori){
        if($arrCoppieValori[0]!=$arrCoppieValori[1]){
            header("Refresh -> homepage + $campo errato");
        }
    }
}

controlla($arrControllare); 

[sitodue]

Perfetto, grazie ad entrambi!