Discussion :

[Dadoue]

Bonsoir à toutes et à tous!

Je travaille actuellement sur l'élaboration d'un logiciel capable de générer des devis et des factures à travers l'utilisation de dataGridView.Je dispose pour cela d'une base de données ACCESS...

Je m'interroge de plus en plus sur la possibilité d'incorporer une instruction INSERT dans une boucle for.

• -A partir d'un 1er datagridview nommé dataGridView1, j'affiche les produits que l'entreprise vends aux particulier, et cela grace à la table "SERVICES" de ma BDD. Ces produits constituent ainsi les produits que le client se voit facturer.
  
• -A laide d'un bouton "ajouter" j'utiliser les instructions INSERT afin d'inserer mes produits dans les tables FACTURER(NumFacture,NumService,qte)

Et avec l'aide d'autre tables, j'établi une facture comprenant les services vendus et autres choses diverses et variées.

En fait ce que je voudrais faire est la chose suivante:

a partir d'un deuxième datagridview comprenant les produits séléctionnés dans le datagridview1, je cherche a enregistrer les produits dans ma table facturer grace a une boucle for qui parcours mon datagridview2.

voici le code:

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
18
19
20
21
for (i = 0; i < (dataGridView2.Rows.Count - 1); i++)
                {
                    service = Convert.ToString(dataGridView2[1, i].Value);//Nom du produit
                    qte = Convert.ToInt16(dataGridView2[3, i].Value);//qté commandée
 
                    OleDbCommand sel = new OleDbCommand("SELECT NumService FROM SERVICES WHERE NomService='" + service + "'", co);
                    ns = Convert.ToInt16(sel.ExecuteScalar());//récupération du Numéro de produit correspondant au nom du produit correspondant
 
                    OleDbCommand insFacrer = new OleDbCommand("INSERT INTO FACTURER VALUES(@nf,@ns,@qte)", co);
                    insFacrer.Parameters.Add("@nf", OleDbType.Integer);//Numéro de facture
                    insFacrer.Parameters.Add("@ns", OleDbType.Integer);//Numéro de service
                    insFacrer.Parameters.Add("@qte", OleDbType.Integer);//quantité facturée pour le client
 
                    insFacrer.Parameters["@nf"].Value = fact2;
                    insFacrer.Parameters["@ns"].Value = ns;
                    insFacrer.Parameters["@qte"].Value = qte;
 
                    insFacrer.ExecuteNonQuery();
                    Da.InsertCommand = insFacrer;
                    co.Close();
                }

Le résultat est que cela ne fait rien du tout... Je me creuse la tête depuis plusieurs heures sans pouvoir trouver la solution...

Peut-être est-ce un problème d'instruction INSERT, mais je ne vois pas où il pourrait-être...

Quand a la possibilité d'utiliser un foreach, je n'est pas réussi a bien me servir de cette expression...

En espérant une réponse rapide,
je vous remercie d'avance.

[tomlev]

Je ne vois rien de fondamentalement incorrect dans ton code... à part peut-être cette ligne dont je ne vois pas l'utilité :

Code :

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

Da.InsertCommand = insFacrer;

Mais ce n'est sûrement pas ça qui cause le problème.

Le fait que ce soit dans une boucle for n'a a priori aucun rapport non plus avec le problème.

Est-ce que tu ne serais pas dans une transaction que tu n'aurais pas commité ?

Est-ce que ta base de données appartient au projet ? Si oui, vérifie qu'il n'y a pas l'option "Copier dans le répertoire de sortie" réglée sur "Toujours copier". Si c'est le cas, la base dans le répertoire de travail est écrasée à chaque recompilation...

Sinon, tu as essayé d'exécuter en pas-à-pas ? Regarde la valeur de retour de ExecuteNonQuery, ça indique le nombre de lignes insérées.

EDIT: Ah si, y a un truc louche quand même : tu fermes la connexion à chaque itération... donc a priori seule la première requête devrait passer, les autres devraient déclencher une exception. Si ce n'est pas le cas, c'est probablement que tu ne rentres jamais dans la boucle...

[Graffito]

Le résultat est que cela ne fait rien du tout...

Dans la base ou dans ton IHM ?

Quels sont les champs de la table FACTURER ?

[SetaSensei]

Bonjour,

Je vais partir du postulat que vous ne voyez pas l'insertion dans votre appli. Car comme l'a indiqué Graffito, vous pouvez avoir plusieurs types d'erreur vu le message que vous avez posté.

Je suppose que vous utilisez un DataSet pour populariser le DataSource de votre DataGridView. Si vous fermez votre connexion une fois le DataSet rempli (j'espère que c'est ce que vous faites ), les données ne sont pas mises à jour sur le poste client.
Du coup, si vous effectuez des modifications sur la BDD mais que votre DataSource n'est pas rafraîchi, vous ne voyez pas le résultat de votre requête.
Il faut donc relancer la population de votre DataGridView depuis une nouvelle connexion à la BDD.
Si vous ne voyez pas de résultat en base non plus, le problème est autre. Notamment le :

Code :

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

co.Close()

qui empêche, comme l'a dit tomlev. Mais normalement vous auriez dû enregistrer une ligne pour chaque facture.

[tomlev]

Je suppose que vous utilisez un DataSet pour populariser le DataSource de votre DataGridView

Il me semble que c'est une traduction un peu foireuse du terme anglais "populate"
J'aurais dis "peupler", ou "remplir", tout simplement


Sinon c'est une théorie qui se tient, j'y avais pas pensé... on va voir si le PO confirme

[SetaSensei]

Il me semble que c'est une traduction un peu foireuse du terme anglais "populate"
J'aurais dis "peupler", ou "remplir", tout simplement


Sinon c'est une théorie qui se tient, j'y avais pas pensé... on va voir si le PO confirme

Désolé, pas encore pris mes 2 litres de café...

[Dadoue]

Bonjour a tous,

Merci a tous d'avoir répondu a mon problème aussi vite!

Donc, j'ai bien regardé, et Tonlev avait raison, mais partiellement... Une étourderie s'étais glissé dans mon code, mais vous ne pouviez pas le savoir...

Alors, certes, le co.close() est un problème , ainsi donc, je l'ai sorti de ma boucle,avec le co.Open(). Plus aucun souci a se faire de ce coté-ci.

Ma principale erreur, que je n'ai pas vu hier, (evidemment avec 5 heures de boulot non-stop l'après midi, on commence a faire un peu n'importe quoi avec son code...) était le fait que je ne piochait pas les bonnes valeurs dans mon dataGridView2. En fait, je me suis trompé de colonne, ce n'était pas:

Code :

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

1
2
service = Convert.ToString(dataGridView2[1, i].Value);
                    qte = Convert.ToInt16(dataGridView2[3, i].Value);

mais bien:

Code :

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

1
2
service = Convert.ToString(dataGridView2[2, i].Value);
                    qte = Convert.ToInt16(dataGridView2[4, i].Value);

Donc, pour ne pas me faire culpabiliser de vous avoir fait perdre votre temps, je vous remercie sincèrement de m'avoir aidé dans la recherche de mes erreurs...

Vous me sauvez la vie vous savez?
Bonne journée a vous tous! Et soyez fort!