Discussion :

[Msysteme]

Bonjour, prière de m'aider à trouver l'erreur, je n'arrive plus a la détecter :

Code :

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

1
2
3
4
5
 
"INSERT INTO [Appelsdoffre] VALUES (@id,..., @AvecPonderation, @FormuleEvaluation, ...)";
 
myCommand.Parameters.Add(new SqlParameter("@AvecPonderation", SqlDbType.Bit));
myCommand.Parameters.Add(new SqlParameter("@FormuleEvaluation", SqlDbType.VarChar, -1));

...

Code :

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

1
2
                myCommand.Parameters.Add(new SqlParameter("@AvecPonderation", SqlDbType.Bit));
                myCommand.Parameters.Add(new SqlParameter("@FormuleEvaluation", SqlDbType.VarChar, -1));

Code :

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

1
2
3
4
5
6
                myCommand.Parameters["@AvecPonderation"].Value = chkPonderation.CheckState;
 
                if (chkPonderation.Checked == true)
                    myCommand.Parameters["@FormuleEvaluation"].Value = cmbNT.SelectedItem.ToString() + "NT+" + cmbNF.SelectedItem.ToString() + "NF"; // l'erreur
                else
                    myCommand.Parameters["@FormuleEvaluation"].Value = DBNull.Value;

il me dit que impossible de convertir cmbNT.SelectedItem.ToString() + "NT+" + cmbNF.SelectedItem.ToString() + "NF"; en bit alors que FormuleEvaluation est un varchar(MAX)

[ZenZiTone]

Dans notre métier, les erreurs illogiques n'existent pas

Es-tu sûr que dans "VALUES", toutes les valeurs sont dans le bon ordre? Pour en être sûr, préfère utiliser la syntaxe :

Code :

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

1
2
INSERT INTO table (nom_colonne_1, nom_colonne_2, ...
 VALUES ('valeur 1', 'valeur 2', ...)

[Pol63]

fais shift+f9 sur myCommand.Parameters["@FormuleEvaluation"] via un point d'arrêt, tu pourras vérifier le type du paramètre

Pour en être sûr, préfère utiliser la syntaxe :

Code :

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

1
2
INSERT INTO table (nom_colonne_1, nom_colonne_2, ...
 VALUES ('valeur 1', 'valeur 2', ...)

ca serait moi qui ferait les règles du forum, je mettrais une semaine d'expulsion pour conseil de concaténation de paramètre
surtout que l'ordre des paramètres n'importe pas sur les vrais sgbdr vu qu'il utilise le nom (sur access par contre il prend dans l'ordre et ca peut faire n'importe quoi)

[ZenZiTone]

ca serait moi qui ferait les règles du forum, je mettrais une semaine d'expulsion pour conseil de concaténation de paramètre
surtout que l'ordre des paramètres n'importe pas sur les vrais sgbdr vu qu'il utilise le nom (sur access par contre il prend dans l'ordre et ca peut faire n'importe quoi)

? Si tu ne précises pas le nom des colonnes, l'ordre des paramètres importe... Ou alors il faudra m'expliquer comment il s'en sort pour affecter les bonnes valeurs aux bonnes colonnes !

[StringBuilder]

Est-tu sûr que "myCommand.Parameters["@FormuleEvaluation"].Value" correspond bien à ton paramètre @FormuleEvaluation ?

J'avoue que je ne gère jamais mes paramètres de cette manière, donc pas certain que l'accesseur pointe bien sur le nom.

Habituellement je fais :

Code csharp :

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

1
2
3
4
5
6
7
 
SqlParameter parFormuleEvaluation = new SqlParameter("@FormuleEvaluation", SqlDbType.VarChar, -1);
myCommand.Parameters.Add(parFormuleEvaluation);
 
// [...]
 
parFormuleEvaluation.Value = string.Concat(cmbNT.SelectedItem, "NT+", cmbNF.SelectedItem, "NF");

Accessoirement, ce sera plus rapide puisque je conserver un pointeur sur chacun de mes paramètres et donc je ne passe pas par un accesseur itératif à chaque appel à un des paramètres.

Bon, après, c'est vrai que c'est un peu plus verbeux.

[Msysteme]

Merci pour vos aides :
quand j'ai rectifié ma requête comme suit :

Code :

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

1
2
3
4
1
2
INSERT INTO table (nom_colonne_1, nom_colonne_2, ...
 VALUES ('valeur 1', 'valeur 2', ...)

ça a marché

[Pol63]

? Si tu ne précises pas le nom des colonnes, l'ordre des paramètres importe... Ou alors il faudra m'expliquer comment il s'en sort pour affecter les bonnes valeurs aux bonnes colonnes !

au temps pour moi je n'avais pu qu'il ne précisait pas le nom des colonnes
c'est surtout les 'value 1' concaténé dans la requête qui est déconseillé
et puis le problème est avant l'exécution de la requête à priori, et bien sur le parameter