If ed header inesistenti

[Kroenen91]

Quest'anno ho iniziato a studiare php e mi è stato chiesto ieri di simulare un'iscrizione, un login e un logout con l'utilizzo delle sessioni ma per qualche strano motivo gli if del codice non vengono visti, tranne l'ultimo, cioè il controllo sul recordset della query.

<?php     
            session_start();     
            $user=trim($_POST["user"]);     
            if($user=="" )     
            {         
                  header("Location: Login_form.php?errorcode=2");         
                }        
             $password=trim($_POST["pass"]);     
             if($password=="")     
             {         
                 header("Location: Login_form.php?errorcode=3");     
              }        
               $conn=mysql_connect("localhost","root","")or die("Connessione non effettuata");         
               mysql_select_db("Utenti");     
               $query="SELECT nome FROM utenti where user= ".$user ." AND password=" .$password ."";     
               if(!$ris=mysql_query($query))     
               {         
                           header("Location: Login_form.php?errorcode=1");         
                   }     
                $_SESSION["user"]=$user; ?>

[m4rko80]

Ciao, hai provato a cambiare i controlli?

            if( isset($_POST["user"]) && $_POST["user"]=='' || !isset($_POST["user"]) )     
            {         
                  header("Location: Login_form.php?errorcode=2");         
                }   
....

[piero.mac]

A meno che $_POST debba ricevere il valore 0 conviene usare empty() al posto di isset(). Poi si deve controllare "prima" di passarla alla variabile e non dopo.

<?php     
            session_start();

            if(!empty($_POST["user"])) {
            $user = trim($_POST["user"]);     
            }  else {         
                       header("Location: Login_form.php?errorcode=2");
                       exit;         
                      }

C'e' poi un errore di valutazione nel controllo del risultato della query:

if(!$ris=mysql_query($query)) 

sara' false solo in caso di errore della query, ma non lo sara' in caso di nessun record trovato. Nessun record non e' un errore e si controlla contando il numero delle righe estratte perche' $ris conterra' comunque un resource id# in caso di query valida.

[Kroenen91]

Grazie ho risolto con un'altra soluzione, ma proverò le vostre soluzioni così da imparare metodi più efficenti, grazie anche del consiglio sul controllo della query , hai pienamente ragione ma sono alle prime armi ancora non conosco il php a un livello base... grz ancora se mi dite come chiudere la discussione mi fate un favore XD