Consulta SQL y Access con cDate

[GeoAvila]

hola bueno resulta que estoy trasladondo un sitio de access a SQL Server e hice esta modificación y quiero saber si va bien al menos me funciona, pero saber si no me dará problemas..

con access

Código:

Rs.Open "SELECT * from reservas where cdate(fecha5)>=#" & fec1 & "# and cdate(fecha5)<#" & fec2 & "# and mhotel3= '" & p4 & "' and mncliente= '" & p5 & "' and mestado='" & "I" & "'", Conn, 1,3

con SQL

Código:

Rs.Open "SELECT * from reservas where fecha1>='" & cdate(fec1) & "' and fecha2<'" & cdate(fec2) & "' and mhotel1= '" & p4 & "' and mncliente= '" & p5 & "' and mestado='" & "I" & "'", Conn, 1,3

gracias por informarme..
(revisar solo la sintaxis por favor..)
nos vemos..

[Myakire]

Hola GeoAvila

Pues mira, en SQL-Server es igual que en Access, solo que con apóstrofes en lugar de las almohadillas:

Rs.Open "SELECT * from reservas where fecha1>='" & fec1 & "' and fecha2<'" & fec2 & "' and mhotel1= '" & p4 & "' and mncliente= '" & p5 & "' and mestado='" & "I" & "'", Conn, 1,3

Siempre y cuando fec1 y fec2 sean un string de formato dd/nn/aaaa o mm/dd/aaaa

Saludos

PD. Quiza un between sería más claro.

[GeoAvila]

entonces no le pongo el Cdate para encerrar la variable.???

unicamente esta declaracion de variable. le quito el Cdate trambien.

p2=cdate(fe1&"/"&fe2&"/"&fe3)

[Myakire]

Si, quitale el CDate
Si vas a formar el SQL para mandarselo de paràmetro, necesitas que sea un string.....vamos necesita estar exactamente como lo pondrías en el Analizador de Consultas de SQl-Server, ahí no van esos Cast's

El CDate es necesario, cuando tienes un objeto Command con type a SP, y le vas a mandar los parámetros por sus mètodos, ahí si necesitas inicializar el paràmetro del tipo correcto (si el parámetro es fecha, el valor también debe ser fecha), pero como lo estas haciendo, no es necesario.

La prueba del analizador de consultas es la clave

Saludos amigo GeoAvila

[GeoAvila]

Gracias se te Agredece cuando tengas alguna duda de VB echate una vuelta por el changarro de VB para echarte una mano...

nos vemos..