aiuto codice registrazione

aiuto codice registrazione postato il 30/03/2009 21:45:20 nel forum programmazione, gdrcd e open source

Alura il problema è semplice ho provato a fare una semplice pagina di registrazione ma mi sono sorti due problemi,quando NON inserisco nome cognome ed email mi da errore ( e fin qui è giusto),quando però inserisco il nome e cognome e lascio il campo dell'email vuoto non mi da nessun errore ma mi stampa i dati normalmente e mi invia i dati al db,se però inserisco un email non valida ad esempio metto due @@ mi da errore.In pratica non mi fa il controllo se il campo dell'email è vuoto.
Poi quando inserisco un nome già presente nel db (ad esempio prova) e un cognome già presente (ad esempio super) mi da errore (cosa buona e giusta u.u) ma se inserisco un nome già presente nel db(ad esempio prova) e un cognome che non è presente (ad esempio x) non mi da l'errore ma mi stampa i dati normalmente,lo stesso succede in modo inverso.
Posto qui il codice.Oltre a questi due problemi accetto consigli per migliorare il codice XD.

<?
//ci si connette al db
$connection = mysql_connect("localhost", "xxxxxx", "xxxxxx")
or die ("Errore nella connessione. ");
$db=mysql_select_db('my_xxxxxx',$connection)
or die("errore nella connessione");

// recuperiamo i dati dei form
$nome = htmlentities( $_POST['input']);
$cognome = htmlentities($_POST['cogno']);
$email=htmlentities($_POST['email']);

//variabile per controllare se l'email è valida
$num_at = count(explode( '@', $email )) - 1;
//controllare se i campi sono vuoti

if(trim($nome) == "" || trim($cognome) == "" || trim($email) == ""){ echo " completare i campi";}
//controllare se l'email è valida
elseif(trim($email) == ""){ echo " inserire un email valida" ;}
elseif(!ereg("@",$email) || $num_at != 1){ echo " inserire un email valida" ;};

//controllare che la pass non sia già presente
$query="SELECT * FROM personaggi WHERE cognome='$cognome' AND nome='$nome' ";
$result=mysql_query($query)
or die ("query fallita:" . mysql_error());
$row=mysql_fetch_array($result);
//recuperiamo i dati
$nom=$row['nome'];
$cognom=$row['cognome'];
if ($cognom == $cognome AND $nom == $nome) {echo " cognome o nome già utilizzato";}

elseif($cognome == "" ){echo " inserire cognome ";}elseif
($nome == ""){echo " inserire nome ";}else
{ echo "Dati: nome: "
. $nome .
" cognome: "
.$cognome ;

$query="INSERT INTO personaggi(nome,cognome) VALUES('$nome','$cognome')";
$result=mysql_query($query)
or die( "query fallita:" . mysql_error());

mysql_close($connection);

}

?>

oorazoroo 20.58
l´unico pc sicuro è un pc spento

mook 14.12
beh errare è umano perseverare è da ban

Risposte alla discussione

Pagine: 1

blancks

30/03/2009 21:59:29

Ad una rapida analisi non termini l'esecuzione dello script quando c'è un errore, prova con die(); o exit(); al posto di echo per stampare il messaggio d'errore

Cuisine Royale
Un Battle Royale unico, nato come pesce d'aprile, in cui combattere contro altri 30 giocatori armati di utensili da cucina!

jed,vitius

30/03/2009 22:15:01

Grazie blancks facendo come hai detto ho risolto il primo problema😃.
Però il secondo rimane.

oorazoroo 20.58
l´unico pc sicuro è un pc spento

mook 14.12
beh errare è umano perseverare è da ban

Modificato il 30/03/2009 22:29:54 da jed,vitius

blancks

30/03/2009 22:19:17

//controllare che la pass non sia già presente

$query="SELECT * FROM personaggi WHERE cognome='$cognome' OR nome='$nome' ";
$result=mysql_query($query) or die ("query fallita:" . mysql_error());

if ($row=mysql_fetch_array($result))
die("cognome o nome già utilizzato");

In primis va sostituita la clausola AND con OR nella query, in secondo se ci sono risultati significa che sul db sono presenti e quindi va terminata l'esecuzione.

Travian
Travian è il più famoso browsergames al mondo. Come giocatore di Travian, costruirai il tuo impero, recluterai un potente esercito!

jed,vitius

30/03/2009 22:30:39

Grazie blancks ora funziona tutto. 😃

oorazoroo 20.58
l´unico pc sicuro è un pc spento

mook 14.12
beh errare è umano perseverare è da ban

blancks

30/03/2009 22:55:55

di nulla 😉

OGame
In OGame migliaia di giocatori da tutto il mondo competono tra di loro per conquistare l'intero universo!

jed,vitius

31/03/2009 13:51:39

avevo fatto un controllo simile solamente che al posto di $_POST['email']
avevo messo if(trim($nome) == "") comunque ho risolto facendo come ha detto blanck :).
Ma grazie lo stesso.

oorazoroo 20.58
l´unico pc sicuro è un pc spento

mook 14.12
beh errare è umano perseverare è da ban

Pagine: 1

Discussione seguita da: » jed,vitius

Non puoi scrivere in questo forum