Discussion :

[Avatar69]

Bonjour,

j'ai un petit souci, je sais ou est mon pb mais je sais pas comment le résoudre !!!
En gros j'additionne des choux avec des carottes et ca marche pas forcement !!!

Code :

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

1
2
3
4
5
6
7
8
9
10
11
sqlstr2 = "select * from F_DOCLIGNE where DO_PIECE like '"+row[2]+"'";
SqlCommand oCmd3 = new SqlCommand(sqlstr2, connection2);
rdr2 = oCmd3.ExecuteReader();
Object[] row2 = null;
while (rdr2.Read())
{
    row2 = new Object[rdr2.FieldCount];
    rdr2.GetValues(row2);
    montant_brut = montant_brut + row2[77];
    nb_ligne=nb_ligne+1;
}

Mon souci et sur cet ligne la vous vous en doutiez

Code :

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

montant_brut = montant_brut + row2[77];

Merci pour votre aide
Thierry

[Arthis]

Regardes du coté de la class Convert, elle possède plein de méthodes statiques pour transformer choux en carottes et inversement...

[calagan99]

Comme le dit Arthis, il faut que tu castes le contenu de "row2[77]".

Autre point important : utilise des requêtes paramétrées !!!

[laedit]

Autre point important : utilise des requêtes paramétrées !!!

+1

http://dotnet.developpez.com/faq/asp...onet_reqparams

[Avatar69]

Bonjour,
merci pour vos réponses, j'ai un peu cherche j'ai trouvé ca Ms MSDN

Mais j'avoue que j'y comprend pas trop .... je voie pas ce que je dois faire par rapport a mon code.

Sinon pour les requetes parametrables je les utilises car sinon j'ai des souci sur des requetes trop longues.

Merci pour votre aide
Thierry

[laedit]

montant_brut est d'un certain type, tandis que row[77] est un objet, tu ne peux donc pas concaténer ou additionner les deux, il te faudra convertir row[77] en type de montant_brut.

[Avatar69]

Re,
oui ca je voie bien ou est mon souci apres c comment le résoudre j'ai essayé pas mal de chose du genre

Code :

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

1
2
montant_brut = montant_brut + row2[77].GetDecimal;
montant_brut = montant_brut + GetDecimal(row2[77]);

et j'en passe des trucs plus farfelu....

Merci encore
Thierry

[laedit]

Pour qu'on puisse t'aider, tu pourrais tout simplement dire de quel type est montant_brut...

[EDIT] au vu de ton dernier post, ce doit être un décimal. Tente donc quelque ceci :

Code :

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

montant_brut = montant_brut + Convert.ToDecimal(row[77]);

[calagan99]

Je dirais même plus :

Code :

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

1
2
3
decimal mb;
if (Decimal.TryParse(row2[77], out mb))
    montant_brut = mb;

C'est un poil plus complexe mais la fonction TryParse t'assure du fait que le contenu de row2[77] sera bien du type Decimal.

Si tu veux en savoir plus sur TryParse, c'est ici.

D'autre part, concernant les requêtes paramétrées :

Sinon pour les requetes parametrables je les utilises car sinon j'ai des souci sur des requetes trop longues.

Non seulement ne pas les utiliser est source d'erreur comme tu le fais remarquer à juste titre.
Mais c'est surtout une grosse faille de sécurité et un trou béant dans ton application.
Sans compter que cela t'apportera de meilleures performances.

Tu as donc tout intérêt à les utiliser

[Avatar69]

Merci pour vos réponses
Cela fonctionne du coup

Bonne journée

Thierry