Results 1 to 2 of 2

Thread: Updating database problem

  1. #1

    Thread Starter
    Lively Member
    Join Date
    Jun 2009
    Posts
    70

    Updating database problem

    Hi, I am using the following code to insert record into my database but I keep getting the "Error while inserting record on table...Invalid column name 'xxx'." but when I tried to issue the command in SQL server:

    INSERT Movie_Table VALUES(100, 'Avatar');

    I am okay.

    Thanks for the help!

    Code:
    Private Sub store_Data()
            Dim con As New SqlConnection
            Dim cmd As New SqlCommand
            Dim iCounter As Integer
    
            Try
                con.ConnectionString = "Data Source=HPEnvy-HP; Initial Catalog=Cinema; User Id=sa; Password=8034615h;"             
                con.Open()
                cmd.Connection = con
    
                For Each cCtrl As Control In Panel1.Controls
                    If TypeOf cCtrl Is TextBox Then
                        Dim txtBox As New TextBox
                        txtBox = cCtrl
                        iCounter += 1
                            cmd.CommandText = ("INSERT INTO Movie_Table(Movie_ID, Movie_Title) VALUES(" & ("00" & iCounter.ToString) & " ," & (txtBox.Text) & ")") 
    ' The txtBox.Text is causing the problem which I don't understand why.
                    End If
    
                Next
    
                cmd.ExecuteNonQuery()
                MsgBox("Record saved successfully")
                iCounter = 0
    
            Catch ex As Exception
                MessageBox.Show("Error while inserting record on table..." & ex.Message, "Insert Records")
            Finally
                con.Close()
            End Try
    
        End Sub

  2. #2
    Super Moderator si_the_geek's Avatar
    Join Date
    Jul 2002
    Location
    Bristol, UK
    Posts
    41,974

    Re: Updating database problem

    Rather than just looking at the code that builds the string, also look at the string it creates (which will be in cmd.CommandText after that line runs).

    In this case there are clear differences between what it creates and your example.


    In addition to that, string building is not a wise move... For an explanation of why you should be using parameters (and links to code examples), see the article Why should I use Parameters instead of putting values into my SQL string? from our Database Development FAQs/Tutorials (at the top of the Database Development forum).

Tags for this Thread

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  



Click Here to Expand Forum to Full Width