Discussion :

[diaboloche]

Bonjour,

Petit problème...quoiqu'il arrive mon Select Count me renvois 1...même quand ma table est vide...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
$nombre=mysql_query("SELECT COUNT(email) FROM profiles WHERE email='$email'") or die ("<h1>erreur insert ".mysql_error()."</h1>");
    $nombre=mysql_num_rows($nombre);
	print $nombre;
 
	if($nombre<0) //---Si il n'existe pas, alors on ajoute---//
	{
	 mysql_query("INSERT INTO profiles (email) VALUES ('$email') ") or die ("<h1>erreur insert ".mysql_error()."</h1>");
	 print ("<h1>insertion réalisée !</h1><br>"); //---OK ajout effectué---//
	}
	else print ("<h1>Email déjà existant !</h1><br>"); //---Email déjà existant---//

Si quelqu'un peut m'aider...

[Invité]

peux-tu afficher ta requête sur ta page et voir si la condition est bien respectée?

Code :

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

print "SELECT email FROM profiles WHERE email='".$email."'";

[Arnaud F.]

Non je pense qu'il faut enlever les simples quotes, moi j'aurai écris

Code :

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

print "SELECT email FROM profiles WHERE email=$email;

Au passage,

Code :

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

print "SELECT email FROM profiles WHERE email='".$email."'";

et

Code :

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

print "SELECT COUNT(email) FROM profiles WHERE email='$email'"

sont identiques

[maximenet]

essaie de faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
$nombre=mysql_query("SELECT COUNT(email) as nbr_email FROM profiles WHERE email='$email'") or die ("<h1>erreur insert ".mysql_error()."</h1>");
    $nombre=mysql_fetch_object($nombre);
	//print $nombre;
	
	if($nombre->nbr_email==0) //---Si il n'existe pas, alors on ajoute---//
	{
	 mysql_query("INSERT INTO profiles (email) VALUES ('$email') ") or die ("<h1>erreur insert ".mysql_error()."</h1>");
	 print ("<h1>insertion réalisée !</h1><br>"); //---OK ajout effectué---//
	}
	else print ("<h1>Email déjà existant !</h1><br>"); //---Email déjà existant---//

[Amara]

Buchs => Si tu enlèves les quotes la requête ne marchera pas (car je suppose que le champ mail est une chaîne de caractères...).

[Arnaud F.]

Buchs => Si tu enlèves les quotes la requête ne marchera pas (car je suppose que le champ mail est une chaîne de caractères...).

Ben je sais pas, il a pas préciser ou chaine de caractere et si c'est un int, les 2 manières de l'écrire fonctionne

[maximenet]

avec une variable qui s'appelle $email on s'imagine bien que c'est une chaine de caractère

[Amara]

Ben je sais pas, il a pas préciser ou chaine de caractere et si c'est un int, les 2 manières de l'écrire fonctionne

Arf oui tu as raison dans ma tête un mail c'est du texte mais en fait c'est peut-être l'id du mail (ça serait plus logique mais le nom du champ est pas très bien choisi alors). Dsl

[diaboloche]

Merci de vos nombreuses réponses

Ma requête donne ceci (j'ai corriger en concaténé de façon plus propre) :

Code :

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

1
2
 
$nombre=mysql_query(" SELECT COUNT(email) FROM profiles WHERE email='".$email."'") or die ("<h1>erreur insert ".mysql_error()."</h1>");

A l'affichage :

Code :

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

1
2
 
SELECT COUNT(email) FROM profiles WHERE email='XxX@hotmail.com'

Donc c'est Ok pour moi...

mais il me renvoit tjrs 1...que j'ai 30 000 email identique ou non...

J'ai tenter en mettant le AS NBRE et faire le test sur NBRE mais le résultat est le même...

J'en suis donc là :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
$nombre=mysql_query(" SELECT COUNT(email) FROM profiles WHERE email='".$email."'") or die ("<h1>erreur insert ".mysql_error()."</h1>");
    $nombre=mysql_num_rows($nombre);
	print $nombre;   	
	if($nombre<=0) //---Si il n'existe pas, alors on ajoute---//
	{
	 mysql_query("INSERT INTO profiles (email) VALUES ('$email') ") or die ("<h1>erreur insert ".mysql_error()."</h1>");
	 print ("<h1>insertion réalisée !</h1><br>"); //---OK ajout effectué---//
	}
	else print ("<h1>Email déjà existant !</h1><br>"); //---Email déjà existant---//

Où est ce que j'aurais pu me tromper ?

[Arnaud F.]

Arf oui tu as raison dans ma tête un mail c'est du texte mais en fait c'est peut-être l'id du mail (ça serait plus logique mais le nom du champ est pas très bien choisi alors). Dsl

Non c'est moi qui m'excuse, j'avais pas fait attention au nom de la variable, sinon j'aurai pas réagis comme ça

Donc ton comptage te rend toujours UN maintenant?
Essai d'afficher ta requete sur une page web, tu verra ce que ta BDD recois comme ça

[maximenet]

ben je dit pas que ce que tu fait est faux mais moi j'y trouve bizzare car tu fait un select count puis un mysql_num_rows de çà

select count compte le nombre de résultat retourné donc un (int)
et mysql_num_rows te retourne le nombre de ligne que le résultat contient.

peut tu afficher le code si tu l'a fait à ma maniere ?

[Arnaud F.]

ben je dit pas que ce que tu fait est faux mais moi j'y trouve bizzare car tu fait un select count puis un mysql_num_rows de çà

select count compte le nombre de résultat retourné donc un (int)
et mysql_num_rows te retourne le nombre de ligne que le résultat contient.

peut tu afficher le code si tu l'a fait à ma maniere ?

Ben ouai forcement, select count(email) ne renvoi qu'une seule colonne, celle de COUNT et pas d'autre, donc si tu les comptes, c'est logique qu'il te renvoi toujours un !

[guitou12]

A mon avis ton erreur vient de là

Code :

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

$nombre=mysql_num_rows($nombre);

Car tu comptes le nombre de lignes renvoyées, or si je ne dis pas de bêtises ta requête va te renvoyer 1 ligne avec le nombre d'occurences de ton email trouvé !!!

Pour corriger déjà en 1er lieu donne une valeur à ton count --> count(email) as nb_emails

ensuite teste la valeur du champ nb_emails retourné

EDIT : grilled par les autres au dessus

[diaboloche]

Merci de vos réponse...

Ca marche finalement... j'avais fais une petite erreur au vu de l'exemple proposé par maximenet.

Voici la solution si cela peut aider quelqu'un à l'avenir :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
    $nombre=mysql_query(" SELECT COUNT(email) as NBRE  FROM profiles WHERE email='".$email."'") or die ("<h1>erreur insert ".mysql_error()."</h1>");
    $nombre=mysql_fetch_object($nombre);
 
	if($nombre->NBRE<=0) //---Si il n'existe pas, alors on ajoute---//
	{
	 mysql_query("INSERT INTO profiles (email) VALUES ('$email') ") or die ("<h1>erreur insert ".mysql_error()."</h1>");
	 print ("<h1>insertion réalisée !</h1><br>"); //---OK ajout effectué---//
	}
	else print ("<h1>Email déjà existant !</h1><br>"); //---Email déjà existant---//

[maximenet]

si tous tes emails sont unique tu avais aussi le possibilité de faire un

" SELECT id FROM profiles WHERE email='".$email."'"

et sur ce résultat là tu peux faire un mysql_num_rows pour voir s'il te retourne au moin une ligne alors, l'email est présent, s'il te retourne 0 ligne, alors il n'est pas présent.