isNull renvoie "false" au lieu de "0"

Version imprimable

[résolu]isNull renvoie "false" au lieu de "0"

Bonjour,

ce code :
Code:

1 2 3 4 5 SQLQuery = "SELECT isnull(sum(PrixAchat),0) AS cumul FROM gratuit WHERE (prevision = 1)"; sqlCommand = new SqlCeCommand(SQLQuery, connexion); reader = sqlCommand.ExecuteReader(); reader.Read(); lblprvmontant.Text = reader["cumul"].ToString();
inscrit "false" dans le label lblprvmontant. C'a n'est pas faux en soi, mais j'aimerais qu'il me mette un zéro à la place

Merci pour vos éclaircissements

03/03/2009, 12h29
AJemni

je croix qu'il faut inverser :

Code:

select sum(isnull(PrixAchat,0)) As cumul From gratuit where ...
03/03/2009, 12h38
talapoga

Citation:

Envoyé par AJemni

je croix qu'il faut inverser :

Code:

select sum(isnull(PrixAchat,0)) As cumul From gratuit where ...

J'ai essayé cette syntaxe, que j'ai lue sur le net, mais je me prends une erreur "La valeur d'argument spécifié pour cette fonction n'est pas valide". en outre, je ne suis pas sur que s'il n'y avait pas d'erreur, elle me renverra un 0 plutôt que false.

D'autre part, comme je cherche soit 0 soit la valeur cumulée de la colonne PrixAchat, il me semblais logique de tester après la totalisation. Peut être ai-je tout faux ?
03/03/2009, 13h03
AJemni

Q : Quelle est ton SGBD?
05/03/2009, 14h38
talapoga

Citation:

Envoyé par AJemni

Q : Quelle est ton SGBD?

Bonjour,

C'est SqlServeur 2005 Compact Edition
05/03/2009, 14h46
Sergejack

...CAST (isnull(sum(PrixAchat),0) AS INTEGER) ...

Mais c'est bizzare.
05/03/2009, 15h22
talapoga

Citation:

Envoyé par Sergejack

...CAST (isnull(sum(PrixAchat),0) AS INTEGER) ...

Mais c'est bizzare.

Qu'est qui est bizarre ? En tout cas, la il remplace bien "false", mais par 1. Même si je mais 1 dans les parenthèses du cast ... Et je ne comprends pas ce qu'il fait.
05/03/2009, 15h32
AJemni

Salut,
je croix que j'ai pas bien compris ton PB :
qu'est ce que ton label doit afficher au juste :

- La somme des PrixAchat.
- Une valeur boolean : 0, 1, faut, ou vrai, et si c'est le cas, que doit signifier cet valeur
05/03/2009, 15h40
talapoga

Citation:

Envoyé par AJemni

Salut,
je croix que j'ai pas bien compris ton PB :
qu'est ce que ton label doit afficher au juste :

- La somme des PrixAchat.
- Une valeur boolean : 0, 1, faut, ou vrai, et si c'est le cas, que doit signifier cet valeur

En fait, s'il y a des montants, il faudrait afficher la somme de ces montant, sinon affiché 0 (ou mieux 0,00, mais ca c'est plus de l'ordre du
cosmétique ;-) )

Je vais faire "manuellement" le test avec un IF, je crois.

En tout cas merci à vous deux
05/03/2009, 15h43
talapoga

Citation:

Envoyé par talapoga

[...]
Je vais faire "manuellement" le test avec un IF, je crois.

En tout cas merci à vous deux

Oops ! Autant pour moi, le test dans la requête (prévision=1) est faux (prevision =0) est juste et la ca marche.

Résolu, donc ;-)
05/03/2009, 16h11
Sergejack

Mets le code final stp, que la solution finale soit claire et utile.

Citation:

Envoyé par Sergejack

Mets le code final stp, que la solution finale soit claire et utile.

Voilà :

Code:

1
2
3
4
5
6
 // Prevision montant 
SQLQuery = "SELECT cast(isnull(sum(PrixAchat),0) as integer) AS prvmontant FROM gratuit WHERE (prevision = 0)";
sqlCommand = new SqlCeCommand(SQLQuery, connexion);
reader = sqlCommand.ExecuteReader();
reader.Read();
lblprvmontant.Text = reader["prvmontant"].ToString();

:mur:
Citation:
Envoyé par talapoga
Voilà :
Code:

1 2 3 4 5 6 // Prevision montant SQLQuery = "SELECT cast(isnull(sum(PrixAchat),0) as integer) AS prvmontant FROM gratuit WHERE (prevision = 0)"; sqlCommand = new SqlCeCommand(SQLQuery, connexion); reader = sqlCommand.ExecuteReader(); reader.Read(); lblprvmontant.Text = reader["prvmontant"].ToString();
8O franchement ton code n'est pas clair, essay de le modifier :
- ajoute un try-catch
- ajoute
Code:

1 2 3 4 if(reader.Read()) { lblprvmontant.Text = reader["prvmontant"].ToString(); }
il sera plus lisible lors de la maintenance par exemple... :?

05/03/2009, 18h55
talapoga
Citation:

Envoyé par AJemni

:mur:

8O franchement ton code n'est pas clair,

essay de le modifier :
- ajoute un try-catch
- ajoute
Code:

1 2 3 4 if(reader.Read()) { lblprvmontant.Text = reader["prvmontant"].ToString(); }
il sera plus lisible lors de la maintenance par exemple... :?[/QUOTE]

Ah ?

Ben oui, ca se peut que ce soit pas beau/lisible, c'est tirer de copier coller adapté. J'ai un peu de temps devant moi que j'utilise pour m'initier à C#. Là, c'est un extrait ... Il ya un datagridview, plus quelques labels à remplir. Toute la partie connexion est traitée avant et réutilisée tour à tour pour remplir chaqun des contrôles..Il ya deux try/cacth : Un qui vérifié que la chaine de connexion est bien récupérée par un configurationManager, l'autre qui s'assure que la base de donnée est bien "en ligne". toutes mes requêtes sont à l'intérieur de ce seconf Try. Tu en mettrais un pour chaque requête ?

Pour le "if (reader.read())", c'est ce à quoi je pensais quand je disais que j'allais faire le test "manuellement" :)
05/03/2009, 19h01
talapoga

quelques petites précisions
Citation:

Envoyé par AJemni

:mur:

8O franchement ton code n'est pas clair,

essay de le modifier :
- ajoute un try-catch
- ajoute
Code:

1 2 3 4 if(reader.Read()) { lblprvmontant.Text = reader["prvmontant"].ToString(); }
il sera plus lisible lors de la maintenance par exemple... :?[/QUOTE]

Ah ?

Ben oui, ca se peut que ce soit pas beau/lisible, c'est tirer de copier coller adapté. J'ai un peu de temps devant moi que j'utilise pour m'initier à C#. Là, c'est un extrait ... Il ya un datagridview, plus quelques labels à remplir. Toute la partie connexion est traitée avant et réutilisée tour à tour pour remplir chaqun des contrôles..Il ya deux try/cacth : Un qui vérifié que la chaine de connexion est bien récupérée par un configurationManager, l'autre qui s'assure que la base de donnée est bien "en ligne". toutes mes requêtes sont à l'intérieur de ce seconf Try. Tu en mettrais un pour chaque requête ?

Pour le "if (reader.read())", c'est ce à quoi je pensais quand je disais que j'allais faire le test "manuellement" :)
06/03/2009, 09h16
Sergejack

Le read fonctionnera toujours si ce query s'est excécuté sans erreur.
Le if (...) est donc dispensable et à ce titre, mieux vaut ne pas le mettre et économiser de l'indentation.

Contrairement à vous Sergejack, j'irai plus loin encore en écrivant :

Code:

1
2
3
4
5
6
 
if(reader.Read())
{
       if(reader["prvmontant"] != DBNull.Value)
             lblprvmontant.Text = reader["prvmontant"].ToString();
}

09/03/2009, 12h20
Sergejack

Je te signal que tout ce joli travaille est déjà fait au niveau du query.

Mais si tu ne fais pas confiance à l'exécution correcte de ce query, alors c'est le .Net lui même que tu réfutes.