Datagridview vers MS SQL

Version imprimable

Bonjour,

Une idée pourquoi mon code ne fonctionne pas.
Aprés le clique sur le bouton de mon FORM, ce code n'enregistre rien dans ma table SQL..
En mode debug, toutes les "Values" ont bien les valeurs attendues..

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 cnn1.Open()
 
            Dim Cmd As New Data.SqlClient.SqlCommand
            For Each rw As DataGridViewRow In DG_PANIER.Rows
                Cmd = New SqlCommand("INSERT INTO DBO.COMMANDE " & _
                                     "([NUM_CMD],[NOMENCL],[DESIGNATION],[UAP],[USER_WS_MATR],[USER_WS_NOM],[SECTEUR],[QTE_CMD],[DATE_CMD] " & _
                                     ",[DATE_CMD_FERME],[PRIORITE],[USER_ADM],[ETAT_CMD],[MOTIF_ETAT],[COMMENTAIRE_TO_LOGI],[COMMENTAIRE_TO_USER] " & _
                                     ",[ZONE_QUAI],[QTE_LIVRE],[RELIQUAT],[STOCKA0],[TOUT_DISPO],[DEL]) " & _
                                    " VALUES ('" & L_ID_CMD.Text & "','" & rw.Cells(0).Value & "', '" & rw.Cells(1).Value & "', '" & LB_UAP.SelectedItem & "', '" & TB_MATR.Text & "', '" & L_UserName.Text & "', '" & LB_SECTION.SelectedItem & "','" & rw.Cells(4).Value & "', GETDATE() " & _
                                    ", '', '" & L_TYPE_CMD.Text & "', '', 'Ouverte', '', '', '' " & _
                                    ", '" & ChaineCochee() & "', '0','','','','0') ", cnn1)
 
 
            Next
            Cmd.ExecuteNonQuery()
            cnn1.Close()

MERCI ! :aie:

13/07/2017, 14h23
J@ckHerror

Bonjour,

ça pique toujours autant les yeux les requêtes concat :aie: ... Je te proposerais bien de corriger ton problème en profitant des avantages des requêtes paramétrées mais bon ... (oui pour moi tu as 9 changes sur 10 pour que ce soit un problème de typage)

De plus tu dis que ce code n'enregistre rien ... ok mais le message d'erreur qui va avec :roll:

J@ck.
13/07/2017, 14h29
_PascalC_

Cmd.ExecuteNonQuery() n'est pas dans ta boucle.
Tu devrais cependant avoir au moins la dernière ligne dans ta table.

Je te conseille très fortement de consulter les liens ci-dessous :
.Net, SQL Server, et les requêtes paramétrées (C#)
Injection SQL

Formatée de cette façon, la simple saisie d'une quotte par l'utilisateur fera planter ta requête.
Ça t'évitera aussi de gros problèmes de sécurité, typage... etc
13/07/2017, 14h29
guilld

Je viens de trouver !!!

Alors j'avais pas de message d'erreur, parce que j'avais oublié le MSG du CATCH ! donc avec lui j'ai bien eu le message : d'erreur "Echec de conversion INT en VARCHAR".
Ca fonctionne maintenant.

Par contre tu dis que ca pique les yeux.. je veux bien te croire mais quel avantage (hors mis que ca pique plus), de passer des parametres dans la fonction si comme ca c'est tout aussi efficace ?

tu peux montrer ca ? Merci :)
13/07/2017, 14h38
guilld

Merci les gars ! :)
13/07/2017, 14h45
J@ckHerror

Citation:

Envoyé par guilld

Je viens de trouver !!!

Par contre tu dis que ca pique les yeux.. je veux bien te croire mais quel avantage (hors mis que ca pique plus), de passer des paramétrés dans la fonction si comme ca c'est tout aussi efficace ?

tu peux montrer ca ? Merci :)

Suis les liens que _PascalC_ t'as donné.

Les avantages principaux avantages pour moi sont : La sécurité (voir les liens de _PascalC_), les soucis de typages puisque le connecteur se chargera de transtyper la valeur si elle n'est pas dans un bon type... mais alors surtout, pour moi, la seule, la vrai raison c'est que ca prend pas plus de temps ni de code :furieux: je ne comprendrais donc jamais pourquoi les débutants sont si réticent :zoubi: ! Moi on me dit c'est la bonne pratique, c'est facile est rapide, j'en ai pas besoin de plus pour me lancer :D ...

J@ck.
13/07/2017, 15h07
guilld

..
13/07/2017, 15h22
J@ckHerror

Citation:

Envoyé par guilld

bon ok il en reste, mais pas le choix puisqu'il s'agit de liste de choix. Donc non modifiable ...

Aucun rapport avec la choucroute ! 8O
Je vois pas ce qu'il t'empêche de faire comme pour les 4 que tu as fais correctement...

De plus, si tu voulais encore t'améliorer tu pourrais passer par des classes, faire une vrai séparation des couches....

J@ck.