Discussion :

[noobyyy]

et comment tu veux que je trouve la soluce ! j'ai pas de cours ? ta un lien ?


jviens de faire un point d'arret sur:

tout ce qui est la n'est pas traité

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
        private void dataGridView_attente_CellClick(object sender, DataGridViewCellEventArgs e)
        {
            if (e.RowIndex >= 0 && e.ColumnIndex == details.Index)
            {
                MySqlDataAdapter adapter_commande_attente = new MySqlDataAdapter("SELECT co_ref, co_fich, co_dat, co_qte, co_del, co_com, for_nom, gra_nom, pro_nom FROM commande,forma,grammage,produit WHERE co_for=for_id AND co_gra=gra_id AND co_pro=pro_id AND co_sta=0 ORDER BY co_dat DESC", maconnexion);
                // obtenir la DataRow correspondant à la ligne courante du DataGridView
                DataRowView drv = dataGridView_attente.CurrentRow.DataBoundItem as DataRowView;
                drv.Row["co_sta"] = 1;
                adapter_commande_attente.Update(new DataRow[] { drv.Row });
            }
        }

[tomlev]

non c'est sure mais comment tu veux que je trouve l'erreur, je ne sais pas quoi controler en faite

Ben ça suppose déjà que tu aies compris le code que tu as écrit jusqu'ici... et j'ai comme l'impression que ce n'est pas le cas

et comment tu veux que je trouve la soluce ! j'ai pas de cours ? ta un lien ?

Tu pourrais peut-être commencer par faire un tour ici...
http://dotnet.developpez.com/cours/

Et en particulier :
C# : http://tahe.developpez.com/dotnet/csharp/
ADO.NET : http://dotnet.developpez.com/articles/ado1/ et http://dotnet.developpez.com/articles/ado2/
ça concerne SQL Server, mais les principes sont valables pour tous les autres types de BDD (il suffit de remplacer Sql*** par MySql***...)

jviens de faire un point d'arret sur:

tout ce qui est la n'est pas traité

Il ne suffit pas de coller la fonction... il faut la lier à l'évènement CellClick du DataGridView, sinon évidemment elle ne sera jamais appelée. J'ai pas précisé parce que c'était évident. Si ça ne l'est pas pour toi, c'est que tu n'as pas les bases nécessaires pour faire ce que tu essaies de faire... donc commence par te former, parce que sinon tout ce que je te raconte ne sert à rien

[noobyyy]

ok j'ai oublié l'évènement car je ne suis pas habitué et dessuite je ne comprends pas ce code.

une fois le clique sur le bouton effectuer
j'ai une erreur:
La colonne 'co_sta' n'appartient pas à la table commande_attente.
a propos de la ligne ci-dessous:
drv.Row["co_sta"] = 1;


sinon pour les liens j'ai déja regardé il ya longtemps, surtout pour le PDF de 20mo !! que je connais très bien ! dommage qu'il ne traite pas ce dont j'ai besoin actuellement.

On me donne toujours des liens a aller voir, sauf que j'y suis déja aller! ce n'est pas toujours une référence et il y a pas ce que je cherche donc je viens poster ici.
je code depuis vraiment très peu en c# alors désolé une fois de plus d'etre un bon a rien.

On est tous passé par la difficulté...

[tomlev]

une fois le clique sur le bouton effectuer
j'ai une erreur:
La colonne 'co_sta' n'appartient pas à la table commande_attente.
a propos de la ligne ci-dessous:
drv.Row["co_sta"] = 1;

La colonne n'existe pas dans la table parce qu'elle n'est pas dans la requête SELECT du DataAdapter...

[matrix_ceg]

Je crois que le message est explicite.

tu dois avoir une erreur dans le nom de ta colonne.

[noobyyy]

effectivement j'avais pas mis: co_sta (merci)

erreur:
Update requiert un UpdateCommand valide lors du passage de la collection DataRow avec des lignes modifiées.


que signifie cette erreur ?

sur:

Code :

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

1
2
 
adapter_commande_attente.Update(new DataRow[] { drv.Row });

[matrix_ceg]

Code :

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

1
2
 
                MySqlDataAdapter adapter_commande_attente = new MySqlDataAdapter("SELECT co_ref, co_fich, co_dat, co_qte, co_del, co_com FROM commande WHERE co_sta=0 ORDER BY co_dat DESC", maconnexion);

donc col sta n'est pas présente vu que dans ton select tu ne la reprends pas.
La colonne dans la clause where n'est pas obligatoirement importé.

[noobyyy]

ouaip jlai zapé

[matrix_ceg]

effectivement j'avais pas mis: co_sta (merci)

erreur:
Update requiert un UpdateCommand valide lors du passage de la collection DataRow avec des lignes modifiées.


que signifie cette erreur ?

sur:

Code :

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

1
2
 
adapter_commande_attente.Update(new DataRow[] { drv.Row });

Que la requête auto généré ne correspond pas aux données que tu lui a donné.

ne t'embête pas a choisir la ligne, renvoie le dataset, il ne traitera que ce qui a été modifié.

[noobyyy]

je dois faire une requete update ?
en utilisant:
UpdateCommand c'est sa ?

[tomlev]

Update requiert un UpdateCommand valide lors du passage de la collection DataRow avec des lignes modifiées.


que signifie cette erreur ?

Arf... c'est parce que tu n'as pas défini UpdateCommand/DeleteCommand/InsertCommand dans le DataAdapter. Mais normalement le MySqlCommandBuilder est supposé gérer ça...
Ca marche si tu remplace la ligne par ça ?

Code :

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

adapter_commande_attente.Update(dataset_commande, "commande_attente");

Que la requête auto généré ne correspond pas aux données que tu lui a donné.

Non, ça veut plutôt dire qu'aucune requête n'a été auto-générée... et c'est bien ça qui me semble bizarre

[noobyyy]

j'ai du rajouté le dataset d'en haut pour pouvoir l'utiliser, je ne sais pas comment faire pour pourvoir l'utiliser sur toutes mes méthodes.

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
 
        private void dataGridView_attente_CellClick(object sender, DataGridViewCellEventArgs e)
        {
            if (e.RowIndex >= 0 && e.ColumnIndex == details.Index)
            {
                MySqlDataAdapter adapter_commande_attente = new MySqlDataAdapter("SELECT co_ref, co_fich, co_dat, co_qte, co_del, co_com, for_nom, gra_nom, pro_nom, co_sta FROM commande,forma,grammage,produit WHERE co_for=for_id AND co_gra=gra_id AND co_pro=pro_id AND co_sta=0 ORDER BY co_dat DESC", maconnexion);
                DataSet dataset_commande = new DataSet();
                adapter_commande_attente.Fill(dataset_commande, "commande_attente");
 
                // obtenir la DataRow correspondant à la ligne courante du DataGridView
                DataRowView drv = dataGridView_attente.CurrentRow.DataBoundItem as DataRowView;
                drv.Row["co_sta"] = 1;
                       adapter_commande_attente.Update(dataset_commande, "commande_attente");
 
                MessageBox.Show("Commande terminée !", "Validation", MessageBoxButtons.OK, MessageBoxIcon.Information);
            }
        }

ok pas de message d'erreur mais l'update ne s'effectue pas

sinon le faite d'avoir mis co_sta dans le select sa me gene car il affiche une colonne co_sta alors que jle veux pas

[tomlev]

ok pas de message d'erreur mais l'update ne s'effectue pas

Ben c'est normal... tu ne réutilises pas la même DataTable, tu en crées une autre, qui, elle, n'est pas modifiée...

Il faut que tu déclares le dataset comme membre de la classe, et non pas comme variable locale dans liste_Load

[noobyyy]

bah ouai mais la je suis perdu car j'ai mis du code dans le load car c'est au chargement de la fenêtre qu'il y a des choses a traiter si je le change de place comment je vais dire que mon code s'exécute au démarrage de la fenêtre ?

membre de la classe c'est a dire ?

[tomlev]

bah ouai mais la je suis perdu car j'ai mis du code dans le load car c'est au chargement de la fenêtre qu'il y a des choses a traiter si je le change de place comment je vais dire que mon code s'exécute au démarrage de la fenêtre ?

J'ai seulement dit de sortir la déclaration du Load... l'initialisation, tu peux la laisser dedans

membre de la classe c'est a dire ?

Y a quand même des limites... si tu as vraiment lu (et compris) le cours de Serge Tahé, tu devrais savoir ce que c'est. Fais une petite recherche... Je vais pas te faire un cours sur la programmation orientée objet, le forum ne sert pas à ça

[noobyyy]

membre de la classe a l'interieur quoi ? tu veux dire...


sinon je fais les traitements dans le load et les déclarations a l'exterieur

[tomlev]

membre de la classe a l'interieur quoi ? tu veux dire...

euh... je comprends pas ta phrase

sinon je fais les traitements dans le load et les déclarations a l'exterieur

c'est ça

[noobyyy]

déclarations à l'exterieur:

Code :

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

1
2
3
 
        public DataSet dataset_commande = new DataSet();
        public DataSet dataset_termine = new DataSet();

ce qui me donne d'une part:

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
22
23
24
25
26
27
28
29
30
31
 
        private void liste_Load(object sender, EventArgs e)
        {
            try
            {
                timer_reload.Enabled = true;
 
                // Connexion à la base de données
                string connexion = "Server=IP;Port=3306;Database=db_commande;Uid=PSEUDO;Pwd=MDP;";
                maconnexion = new MySqlConnection(connexion);
                maconnexion.Open(); // Ouverture
 
                MySqlDataAdapter adapter_commande_attente = new MySqlDataAdapter("SELECT co_ref, co_fich, co_dat, co_qte, co_del, co_com, co_sta, co_fac, co_imp, for_nom, gra_nom, pro_nom FROM commande,forma,grammage,produit WHERE co_for=for_id AND co_gra=gra_id AND co_pro=pro_id AND co_sta=0 ORDER BY co_dat DESC", maconnexion);
                MySqlDataAdapter adapter_commande_termine = new MySqlDataAdapter("SELECT co_ref, co_fich, co_dat, co_qte, co_del, co_com, co_sta, co_fac, co_imp, for_nom, gra_nom, pro_nom FROM commande,forma,grammage,produit WHERE co_for=for_id AND co_gra=gra_id AND co_pro=pro_id AND co_sta=1 ORDER BY co_dat ASC LIMIT 10", maconnexion);
 
                MySqlCommandBuilder builder = new MySqlCommandBuilder(adapter_commande_attente);
 
 
                adapter_commande_attente.Fill(dataset_commande, "commande_attente");                
                adapter_commande_termine.Fill(dataset_commande, "commande_termine");
 
                DataView MonDataView = dataset_commande.Tables["commande_attente"].DefaultView;
                dataGridView_attente.DataSource = MonDataView;
                DataView MonDataView2 = dataset_commande.Tables["commande_termine"].DefaultView;
                dataGridView_termine.DataSource = MonDataView2;                
            }
            catch (Exception)
            {
                MessageBox.Show("Connexion impossible avec le serveur, veuillez vérifier son état !", "Erreur", MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
        }

et d'autre part:

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
 
        private void dataGridView_attente_CellClick(object sender, DataGridViewCellEventArgs e)
        {
            if (e.RowIndex >= 0 && e.ColumnIndex == details.Index)
            {
                MySqlDataAdapter adapter_commande_attente = new MySqlDataAdapter("SELECT co_ref, co_fich, co_dat, co_qte, co_del, co_com, co_fac, co_imp, for_nom, gra_nom, pro_nom, co_sta FROM commande,forma,grammage,produit WHERE co_for=for_id AND co_gra=gra_id AND co_pro=pro_id AND co_sta=0 ORDER BY co_dat DESC", maconnexion);
                adapter_commande_attente.Fill(dataset_commande, "commande_attente");
 
                // obtenir la DataRow correspondant à la ligne courante du DataGridView
                DataRowView drv = dataGridView_attente.CurrentRow.DataBoundItem as DataRowView;
                drv.Row["co_sta"] = 1;
 
                adapter_commande_attente.Update(dataset_commande, "commande_attente");
                MessageBox.Show("Commande terminée !", "Validation", MessageBoxButtons.OK, MessageBoxIcon.Information);
            }
        }

ERREUR:
Update requiert un UpdateCommand valide lors du passage de la collection DataRow avec des lignes modifiées.

[B.AF]

C'est peut être un peu normal, vu que tu n'as positionné qu'un select ???

Faudrait peut être vu ta requête spécifier la condition update...

Extrait de la doc mysql :

During System.Data.Common.DataAdapter.Update, if this property is not set and primary key information is present in the DataSet, the UpdateCommand can be generated automatically if you set the SelectCommand property and use the MySqlCommandBuilder.Then, any additional commands that you do not set are generated by the MySqlCommandBuilder. This generation logic requires key column information to be present in the DataSet.

T'aurais pas un problème de clef par hazard ?

[tomlev]

argh...
j'avais pas vu, ta requête c'est une jointure entre plusieurs tables
Dans ce cas le CommandBuilder ne marche pas, il faut que tu écrives toi-même les requêtes de mise à jour...

Si tu ne comptes mettre à jour que le statut, et que tu n'insères ou supprimes jamais de lignes, il suffit de définir une UpdateCommand, avec des paramètres qui correspondent aux champs de la DataTable. Ca donne quelque chose comme ça (d'après ton premier post) :

Code :

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

1
2
3
4
5
MySqlCommand cmd = maconnexion.CreateCommand();
cmd.CommandText = "update commande set co_sta=@co_sta where co_ref=@co_ref";
cmd.Parameters.Add("@co_sta", MySqlDbType.Int32, 0, "co_sta");
cmd.Parameters.Add("@co_ref", MySqlDbType.String, 0, "co_ref");
adapter_commande_attente.UpdateCommand = cmd;

(à ajouter après la création du dataAdapter, à la place du MySqlCommandBuilder)

déclarations à l'exterieur:

Code :

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

1
2
3
 
        public DataSet dataset_commande = new DataSet();
        public DataSet dataset_termine = new DataSet();

Pas la peine de faire 2 datasets, tu peux mettre 2 DataTables dans le même DataSet. D'ailleurs, vu que la seule différence entre les 2 est le statut, il vaudrait mieux tout charger dans la même DataTable, et à l'affichage, filtrer selon le statut (à l'aide de DataViews)