[C#] missing operator query expression

[Erik888]

Ik heb een access database met gegevens van wedstrijden welke ik benader vanuit een C#-programma. Het openen en tonen van de diverse tabellen gaat prima, het updaten echter geeft problemen. Als ik bijvoorbeeld de poule wil bewerken dan wordt deze als volgt geopend:

C#:

dAdapterPoules = new OleDbDataAdapter("SELECT * FROM poule", connString);
OleDbCommandBuilder cBuilderPoules = new OleDbCommandBuilder(dAdapterPoules);
dTablePoules = new DataTable();
dAdapterPoules.Fill(dTablePoules);
dataGridPoules.DataSource = dTablePoules;

In de datagrid kunnen vervolgens diverse wijzigingen aangebracht worden, het eerste wat ik geprobeerd heb is:

C#:

dAdapterPoules.Update(dTablePoules);

Dit geeft "Syntax error in UPDATE statement".

Als ik de rowstate afvraag en de delete-, insert- en updatecommands samenstel dan krijg ik "Syntax error (missing operator) in query expression '@24sec Coach = @coach'.".
Moet ik per gewijzigde tabelregel de update aanroepen?
Iemand een idee wat ik verkeerd doe?

C#:

foreach (DataRow PouleRij in dTablePoules.Rows)
{
    if (PouleRij.RowState.ToString() == "Added")
    {
        dAdapterPoules.InsertCommand = new OleDbCommand("INSERT INTO poule, tijdsduur, [scheids thuis], [scheids uit], bondscheids, tafel, scorer, [30 sec], Coach, Trainer " +
            "VALUES (@poule, @tijdsduur, @scheids1, @scheids2, @bondscheids, @tafel, @scorer, @24sec, @coach, @trainer");
        dAdapterPoules.InsertCommand.Parameters.Add("@poule", OleDbType.Char, 50, PouleRij["poule"].ToString());
        dAdapterPoules.InsertCommand.Parameters.Add("@tijdsduur", OleDbType.Char, 50, PouleRij["tijdsduur"].ToString());
        dAdapterPoules.InsertCommand.Parameters.Add("@scheids1", OleDbType.Integer, 10, PouleRij["scheids thuis"].ToString());
        dAdapterPoules.InsertCommand.Parameters.Add("@scheids2", OleDbType.Integer, 10, PouleRij["scheids uit"].ToString());
        dAdapterPoules.InsertCommand.Parameters.Add("@bondscheids", OleDbType.Boolean, 1, PouleRij["bondscheids"].ToString());
        dAdapterPoules.InsertCommand.Parameters.Add("@tafel", OleDbType.Char, 50, PouleRij["tafel"].ToString());
        dAdapterPoules.InsertCommand.Parameters.Add("@scorer", OleDbType.Char, 50, PouleRij["scorer"].ToString());
        dAdapterPoules.InsertCommand.Parameters.Add("@24sec", OleDbType.Char, 50, PouleRij["30 sec"].ToString());
        dAdapterPoules.InsertCommand.Parameters.Add("@coach", OleDbType.Char, 50, PouleRij["Coach"].ToString());
        dAdapterPoules.InsertCommand.Parameters.Add("@trainer", OleDbType.Char, 50, PouleRij["Trainer"].ToString());
    }

    if (PouleRij.RowState.ToString() == "Deleted")
    {
        dAdapterPoules.DeleteCommand = new OleDbCommand("DELETE FROM poule WHERE poule = @poule");
        //dAdapterPoules.DeleteCommand.Parameters.Add("@poule", OleDbType.Char, 50, PouleRij["poule"].ToString());
    }

    if (PouleRij.RowState.ToString() == "Modified")
    {
        dAdapterPoules.UpdateCommand = new OleDbCommand("UPDATE poule SET tijdsduur = @tijdsduur, [scheids thuis] = @scheids1, " +
            "[scheids uit] = @scheids2, bondscheids = @bondscheids, tafel = @tafel, scorer = @scorer, [30 sec] = @24sec " +
            " Coach = @coach, Trainer = @trainer WHERE poule = @poule");
        dAdapterPoules.UpdateCommand.Parameters.Add("@tijdsduur", OleDbType.Char, 50, PouleRij["tijdsduur"].ToString());
        dAdapterPoules.UpdateCommand.Parameters.Add("@scheids1", OleDbType.Integer, 10, PouleRij["scheids thuis"].ToString());
        dAdapterPoules.UpdateCommand.Parameters.Add("@scheids2", OleDbType.Integer, 10, PouleRij["scheids uit"].ToString());
        dAdapterPoules.UpdateCommand.Parameters.Add("@bondscheids", OleDbType.Boolean, 1, PouleRij["bondscheids"].ToString());
        dAdapterPoules.UpdateCommand.Parameters.Add("@tafel", OleDbType.Char, 50, PouleRij["tafel"].ToString());
        dAdapterPoules.UpdateCommand.Parameters.Add("@scorer", OleDbType.Char, 50, PouleRij["scorer"].ToString());
        dAdapterPoules.UpdateCommand.Parameters.Add("@24sec", OleDbType.Char, 50, PouleRij["30 sec"].ToString());
        dAdapterPoules.UpdateCommand.Parameters.Add("@coach", OleDbType.Char, 50, PouleRij["Coach"].ToString());
        dAdapterPoules.UpdateCommand.Parameters.Add("@trainer", OleDbType.Char, 50, PouleRij["Trainer"].ToString());
    }
}

[ Voor 5% gewijzigd door een moderator op 04-03-2008 11:33 . Reden: Code tags toegevoegd... ]

[whoami]

Gebruik eens code-tags als je hier code-post, dat maakt de boel er heel wat leesbaarder op.

Als je in de MSDN eens leest wat de Update method van de DataAdapter doet, dan zal je lezen dat de DataAdapter dan dus voor iedere row het juiste command gaat gaan uitvoeren (UpdateCommand als de Rowstate Modified is, DeleteCommand als Rowstate Deleted is, etc... ).

Je hebt je Update command laten genereren door de command-builder; de foutmelding die je krijgt is dat er een syntax error zit in dat UPdate statement.
heb je al eens gekeken hoe dat Update Statement dat gegenereerd werd, er uit ziet ?

_{Trouwens, ik zie dat je de RowState gaat gaan omzetten naar een string, en dan gaat checken op die string. Waarom doe je dat ?
Da's helemaal niet 'veilig'. Wat als je een type-foutje maakt in de string waarop je checked ? Waarom vergelijk je gewoon niet met de verwachte enum ?}

Zowiezo zou ik nooit column-names / parameter-names maken / gebruiken waar spaties in zitten ...

[ Voor 24% gewijzigd door whoami op 04-03-2008 10:33 ]

[bigbeng]

Zit er toevallig een spatie in je parameternaam? Of ben je een komma tussen je SET onderdelen vergeten? De foutmelding is vrij helder.

[ Voor 11% gewijzigd door bigbeng op 04-03-2008 10:31 ]

[gorgi_19]

je mist een komma na @24sec

[Erik888]

Ik heb de fout al gevonden, ik mis een komma na @24sec bij de update. Ik krijg nu een "concurrency violation", dus daar ga ik maar even op zoeken.