Récuperer le nombre de ligne d'une requete SQL

Version imprimable

Salut à tous.
ça fait des heures que je songe comment réussir mon code.
Je veux récupérer le résultat d'un Count(*) d'une requête, mais ça marche pas.
Voici ma classe:
Code:

1 2 3 4 5 6 7 8 9 private int Execute_Select(string requeteString) { var connString = "....Chaine de connection..."; SqlConnection conn = new SqlConnection(connString); conn.Open(); SqlCommand cmd = new SqlCommand(requeteString, conn); int result = (Int32)cmd.ExecuteScalar(); return result; }
Plus loin dans le programme:
Code:

1 2 3 4 5 6 7 8 string v_sql_search_mail = "SELECT COUNT(*) FROM ....."; if (Execute_Select(v_sql_search_mail)!=0) { MessageBox.Show(" Cette adresse Email est déjà inscrite chez nous!", "Information", MessageBoxButtons.OK, MessageBoxIcon.Warning); v_test_saisi = false; return; }
Lors de l'exécution, j'ai toujours le message de la BOX, quand j'ai affiché le contenu de "Execute_Select(v_sql_search_mail)", elle me donne 1... Je ne comprend rien...
Merci à vous.

11/05/2011, 18h22
giova_fr

vu de loin ca a l'air plutot correcte...

tu as testé ta requette en base?
11/05/2011, 18h26
calagan99

Utilise la balise CODE, ton post sera plus lisible...
Sinon, ta question n'est pas très claire.
Si

Code:

Execute_Select(v_sql_search_mail)

est différent de zéro, il est normal que tu vois ton texte puisque tu vas forcément rentrer dans le contenu de ta clause "if".
Il faut probablement que tu ajoutes un else dans lequel, si

Code:

Execute_Select(v_sql_search_mail)

est différent de zéro, tu affiches la valeur retournée par ta requête plutôt que ton message.
11/05/2011, 18h39
drissi

Merci à Deepin pour la mise en forme code.
L'idée, c'est de chercher si l'adresse mail entrée par un nouvel utilisateur est déjà inscrite dans la base de donnée, on lui affiche un message.
Mais quand je teste avec une adresse mail inexistante dans la base, j'ai le message de la BOX.
Sinon la requête marche très bien, la voila:

Code:

string v_sql_search_mail = "SELECT COUNT(*) FROM [dbo].[XXOA_TABLE_MAILS] WHERE [MAIL]='" +Email_TextBox.Text.ToString() + "'";

Test comme ça

Code:

1
2
3
4
5
6
7
8
9
10
private int Execute_Select(string requeteString)
{
var connString = "....Chaine de connection...";
SqlConnection conn = new SqlConnection(connString);
conn.Open();
SqlCommand cmd = new SqlCommand(requeteString, conn);
int result =0;
result = (Int32)cmd.ExecuteScalar();
return result;
}

Code:

1
2
3
4
5
6
7
8
string v_sql_search_mail = "SELECT COUNT(*) FROM .....";
 
if (Execute_Select(v_sql_search_mail)>0)
{
MessageBox.Show(" Cette adresse Email est déjà inscrite chez nous!", "Information", MessageBoxButtons.OK, MessageBoxIcon.Warning);
v_test_saisi = false;
return;
}

11/05/2011, 23h24
giova_fr

Hmmm vu ce qu'il y a dans la parenthese :

Code:

(Execute_Select(v_sql_search_mail)>0)

le resultat de Execute_Select devrait implicitement etre converti en int...

J'avous ne pas trop voir où est le probleme, je suspecte plus le comportement de la requette. Mais c'est toujours moins facile de trouver quand on est pas face au debugger...
11/05/2011, 23h28
giova_fr
Hmmm vu ce qu'il y a dans la parenthese :

Code:

(Execute_Select(v_sql_search_mail)>0)

le resultat de Execute_Select devrait implicitement etre converti en int...

J'avous ne pas trop voir où est le probleme, je suspecte plus le comportement de la requette. Mais c'est toujours moins facile de trouver quand on est pas face au debugger...

juste au cas où, @drissi : tu es bien conscient que les emails doivent dans ton code etre parfaitement identique? que si dans la base tu as fais des tests en mettant l'email en majuscule, ou avec un espace apres, tu n'auras pas le meme resultat que dans ton code? (bete à dire mais personne n'est à l'abris de ce genre d'erreur stupide et moi le premier...)

du coup j'en profite pour rajouter :

Citation:

est différent de zéro, il est normal que tu vois ton texte puisque tu vas forcément rentrer dans le contenu de ta clause "if".
Il faut probablement que tu ajoutes un else dans lequel, si
Code :Sélectionner tout - Visualiser dans une fenêtre à partExecute_Select(v_sql_search_mail)est différent de zéro, tu affiches la valeur retournée par ta requête plutôt que ton message.

ca me parrait inutile, dans son code, si tout va bien il continue, sinon il affiche un message d'erreur. Par contre dans son cas j'afficherai en effet le resultat en cas de pb (histoire de comprendre ce qui se passe):
Code:

1 2 3 4 5 int resulat = Execute_Select(v_sql_search_mail); if(resultat >0) { MessageBox.Show(String.Format"Email Deja inscrit ({0})",resultat); }
12/05/2011, 10h32
drissi

ça a marché.
Bon, j'ai fait une erreur car j'ai laissé ma requête qui contient une adresse mail figée que j'ai renseigné, et j'ai changé le "!=0" vers ">0"...
Merci à vous pour vos réponses.