Discussion :

[Scots]

Bonjour,

Je cherche à faire un test si un internaute existe dans ma base (champs email) puis faire un insert si non ou un update si oui.

Voici mon code (cette page est après la validation du formulaire) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
//Vérification email
$internaute = $bdd->prepare('SELECT email FROM profil WHERE email = ? ');
$internaute->execute(array($_GET['email']));
if (empty($internaute->fetch()) {
	//Insertion profil
	$req = $bdd->prepare('INSERT INTO profil(prenom, nom, email, tel, adresse, code_postal, ville, password, ip) VALUES(:prenom, :nom, :email, :tel, :adresse, :code_postal, :ville, :password, :ip)');
	$req->execute(array(
		'prenom' => $prenom,
		'nom' => $nom,
		'email' => $email,
		'tel' => $tel,
		'adresse' => $adresse,
		'code_postal' => $code_postal,
		'ville' => $ville,
		'password' => $password,
		'ip' => $ip
	));
}
else {
	//Update profil
	$req = $bdd->prepare('UPDATE profil SET prenom = :prenom, nom = :nom, email = :email, tel = :tel, adresse = :adresse, code_postal = :code_postal, ville = :ville, password = :password, ip = :ip');
	$req->execute(array(
		'prenom' => $prenom,
		'nom' => $nom,
		'email' => $email,
		'tel' => $tel,
		'adresse' => $adresse,
		'code_postal' => $code_postal,
		'ville' => $ville,
		'password' => $password,
		'ip' => $ip
	));
}

Je pense que mon "if" n'est pas correct. Quelqu'un pourrait-il m'aiguiller ?

Merci de votre aide.

Scots

[sabotage]

Pourquoi penses-tu que ton if n'est pas correct ?
Tu as debugué un minimum ?

Dans le cas ou il n'y a pas de résultat le fetch() sera FALSE, autant écrire precisement le bon résultat plutôt que de faire un test général.

Tu as également la possibilité d'utiliser une requête "ON DUPLICATE KEY UPDATE" qui gérera donc à la fois un INSERT ou un UPDATE ; sauf bien sur si tu veux un traitement distinct supplémentaire pour chaque cas.

[Bovino]

Pourquoi penses-tu que ton if n'est pas correct ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if (empty($internaute->fetch())

empty() attend comme paramètre une variable, donc effectivement, la condition est mal écrite.

[rawsrc]

@Bovino

Depuis PHP 5.5, empty() supporte tous les styles de notations et pas uniquement les variables. Doc

[sabotage]

Donc pour le coup il n'y a pas d’ambiguïté : soit on est > 5.5 et ça fonctionne, soit il y a une erreur explicite.

[Bovino]

Depuis PHP 5.5, empty() supporte tous les styles de notations

soit on est > 5.5 et ça fonctionne, soit il y a une erreur explicite

J'étais parti du principe que si Scots poste un problème, c'est que la probabilité pour que son code ne fonctionne pas était élevée, j'ai donc subodoré une version < 5.5 en me basant sur

Je pense que mon "if" n'est pas correct.

et ai donc considéré inutile d'évoquer le fait que que s'il avait une version plus récente, ça marcherait mieux.

[Scots]

Bonjour à tous et merci de l'aide que vous pourrez m'apporter.

Après vérification, ma version de PHP est la 5.3.13

Si cela peut vous aider ...

[sabotage]

Comme dis dans mon premier message

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if ($internaute->fetch() == FALSE) {

[Scots]

Merci Sabotage, j'avais zappé ...

Testé : Nada. Pas d'update ...

En fait il me fait l'insert à chaque fois au lieu de faire l'update.

Edit :

Petite erreur de ma part. Ta solution fonctionne impec.

sur l'email j'avais mis bêtement un $_GET['email'] au lieu d'un $_POST['email'] ...

Merci à tous !