[C# - Winforms] Update Database

[Nexopheus]

Ik krijg het maar niet voorelkaar om de veranderingen die de gebruiker gemaakt heeft in een datagrid door te voeren in de database.

Ik gebruik de volgende code:

C#:

OleDbCommand selectCmd = new OleDbCommand("select * from " + listBox1.Text, db.connection);
dataAdapter = new OleDbDataAdapter();
dsTables = new DataSet();
dataAdapter.FillSchema(dsTables,SchemaType.Source,listBox1.Text);
dataAdapter.Fill(dsTables, this.database);
dataGrid1.DataSource = dsTables.Tables[this.database].DefaultView;
OleDbCommandBuilder custCB = new OleDbCommandBuilder(dataAdapter);
dataAdapter.UpdateCommand = custCB.GetUpdateCommand();

Verderop in de code ga ik updaten:

C#:

if (dsTables.HasChanges())
            {
                try
                {
                    DataSet changedData = dsTables.GetChanges();                                                
                    int numRows = dataAdapter.Update(changedData, listBox1.Text);                                   
                    MessageBox.Show(numRows + " rows changed");         
                    
                }
                catch(Exception ex)
                {
                    MessageBox.Show(ex.Message);
                }


                
            }       

Wat ik ook doe, elke keer wordt er niet geupdate, terwijl er toch degelijk veranderingen zijn.

Ik krijg geen errors of exceptions

Iemand een idee?

[Nexopheus]

^Kick^

[whoami]

Hoe ziet je updatecommand eruit?
Hoe zit het met de parameters van dat updatecommand , zijn ze aan de juiste columns enzo gebinded?

[Nexopheus]

Volgens mij wel. Ik genereer ze via de OleDbCommandBuilder.
Ik heb de selectCommand gezet. Volgens de documentatie worden de update, insert en delete commando's dan automatisch gebruikt.

Zal eens een breakpoint op de update zetten, eve kijken hoe die update eruit ziet.
[ik krijg overigens geen error !]

[whoami]

Ga je mij nu vertellen dat je zelf nog niet eerst eens even de moeite hebt genomen om het geheel eens te debuggen?

[Nexopheus]

Niet helemaal.

De update query is een parameterized query. Ten minste op mijn gezette breakpoint. Wanneer ik verder stap, gaat het programma gewoon door. geen exception of iets dergelijks.

Ik ging er een beetje vanuit dat de ADO.NET functies er voor zorgen dat de juiste waarden ingevuld zullen worden.

VB:

code:

UPDATE overzicht SET azg = ? , afd no = ? , diagnose = ? , gelst = ? , CPK/neurol = ? , biopt = ? , familiair = ? , ernst = ? , linkage = ? , LMNA = ? , DSG = ? WHERE ( (Id = ?) AND ((? = 1 AND azg IS NULL) OR (azg = ?)) AND ((? = 1 AND afd no IS NULL) OR (afd no = ?)) AND ((? = 1 AND diagnose IS NULL) OR (diagnose = ?)) AND ((? = 1 AND gelst IS NULL) OR (gelst = ?)) AND ((? = 1 AND CPK/neurol IS NULL) OR (CPK/neurol = ?)) AND ((? = 1 AND biopt IS NULL) OR (biopt = ?)) AND ((? = 1 AND familiair IS NULL) OR (familiair = ?)) AND ((? = 1 AND ernst IS NULL) OR (ernst = ?)) AND ((? = 1 AND linkage IS NULL) OR (linkage = ?)) AND ((? = 1 AND LMNA IS NULL) OR (LMNA = ?)) AND ((? = 1 AND DSG IS NULL) OR (DSG = ?)) )

Ik kan niet bij het commando komen op het exacte moment dat het uitgevoerd wordt.

Ik zal nog wat meer debug uurtjes maken

[ Voor 46% gewijzigd door Nexopheus op 02-09-2003 15:24 ]

[whoami]

Normaal gezien moeten de parameters van je query 'gebinded' worden aan een source-column. (Bekijk de constructor van een OleDbParameter eens). Aangezien je gebruik maakt vn die commandbuilder zou ik ook vermoeden dat dit automatisch moet gebeuren.

Kan het te maken hebben met het feit dat je select * doet ? (* whoami roept maar wat)

[Nexopheus]

Je bedoelt ipv * alle kolomnamen van de database?

[LoekD]

Kijk eens naar dataAdapter.UpdateCommand.CommandText,
wat staat daar zoal in?
Als je het daar niet mee eens bent kun je die overschrijven met je eigen statement.

En heb je niet p.o. dsTables.AcceptChanges() aangeroepen voordat je je update gedaan hebt?

[Nexopheus]

die text (dus updatecommand.commandtext) heb ik hierboven al geplaatst.
het gaat er juist op dat het dynamisch gebeurt. Ik heb geen idee wat voor tabel er geladen wordt. Dus handmatig aanpassen is niet echt een optie

[LoekD]

Zoals je ziet is het gegenereerde Update statement niet goed.
Dat kan betekenen dat je listBox1.Text leeg is op het moment dat het OleDbCommand object gemaakt wordt.
Vul daar eens een bestaande tabelnaam in.

[Nexopheus]

Daar staat al een bestaande tabelnaam in. De datagrid wordt gevuld adhv de waarde in deze listbox.

[LoekD]

ohja.. en:
After the SQL statement is first generated, the application must explicitly call RefreshSchema if it changes the statement in any way. Otherwise, the GetUpdateCommand will be still be using information from the previous statement, which might not be correct. The SQL statements are first generated either when the application calls Update or GetUpdateCommand.

[ bron: ms-help://MS.NETFrameworkSDKv1.1/cpref/html/frlrfsystemdataoledboledbcommandbuilderclassgetupdatecommandtopic.htm ]

[Nexopheus]

mmm, zal deze eens proberen

Is enkel nodig wanneer de select statement is gewijzigd.
--> toch niet, Beter zoeken in de help.

Remarks
An application should call RefreshSchema whenever the SELECT statement associated with the OleDbCommandBuilder changes.

An application should call RefreshSchema whenever the SelectCommand value of the OleDbDataAdapter changes.

bron: ms-help://MS.VSCC.2003/MS.MSDNQTR.2003FEB.1033/cpref/html/frlrfSystemDataOleDbOleDbCommandBuilderClassRefreshSchemaTopic.htm

[ Voor 107% gewijzigd door Nexopheus op 02-09-2003 17:09 ]

[Nexopheus]

Krijg nu in iedergeval een syntax error exception. En er komt ook een mooi rood kruisje in de datagrid te staan. De query is helemaal niet goed, lijkt helemaal nergens naar. Nog steeds die rare vraag tekens.

Snap er helemaal niets van.....!

Dan toch maar de command objecten zelf maken.

code:

"UPDATE Tabel1 SET famno = ? , overleden = ? , afd no = ? , diagnose = ? , gelst = ? , CPK/neurol = ? , biopt = ? , familiair = ? , ernst = ? , DNA? = ? , linkage = ? , LMNA = ? , delta SG = ? , Veld3 = ? , Veld2 = ? , Veld1 = ? , genealogie = ? WHERE ( (Id = ?) AND ((? = 1 AND famno IS NULL) OR (famno = ?)) AND ((? = 1 AND overleden IS NULL) OR (overleden = ?)) AND ((? = 1 AND afd no IS NULL) OR (afd no = ?)) AND ((? = 1 AND diagnose IS NULL) OR (diagnose = ?)) AND ((? = 1 AND gelst IS NULL) OR (gelst = ?)) AND ((? = 1 AND CPK/neurol IS NULL) OR (CPK/neurol = ?)) AND ((? = 1 AND biopt IS NULL) OR (biopt = ?)) AND ((? = 1 AND familiair IS NULL) OR (familiair = ?)) AND ((? = 1 AND ernst IS NULL) OR (ernst = ?)) AND ((? = 1 AND DNA? IS NULL) OR (DNA? = ?)) AND ((? = 1 AND linkage IS NULL) OR (linkage = ?)) AND ((? = 1 AND LMNA IS NULL) OR (LMNA = ?)) AND ((? = 1 AND delta SG IS NULL) OR (delta SG = ?)) AND ((? = 1 AND Veld3 IS NULL) OR (Veld3 = ?)) AND ((? = 1 AND Veld2 IS NULL) OR (Veld2 = ?)) AND ((? = 1 AND Veld1 IS NULL) OR (Veld1 = ?)) AND ((? = 1 AND genealogie IS NULL) OR (genealogie = ?)) )"

Hieboven nogmaals de update sql. Bagger is het.

[ Voor 67% gewijzigd door Nexopheus op 02-09-2003 17:22 ]

[Nexopheus]

Of moet ik zelf nog eerst de parameter waardes invoeren ?

[Nexopheus]

Het is opgelost hoor!!!!
Deze regeltjes doen het!

code:

OleDbCommandBuilder custCB = new OleDbCommandBuilder(custDA);
custCB.QuotePrefix = "[";
custCB.QuoteSuffix = "]";

Raar maar waar!

[whoami]

En kan je er ook nog ff wat meer uitleg bij geven?

Ah, ik zie het al.... Het probleem heeft te maken met het feit dat jij spaties in je veldnamen zitten hebt.
Eigenlijk is dat een bad practice. Nooit spaties in je columnnames gebruiken. Gebruik dan underscores ofzo.

[ Voor 68% gewijzigd door whoami op 02-09-2003 21:44 ]

[Nexopheus]

Ik wou dat ik dat kon!

Ik heb nog maar eens gezocht in de VS.NET Help files. Hier kwam ik een stukje tegen met de titel:"
"Automatically Generated Commands" Alles wat hierin stond deed ik dus al, behalve die prefix en suffix niet.

Ik heb vervolgens een nieuwe applicatie gemaakt waarbij ik alles letterlijk over heb getyped.
zoals het in de help stond:
zie hieronder voor de code

C#:

OleDbDataAdapter custDA;
        OleDbConnection nwindConn;
        DataSet custDS = new DataSet();
        private void button1_Click(object sender, System.EventArgs e)
        {
            nwindConn  = new OleDbConnection("Provider=Microsoft.Jet.OLEDB.4.0;Data Source=C:\\DCM.mdb");
            nwindConn.GetOleDbSchemaTable(
            custDA = new OleDbDataAdapter("SELECT * FROM Tabel1",nwindConn);
            OleDbCommandBuilder custCB = new OleDbCommandBuilder(custDA);
            custCB.QuotePrefix = "[";
            custCB.QuoteSuffix = "]";
            
            nwindConn.Open();           
            custDA.Fill(custDS,"Tabel1");                       
            
            dataGrid1.DataSource = custDS;
            dataGrid1.DataMember = "Tabel1";    


        }

        private void button2_Click(object sender, System.EventArgs e)
        {
            custDA.Update(custDS,"Tabel1");         
            nwindConn.Close();          
        }

Bovenstaande code heb ik een beetje aangepast nadat ik bewijs had dat het werkte.
Wanneer ik de prefix en suffix weghaal, lukt het niet meer.

De query heeft nu de volgende vorm:

code:

"UPDATE [Tabel1] SET [famno] = ? , [overleden] = ? , [afd no] = ? , [diagnose] = ? , [gelst] = ? , [CPK/neurol] = ? , [biopt] = ? , [familiair] = ? , [ernst] = ? , [DNA?] = ? , [linkage] = ? , [LMNA] = ? , [delta SG] = ? , [Veld3] = ? , [Veld2] = ? , [Veld1] = ? , [genealogie] = ? WHERE ( ([Id] = ?) AND ((? = 1 AND [famno] IS NULL) OR ([famno] = ?)) AND ((? = 1 AND [overleden] IS NULL) OR ([overleden] = ?)) AND ((? = 1 AND [afd no] IS NULL) OR ([afd no] = ?)) AND ((? = 1 AND [diagnose] IS NULL) OR ([diagnose] = ?)) AND ((? = 1 AND [gelst] IS NULL) OR ([gelst] = ?)) AND ((? = 1 AND [CPK/neurol] IS NULL) OR ([CPK/neurol] = ?)) AND ((? = 1 AND [biopt] IS NULL) OR ([biopt] = ?)) AND ((? = 1 AND [familiair] IS NULL) OR ([familiair] = ?)) AND ((? = 1 AND [ernst] IS NULL) OR ([ernst] = ?)) AND ((? = 1 AND [DNA?] IS NULL) OR ([DNA?] = ?)) AND ((? = 1 AND [linkage] IS NULL) OR ([linkage] = ?)) AND ((? = 1 AND [LMNA] IS NULL) OR ([LMNA] = ?)) AND ((? = 1 AND [delta SG] IS NULL) OR ([delta SG] = ?)) AND ((? = 1 AND [Veld3] IS NULL) OR ([Veld3] = ?)) AND ((? = 1 AND [Veld2] IS NULL) OR ([Veld2] = ?)) AND ((? = 1 AND [Veld1] IS NULL) OR ([Veld1] = ?)) AND ((? = 1 AND [genealogie] IS NULL) OR ([genealogie] = ?)) )"

Blijkbaar zijn de haken nodig voor verder verwerking...

[ Voor 3% gewijzigd door Nexopheus op 02-09-2003 21:51 ]

[whoami]

Die haken zijn nodig omdat -zoals ik al zei- je spaties hebt in je column-names.
Als je die niet hebt, dan heb je die haken niet nodig.

(Vergelijk het met de backticks in MySQL)

[ Voor 15% gewijzigd door whoami op 02-09-2003 22:42 ]

[Nexopheus]

Justum, weer wat wijzer.