Hi All,I am coidng a movie database using vb.ebt (VS 2013) and using access (2013) I am trying to code my save button, but every time i run the program and try to save my data I get the below error

An unhandled exception of type 'System.Data.OleDb.OleDbException' occurred in System.Data.dll

Additional information: Syntax error in INSERT INTO statement.

here is my code:

  If inc <> -1 Then

            Dim cb As New OleDb.OleDbCommandBuilder(da)
            Dim dsnewrow As DataRow
            dsnewrow = ds.Tables("Films").NewRow()
            dsnewrow.Item("Title") = txtTitle.Text
            dsnewrow.Item("Director") = txtDirector.Text
            dsnewrow.Item("Certificate") = cmbCertificate.Text
            dsnewrow.Item("Genre") = cmbGenre.Text
            dsnewrow.Item("Date") = dpDate.Text
            dsnewrow.Item("IMDB Link") = txtIMDB.Text
            dsnewrow.Item("Youtube Link") = txtYotube.Text
            da.Update(ds, "Films") 'this is where the error is
            MessageBox.Show("Record Added to Database")
        End If

Any help will be appreciated.

PS new to VB, so please go easy


Recommended Answers

All 13 Replies

Is there anything dodgy in any of those text fields?

Well in the date and genre fields these have values left in after the textboxes have been cleared. Would that make a difference?

I tried updating one text field "Title" and nothing else and still got the same error

I don't think so - the error is a syntax error in the insert statement.

Do you have any single apostrophes maybe?

No, I dont have an insert statement.There is nothing in teh guide i am using to say that i need to use an insert statement.

The code under the hood will be generating an insert statement and that is where the problem is. Do all of these dsnewrow.Item("Title") match database column names exactly - the spaces may be an issue?

Yes they do match the columns exactly. I have the delete button working perfectly, and it makes changesd to the access file,but that doesnt refernce particualr items, just deletes the entire record.

Sorry can't think of anything else then.

Thank you for your help, much appreciated

One last thing - if it's easy then change your table columns to have no spaces and change them in your code - worth a try if its easy to do.

does it mater that my program is using an OLEDB and ODBC?

Not sure - does any other database code work? If yes then it should be ok.

That is why I prefer to avoid the complicated method. Too many ways to bollux things up. Try this

Dim con As New OleDbConnection(your connection string here)
Dim cmd As New OleDbCommand("", con)

cmd.CommandText = "INSERT INTO Films " _
                & "  (Title, Director, Certificate, Genre, " _
                & "   [Date], [IMDB Link], [Youtube Link] " _
                & "VALUES(?, ?, ?, ?, ?, ?, ?)"

cmd.Parameters.AddWithValue("@parm1", txtTitle.Text)
cmd.Parameters.AddWithValue("@parm2", txtDirector.Text)
cmd.Parameters.AddWithValue("@parm3", cmbCertificate.Text)
cmd.Parameters.AddWithValue("@parm4", cmbGenre.Text)
cmd.Parameters.AddWithValue("@parm5", dpDate.Text)
cmd.Parameters.AddWithValue("@parm6", txtIMDB.Text)
cmd.Parameters.AddWithValue("@parm7", txtYotube.Text)

Dim result As Integer = cmd.ExecuteNonQuery()

Note the square brackets around fields with embedded blanks and reserved names like Date.

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.