Discussion :

[Avatar69]

Bonjour,
j'ai un petit souci dans une requete SQL

mon souci vien de cette partie de ma requete

Code :

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

AND DL_QteBL > '" + rdr4.GetString(1) + "'

Le rdr4 viens d'une précédente requete que voici

Code :

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

sqlstr4 = "select distinct CATEGORIE,SEUIL_NDP,AR_Ref from F_ARTICLE WHERE CATEGORIE <> 'NULL' ";

Dans les deux tables les colones de ses champs sont en numeric.

Quand j'ai des colones en varcher avec du GetString ca marche mais la il en veux pas j'ai essayer pas mal de get different (decimal, float, int) mais il me fait toujours une erreur sur cette partie de ma requete.

Merci pour votre aide

Thierry

[Arthis]

je ne pense pas que ce soit réellement le bon forum pour poster vu qu'il s'agit de sql, mais avec mes maigres compétences je vais essayer un début de réponse:

Code :

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

AND DL_QteBL > '" + rdr4.GetString(1) + "'

Dans les deux tables les colones de ses champs sont en numeric.

essaye d'éviter les quotes si les deux sont du numérique.

Code :

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

AND DL_QteBL > " + rdr4.GetString(1) + "

[Avatar69]

Merci pour cet reponse rapide, mais ce ne change pas mon souci.
Je met le message exacte de l'erreur

Impossible d'effectuer un cast d'un oblet de type "System.Decimal" en type "System.String"

Merci

Thierry

[bossun]

Code :

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

rdr4.GetString(1).ToString()

Erreur de débutant

[Avatar69]

Merci pour cet réponse

Alors avec

Code :

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

AND DL_QteBL >'" + rdr4.GetString(1).ToString() + "'

ou

Code :

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

AND DL_QteBL >" + rdr4.GetString(1).ToString() + "

j'ai la meme erreur a savoir

Impossible d'effectuer un cast d'un oblet de type "System.Decimal" en type "System.String"

j'ai essayer avec

Code :

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

AND DL_QteBL > rdr4.GetString(1).ToString()

et la j'ai une autre erreur ailleurs bizarre qui est

"Ligne 1 : syntaxe incorrecte vers '.'."

Merci de votre aide

Thierry

[bossun]

poste ta ligne qui crée l'erreur...

Code :

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

AND DL_QteBL >'" + rdr4.GetString(1).ToString() + "'"

[Avatar69]

Voici ma lrequete en totalité

Code :

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

sqlstr = "select * from F_DOCLIGNE where YEAR(DL_DateBL) = '" + textBox2.Text + "' AND MONTH(DL_DateBL) = '" + textBox1.Text + "' AND AR_Ref LIKE '" + rdr4.GetString(2) + "' AND DL_QteBL > '" + rdr4.GetString(1).ToString() + "'";

Sachant que cet requete marche

Code :

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

sqlstr = "select * from F_DOCLIGNE where YEAR(DL_DateBL) = '" + textBox2.Text + "' AND MONTH(DL_DateBL) = '" + textBox1.Text + "' AND AR_Ref LIKE '" + rdr4.GetString(2) + "'";

Merci pour votre patience

Thierry

[bossun]

Code :

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

sqlstr = "select * from F_DOCLIGNE where YEAR(DL_DateBL) = '" + textBox2.Text + "' AND MONTH(DL_DateBL) = '" + textBox1.Text + "' AND AR_Ref LIKE '" + rdr4.GetString(2) + "' AND DL_QteBL > " + rdr4.GetString(1).ToString() ;

il faut enlever les ' quand tu as du numérique

[Avatar69]

Bonsoir,

même message d'erreur avec cet version

Code :

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

sqlstr = "select * from F_DOCLIGNE where YEAR(DL_DateBL) = '" + textBox2.Text + "' AND MONTH(DL_DateBL) = '" + textBox1.Text + "' AND AR_Ref LIKE '" + rdr4.GetString(2) + "' AND DL_QteBL > " + rdr4.GetString(1).ToString() ;

Impossible d'effectuer un cast d'un oblet de type "System.Decimal" en type "System.String"

Merci de votre aide

Thierry

[tortuegenie]

test ta requete dans le requeteur de visual studio si tu peux et regarde dans le debugger ce que te renvoie rdr4.GetString(1).ToString()

[_PascalC_]

Salut, l'utilisation des requêtes paramétrées devrait permettre de résoudre ton problème et aussi de contourner d'autres soucis (L'injection SQL et autres joyeusetés comme les apostrophes, format de date, nombre...)

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
 
using (SqlConnection c = new SqlConnection("..."))
{
    c.Open();
    using (SqlCommand cmd = c.CreateCommand())
    {
        cmd.CommandText = "select * from F_DOCLIGNE where YEAR(DL_DateBL) = @Year AND MONTH(DL_DateBL) = @Month AND AR_Ref LIKE @Ref AND DL_QteBL > @Qte";
        cmd.Parameters.AddWithValue("@Year", textBox2.Text);
        cmd.Parameters.AddWithValue("@Month", textBox1.Text);
        cmd.Parameters.AddWithValue("@Ref", rdr4.GetString(2));
        cmd.Parameters.AddWithValue("@Qte", rdr4.GetDecimal(1));
        using (SqlDataReader reader = cmd.ExecuteReader())
        {
            //...
        }
    }
}

Accessoirement pense à nommer tes composants car TextBox1, TextBox2... ce n'est pas super explicite surtout si un jour une autre personne que toi travaille sur le projet .

Concernant le champs "DL_DateBL" je suppose qu'il est de type DateTime (ou équivalent). Dans ce cas essaie plutôt de faire un requêtage de ce type :

Pour les données de sept. 2009 par exemple :

Code :

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

1
2
3
 
SELECT * FROM F_DOCLIGNE 
WHERE (DL_DateBL >= '20090901' AND DL_DateBL < '20091001')

Une des principales raisons c'est qu'avec ta requête même si le champs DL_DateBL est un jour indexé, cet index ne sera jamais utilisé et le moteur de la DB sera obligé de parcourir toute la table.

[emaluk]

Salut Avatar69,

Juste une suggestion, si tu essaies de caster:

Code :

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

1
2
3
4
 
AND DL_QteBL > CAST(rdr4.GetString(1) AS decimal)
Ou
CONVERT(decimal, rdr4.GetString(1))

A+

[_PascalC_]

Le problème c'est que le champs à lire est apparemment de type Decimal et donc "rdr4.GetString(1)" ne marchera pas. Un GetString est fait pour récupérer une string, pas un nombre ou une date. Pour récupérer un Decimal il faut taper sur la méthode GetDecimal() ou - à la rigueur - faire GetValue(1)

[Avatar69]

Je vous remercie pour toutes ses réponses.

j'ai un peu creusé le sujet et j'ai peut etre trouvé mon souci.

Mes deux colones sont bien "numeric (24,6)" par contre les nombres enregistrer dedans sont avec des virgules et non des points et je pense que mon souci viens de la (et je ne peu pas modifier les format des colonnes)

En plus on ne se sert pas de la partie décimale.....

il faut donc que je recupere la partie entiere de mon nombre et ca devrait marché ou que je remplace ma virgule par un point

Merci a tous

Thierry

[_PascalC_]

les nombres enregistrer dedans sont avec des virgules et non des points

Non ! Ton SGBD stocke un nombre sur x octet(s) et pas une chaine de caractères représentant un nombre. Cette histoire de point et de virgule ce n'est que de l'affichage, dont - pour rappel - tu n'aurais pas à te soucier avec des requêtes paramétrées

[Immobilis]

Salut,

Non ! Ton SGBD stocke un nombre sur x octet(s) et pas une chaine de caractères représentant un nombre.

Ca dépend pas un peu des options régionnales de l'OS sur lequel se trouve la base, ainsi que de la culture de la base elle-même?

Franchement, si tu veux 0 soucis y'a pas 36 solutions:

1. Procédures stockées => sécurité et performance
2. Spécification du type des valeurs passée à la procédure => plus d'erreur sur les dates et les nombres. Tu as un champ texte pour récupérer la date. Il vaut mieux utiliser un calendrier .Net ou alors être certain que tu sais où se trouve le mois et le jour. La date 01/02/2009 ne signifie pas la même chose en français et en anglais.
3. Conversion des valeurs reçues => plus d'erreur sur les dates et les nombres

Des exemples ici: http://dotnet.developpez.com/faq/asp...=adonetrequete

A+

[Avatar69]

Merci a tous pour vos réponses, j'ai réussi a m'n sortir grace au requetes paramétrées

Bravo

Thierry