954,566 Members — Technology Publication meets Social Media
Username:
Password:
Lost login information?
Have something to say? Contribute New Article Reply to this Article

access UPDATE problems

i have a very simple access DB and im tryin to update the records but the update never works. im using asp.net/vb.net and i've include the execute in a try/catch block and it always prints the sql string (the catch). here is my code, perhaps i have an error that ive missed:

SQL = "UPDATE members " & _
         "SET username = '" & txtUsername.Text.Trim() & "', " & _
         "password = '" & txtPassword1.Text.Trim() & "', " & _
         "first_name = '" & txtFirstName.Text.Trim() & "', " & _ 
         "last_name = '" & txtLastName.Text.Trim() & "', " & _
         "email = '" & txtEmail.Text.Trim() & "'" & _
         "WHERE first_name = " & Session("first_name") & _
         "AND last_name = " & Session("last_name") & ""

' Check if connection to DB is already open
If MyConn.State = ConnectionState.Closed Then
    MyConn.Open()
End If

Dim MyCmd As New OleDbCommand(SQL, MyConn)

' Try/Catch...in case DB error
Try
    MyCmd.ExecuteNonQuery()
    MyConn.Close()
    MyConn.Equals("")

    Response.Redirect("default.aspx")

Catch ex As Exception
    profile_error.Text = SQL 
    profile_error.Style("Color") = "red"
End Try


im also adding a user ID soon and will be changing the where to check for the unique ID.

dru987
Newbie Poster
13 posts since Nov 2004
Reputation Points: 10
Solved Threads: 0
 

G'd evening Dru!
The sintax seems to be fine, but we don't know the values you are sending nor their types. I would sugest to store (just for tests purposes) the sql string in a varible and then write its content in the inmediate window.
Good luck
Estuardo

Estuardo
Newbie Poster
13 posts since Jul 2005
Reputation Points: 10
Solved Threads: 0
 

What is the error-message?

Maybe you have to put an extra spcace before the last line (" AND last_name = " & Session("last_name") & "")

weilander
Newbie Poster
5 posts since Aug 2005
Reputation Points: 10
Solved Threads: 0
 

This article has been dead for over three months

Post: Markdown Syntax: Formatting Help
You
View similar articles that have also been tagged: