Ayuda con Sintaxis asp

[Laika]

Hola,

Me da error al ejecutar una página .asp en esta linea:

Set Ob_RS=Conn.Execute ( " DELETE FROM newsletter where email = '" & Request.QueryString("email") & "' AND id = '" & Request.QueryString("id") & "' ")

Este es el error:

Microsoft OLE DB Provider for ODBC Drivers error '80040e14'

[Microsoft][Controlador ODBC Microsoft Access] Instrucción SQL no válida; se esperaba 'DELETE', 'INSERT', 'PROCEDURE', 'SELECT' o 'UPDATE'.

/plantilla_mail/baja.asp, line 5

¿Pueden ayudarme por favor?

[ElAprendiz]

Set Ob_RS=Conn.Execute ( " DELETE FROM newsletter where email = '" & Request.QueryString("email") & "' AND id = '" & Request.QueryString("id") & "' ")

entre delete y from debes colocar los campos que quieres seleccionar, si deseas seleccionarlos todos coloca *

[Laika]

He colocado el * y me da este error:

Microsoft OLE DB Provider for ODBC Drivers error '80040e07'

[Microsoft][Controlador ODBC Microsoft Access] No coinciden los tipos de datos en la expresión de criterios.

/plantilla_mail/baja.asp, line 5

¿Tiene esto algo que ver con que el campo id es autonumérico?. ¿Cómo debería de poner si es pr eso?

Muchas gracias

[CHuLoSoY]

[Microsoft][Controlador ODBC Microsoft Access] No coinciden los tipos de datos en la expresión de criterios. --> este error me da que pensar, puede ser que el email que esté en el querystring no coincida con el id que también está en el querystring, es decir, si tienes esta tabla:

id | email
1 | [email protected]
2 | [email protected]
3 | [email protected]


Puede ser que tengas esta dirección: pagina.asp?id=1&[email protected]

Te recomiendo usar sólo un campo, o el id o el email.

Saludos.

[Laika]

¿Pero como puede ser eso posible?, al recorrer un registro cómo me puede pasar eso?

Quiero utilizar los 2 para hacerlo más seguro el que nadie que intente con id aleatoria spueda borrarme registros en mi base de datos.

[CHuLoSoY]

no tengo ni idea, pero primero comprueba si es cierto lo que te he comentado en el msg anterior.

Puedes también hacer esto:

pagina.asp?id=5&password=UNACLAVEQUETÚELIJAS

pagina.asp:

<%
clave=request.querystring("password")

if clave="UNACLAVEQUETÚELIJAS" then
'si se cumple la condición de que la password es correcta entonces borras el registro con id 5
else
'sino mensaje de error
nada="error, password incorrecta"
end if

response.write nada
%>

así es aún mucho más seguro y te ahorras de errores.

[ElAprendiz]

si el id es autonumerico, quitale las comillas simples

Set Ob_RS=Conn.Execute ( " DELETE FROM newsletter where email = '" & Request.QueryString("email") & "' AND id = " & Request.QueryString("id") & " ")


saludos

[maestro]

Set Ob_RS=Conn.Execute ( " DELETE FROM newsletter where email = '" & Request.QueryString("email") & "' AND id = " & Request.QueryString("id") & " ")

<%
varid=Request.QueryString("id")
varemail=Request.QueryString("email")
Set Ob_RS=Conn.Execute ( " DELETE FROM newsletter where email = "&varemail&" AND id = "&varid&" ")
%>

Personalmente prefiero definir antes las variables, eso hace el codigo mas legible y evita los fallos de las comillas simples y dobles.
Asi sabemos mejor que fallla, si la SQL o las definiciones anteriores, fijandonos en el error de la linea:

[RENZO]

primero:

Set Ob_RS=Conn.Execute ( " DELETE FROM newsletter where email = '" & Request.QueryString("email") & "' AND id = '" & Request.QueryString("id") & "' ")

eso no te devuelve un recordset.

segundo:
Si id es numerico creo que debes quitar las comillas simples.
tan solo pon esto:

Conn.Execute ( " DELETE FROM newsletter where email = '" & Request.QueryString("email") & "' AND id = " & Request.QueryString("id") )

y cuentame como te fue.

[Laika]

Me da error:

ADODB.Recordset error '800a0e78'

La operación no está permitida si el objeto está cerrado.

/plantilla_mail/baja.asp, line 6

No entiendo como no me funciona

[maestro]

Muy facil, estas intentando mostrar o actualizar un recodset y te ocurreo lo siguiente.

Te has olvidado de abrirlo
rs.open SQL, oconn,1,2

o bien ya lo has cerrado:
<%rs.close%>

y luego intentas visualizar datos:
<%=rs("campo")%>

o actualizar datos
<%
rs("campo")=variable
rs.update
%>

Revisa a ver cual de estas cosas te pasa.

[Laika]

Hola,

Se me habia olvidado de abrir el recordset.
Ya lo he abierto y no me da por fiinnn no me da ningun error pero el registro no se me borra de la base de datos.

Los datos que le paso a baja.asp son:

http://www.xxx.com/plantilla_mail/baja.asp?id=119 & [email protected]

Y esta es lo que pone en la barra de direcciones al ir a baja.asp:

http://www.xxx.com/plantilla_mail/[email protected]

¿Que pasa ahora? Es pa volverse loco

Gracias
Laika

[maestro]

http://www.xxx.com/plantilla_mail/baja.asp?id=119%20&%[email protected]

Ves ese %20 ?, es el simbolo del espacio ASCII, al construir la URL has dejado un espacio en los parametros del querystring.

[Laika]

¿Donde esta el espacio que me crea aquí? heprobado de varias maneraspero me llega con espacios (%20)

<a href='http://www.midominio.com/plantilla_mail/baja.asp?id="&Rs.Fields("id").Value&" & email="&Rs.Fields("email").Value&"'>aqui.</a>

Gracias
Laika

[AlZuwaga]

poné así:


<a href='http://www.midominio.com/plantilla_mail/baja.asp?id="&Rs.Fields("id").Value&"&email="&Rs.Fields("email").Value&"'>aqui.</a>


(la parte en negrita fue modificada)

[xBlaze]

Hola:

<a href='http://www.midominio.com/plantilla_mail/baja.asp?id="&Rs.Fields("id").Value&" & email="&Rs.Fields("email").Value&"'>aqui.</a>

El espacio esta en: ...Rs.Fields("id").Value&" & email="&Rs...
----------------------------------------------^

donde termina 'Valiue' luego sumas con '&' inicia una cadena con (") y luego pones un espacio que no debe de exisitir, despues pones un (&) y otro espacio que tambpo debe de existir, lo que tu haces te genera:

<a href='http://www.midominio.com/plantilla_mail/baja.asp?id=34 & [email protected]'>aqui.</a>

Donde 34 y el correo son lo que genere el codigo ASP, pero el problema es despues del 34 (...34 & email=...) debe de ser (...34&email=...) sin espacios, si exite algun otro que por los saltos de linea no se vean, tambien quitalos.

Espero y te sirva.

[Laika]

Muchas gracias a todos,

Por fin me funciona el código.

Laika :cantar: