[PDO] controllo su campo e-mail

[passasooz]

buonasera,
sto facendo il mio primo script con le PDO e ho un problemino..

lo script è questo:

<?php
session_start();
$col = 'mysql:host=localhost;dbname=tattoonia';
$db = new PDO($col , 'root', '');
$nome=$_POST['nome'];
$cognome=$_POST['cognome'];
$sesso=$_POST['sesso'];
$email=$_POST['email'];
$password=$_POST['password'];
$tipo=$_POST['tipo'];
$nazione=$_POST['nazione'];
$citta=$_POST['citta'];
$provincia=$_POST['provincia'];
$indirizzo=$_POST['indirizzo'];
$telefono=$_POST['telefono'];
$controllo = $db->prepare("SELECT email FROM utente WHERE email = '$email'");
$controllo->execute();
$risultato = $controllo->fetchAll();
if ($risultato > 0){
    echo "<script>alert(\"indirizzo e-mail occupato\");</script>";
}
else{
$sql = "INSERT INTO utente (nome,cognome,tipo,nazione,citta,provincia,indirizzo,email,password,telefono,sesso) VALUES (:nome,:cognome,:tipo,:nazione,:citta,:provincia,:indirizzo,:email,:password,:telefono,:sesso)";
$q = $db->prepare($sql);
$q->execute(array(':nome'=>$nome,':cognome'=>$cognome,':tipo'=>$tipo,':nazione'=>$nazione,':citta'=>$citta,':provincia'=>$provincia,':indirizzo'=>$indirizzo,':email'=>$email,':password'=>$password,':telefono'=>$telefono,':sesso'=>$sesso));
header("Location:index.php#accedi");
}
?>

il problema è che se io inserisco un indirizzo e-mail che non esiste nella tabella utenti lo script mi dice ugualmente che l'indirizzo è occupato..

qualcuno sa aiutarmi?

[ciro78]

fetchAll restituisce un array vuoto qualora l'esecuzione della query non produca risultati.
Quindi il controllo che fai non va bene.

O conti gli elementi dell'array oppure fai una query del tipo

$controllo = $db->prepare("SELECT count(email) as num FROM utente WHERE email = '$email'");

e da li il resto....

[passasooz]

ho fatto così però comunque non funziona nel senso che se io aggiungo un utente con email che nel db non esiste lo script mi dice che esiste ugualmente...

$controllo = $db->prepare("SELECT count(email) FROM utente WHERE email = '$email'");
$controllo->execute();
if($controllo->rowCount() > 0){
    echo "esiste";
}
else{
    $sql = "INSERT INTO utente (nome,cognome,tipo,nazione,citta,provincia,indirizzo,email,password,telefono,sesso) VALUES (:nome,:cognome,:tipo,:nazione,:citta,:provincia,:indirizzo,:email,:password,:telefono,:sesso)";
    $q = $db->prepare($sql);
    $q->execute(array(':nome'=>$nome,':cognome'=>$cognome,':tipo'=>$tipo,':nazione'=>$nazione,':citta'=>$citta,':provincia'=>$provincia,':indirizzo'=>$indirizzo,':email'=>$email,':password'=>$password,':telefono'=>$telefono,':sesso'=>$sesso));
    header("Location:index.php#accedi");
}

[ciro78]

è ovvio la query che fai restituisce sempre una riga. se usi rowCount devi fare la query come facevi prima

[passasooz]

ok ho usato la query che avevo scritto in precedenza e così funziona:

$controllo = $db->prepare("SELECT email FROM utente WHERE email = '$email'");
$controllo->execute();
if($controllo->rowCount() > 0){
    echo "esiste";
}
else{
    $sql = "INSERT INTO utente (nome,cognome,tipo,nazione,citta,provincia,indirizzo,email,password,telefono,sesso) VALUES (:nome,:cognome,:tipo,:nazione,:citta,:provincia,:indirizzo,:email,:password,:telefono,:sesso)";
    $q = $db->prepare($sql);
    $q->execute(array(':nome'=>$nome,':cognome'=>$cognome,':tipo'=>$tipo,':nazione'=>$nazione,':citta'=>$citta,':provincia'=>$provincia,':indirizzo'=>$indirizzo,':email'=>$email,':password'=>$password,':telefono'=>$telefono,':sesso'=>$sesso));
    header("Location:index.php#accedi");
}

grazie mille dell'aiuto..

[Al_katraz984]

$stmt= $db->prepare("SELECT email FROM utente WHERE email = :email'");
$stmt->execute( array( ':email', $email ) );

if ( $stmt->rowCount() > 0 ) { 

usa i placeholder con pdo altrimenti non gestisci i dati.

[passasooz]

ho modificato il codice come mi hai detto tu e non mi funziona più

[satifal]

ho modificato il codice come mi hai detto tu e non mi funziona più

Se ti limiti al copia/incolla!!!
Magari prendendo un minimo di iniziativa e consultando il buon vecchio manuale (questo sconosciuto):

execute()

[passasooz]

aH VERO ho notato ora che l'execute doveva esser scritto col =>

[satifal]

Dai un'occhiata anche al metodo bindValue()