aiuto codice registrazione

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

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

quota

Pagine → 1

blancks

SCRIVI

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

Metin2 ↗
Sfida i tuoi rivali a battersi con te e dimostra la tua superiorità in questo mondo fantasy!

quota

jed,vitius

SCRIVI

30/03/2009 22:15:01 e modificato da jed,vitius il 30/03/2009 22:29:54

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

quota

blancks

SCRIVI

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.

Enlisted ↗
Guida la tua squadra di soldati in combattimenti su larga scala, con fanteria, veicoli corazzati e aerei della IIa Guerra Mondiale!

quota

jed,vitius

SCRIVI

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

quota

blancks

SCRIVI

30/03/2009 22:55:55

di nulla ;-)

Lineage II ↗
Entra in uno sconfinato mondo fantasy dominato da razze in contrasto tra loro. Scatena i tuoi poteri in uno dei Mmo più famosi al mondo!

quota

jed,vitius

SCRIVI

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

quota

Discussione seguita da › jed,vitius

Pagine → 1

Rispondi alla Discussione Aggiungi ai Preferiti Inoltra Discussione Forum Programmazione, GDRCD, Open Source, Hosting Elenco Forum