Visual Basic.NET - Problema con sentencia UPDATE

Hola, soy un viejo programador de Visual Basic 6 que despues de muchos años he retornado usando Visual Basic 2010.
Despues de usar sentencias SQL de tipo Select o Delete no he tenido ningun problema pero al usar el UPDATE me da error a la hora de ejecutar.
Aqui pongo la parte del codigo donde está el error:

Dim oconn As New OleDbConnection

Dim ocomn As New OleDbCommand

Dim odata As OleDbDataReader

oconn.ConnectionString = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=" + System.AppDomain.CurrentDomain.BaseDirectory + "\usuarios.accdb"

oconn.Open()

ocomn.CommandText = "UPDATE Users SET User='" & TextBox1.Text & "',Pass='" & TextBox2.Text & "' WHERE User='" & login & "'"
ocomn.Connection = oconn

odata = ocomn.ExecuteReader()

Es una simple tabla con usuario y contraseña y lo que quiero es modificar los datos que escriba en el text1 y el text2. La variable login es de tipo string y guarda el usuario original que deseo modificar.

Agradecería cualquier tipo de ayuda. Muchas gracias y un saludo!

y cual es el error?

tomando en cuenta que es una actualizacion, lo mas correcto es utilizar ExecuteNonQuery(), el ExecuteReader() es para retornar resultados de una consulta. Nunca he hecho actualizaciones con ExecuteReader() entonces no podria asegurar que ese sea el problema, pero es una opcion.
Tambien esta lo que dice xve, de realizar un print del update, o puedes depurar y mirar con que valor queda ocomn.CommandText antes de ejecutar la sentencia, te copias ese valor y lo ejecutas directamente en el motor, para que te muestre el error.

Saludos

Error de sintaxis en la instruccion UPDATE.

Odata = ocomn.ExecuteReader()

Esto es lo que me marca y la verdad q no entiendo donde puede estar el fallo. He probado tambien con un oledbcommand y executenonquery pero me marca exactamente lo mismo. La instruccion UPDATE.

Realiza un print del update:
"UPDATE Users SET User='" & TextBox1.Text & "',Pass='" & TextBox2.Text & "' WHERE User='" & login & "'"

Puede ser que la variable login este vacia?

Al ser error de sintaxis, es que hay algo mal escrito en la consulta sql, y que un campo necesario esta vacio...

Es todo un misterio... La variable login no esta vacía porque justo de eso quise asegurarme mostrando un mensaje durante el proceso:

MsgBox("Va actualizandose de " & login & " a " & TextBox1.Text & "")

Eso estaba todo bien ya que login recoge el nombre original antes de ser modificado en el texto.

No queda otra que seguir investigando y volviendose loco xD

Gracias de todos modos

Problema solucionado! Mis campos en la tabla se llamaban User y Pass. Al parecer crea algun tipo de conflicto porque los he cambiao por Usuario y Contra y funciona a la perfeccion..

vaya, a mi me acaba de suceder lo mismo y la respuesta me ayudo bastante, increible que access sea tan chafa, bueno ya sabia que lo era, pero no sabia que lo era tanto

Problema solucionado! Mis campos en la tabla se llamaban User y Pass. Al parecer crea algun tipo de conflicto porque los he cambiao por Usuario y Contra y funciona a la perfeccion..
---------------
increible que access sea tan chafa
---------------
de que es chafa, es chafa, es una reverenda basura..., a mi me pasó lo mismo y estuve un buen rato igual hasta que vi el problema.

gracias, tienes razon, tenia user y pass y ese era el error, tenia 2 horas con ese problema.

tambien tenia el mismo problema pero tu sentencia le falta el where donde estas identificando que usuario en si vas a actualizar cada usuario tiene que tener un id que lo identifique

ejemplo :

Dim consulta2 As String = "update usuario " & _
"set NOMBRE='" & txtnombre.Text & "', APELLIDO= '" & txtapellido.Text & "', IDENTIFICACION ='" & txtid.Text & "', EDAD= '" & txtedad.Text & "'" & _
"where ID= '" & txtcodigo.Text & "'"

espero te sirva

Haber Jóvenes,

Yo he trabajado por muchos años con MSAccess, como base de datos, desde vb4 a 6.0 y también en .NET y nunca he tenido problemas, y en aplicaciones no solo pequeñas, pero si no se documentan bien o no conocen la sintaxis del lenguaje, terminan por echarle la culpa a su base de datos en este caso MSAccess.

Además aprovecho para pues ayudarles en mejorar un poquito en cuanto a evitar toda esa serie de comillas, que veo acostumbran a concatenar sus textboxes, y podrían ser vulnerables a inyección de código.

Que tal si aplican así?.


Bueno ese código está hecho con C#, pero es lo mismo en VB, asi que espero les sirva.

Saludos cordiales

PD:Para el amigo que está mandando a ejecutar un DATAREADER, decirle que para consultas de acción, debe emplear ExecuteNonQuery y no ExecuteReader.