Order by / Case

[chueke]

Gente, tengo la siguiente consulta SQL:

Código SQL:

SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM 
    FROM 
    (SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM, ROW_NUMBER() 
    OVER 
    (ORDER BY
    -- Orden x marca
    CASE WHEN @marc = 'ok'
    THEN VMVNNM END ASC,
    -- Orden x precio descendente
    CASE WHEN @prc = 'desc'
    THEN PSPREC END DESC,
    -- Orden x precio ascendente
    CASE WHEN @prc = 'asc'
    THEN PSPREC END ASC 
    ) 
    AS Recursos 
    FROM dbo.Recursos WHERE CPA01 LIKE  '%' + @palabra + '%') 
    AS Resultados 
    WHERE Recursos BETWEEN 8 * @PageNumber  + 1 AND 8 * (@PageNumber + 1)
 

Hasta aqui todo bien. Esta consulta busca los registros correspondientes mediante un LIKE en la base de datos, los pagina y los ordena mediante el ORDER BY.
En un store procedure comun esto funcionaba bien, pero cuando lo quiero implementar dentro del string en un sp_executesql me arroja el siguiente error:

"Se especificó una expresión no booleana en un contexto donde se esperaba una condición, cerca de 'Recursos'"

Noto que el error esta en los CASES del ORDER BY, cuando dejo uno solo la consulta funciona bien.
Intente cambiar la sentencia decenas de veces pero no consigo que funcione.
Alguna sugerencia, que puede estar mal?

[Libras]

puedes poner tu codigo cuando realizas el string?

[chueke]

Iniciado por Libras

puedes poner tu codigo cuando realizas el string?

Código SQL:

ALTER PROCEDURE [dbo].[bus_pal] 
    -- Add the parameters for the stored procedure here
    @clave NVARCHAR(50),
    @PageNumber INT,
    @marc NVARCHAR(50) = NULL,
    @prc NVARCHAR(50) = NULL
   
AS
BEGIN
    -- SET NOCOUNT ON added to prevent extra result sets from
    -- interfering with SELECT statements.
    SET NOCOUNT ON;
 
    -- Insert statements for procedure here
    DECLARE @SQLString NVARCHAR(500)
    DECLARE @ParmDefinition NVARCHAR(500)
 
    /* Cadena */
    SET @SQLString = N'SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM FROM (SELECT RXFAM1, RXFAM2, RXFAM3, RXFAM4, PSPREC, VMVNNM, ROW_NUMBER() OVER (ORDER BY CASE WHEN @marc = ''ok'' THEN VMVNNM END ASC, CASE WHEN @prc = ''desc'' THEN PSPREC END DESC, CASE WHEN @prc = ''asc'' THEN PSPREC END ASC) AS Recursos FROM Recursos WHERE CPA01 LIKE  @clave + ''%'') AS Resultados WHERE Recursos BETWEEN 8 * @PageNumber  + 1 AND 8 * (@PageNumber + 1)'
    /* Formato del/los parámetros */
    SET @ParmDefinition = N'@clave NVARCHAR(50),@PageNumber INT,@marc NVARCHAR(50),@prc NVARCHAR(50)'
 
    EXEC sp_executesql @SQLString, @ParmDefinition, @clave, @PageNumber, @marc, @prc;
 
END
 

[Libras]

creo que te falta un parentesis cerca del like...

Código SQL:

SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM FROM (SELECT RXFAM1, RXFAM2, RXFAM3, RXFAM4, PSPREC, VMVNNM, ROW_NUMBER() OVER (ORDER BY CASE WHEN @marc = 'ok' THEN VMVNNM END ASC, CASE WHEN @prc = 'desc' THEN PSPREC END DESC, CASE WHEN @prc = 'asc' THEN PSPREC END ASC) AS Recursos FROM Recursos WHERE CPA01 LIKE  [B]([/B]@clave + '%') AS Resultados WHERE Recursos BETWEEN 8 * @PageNumber  + 1 AND 8 * (@PageNumber + 1)
 

prueba y comentas :)

[chueke]

Iniciado por Libras

creo que te falta un parentesis cerca del like...

"Sintaxis incorrecta cerca de la palabra clave 'AS'."


Libras, ese paréntesis es el que cierra el SELECT.

[Libras]

si vemos tu query tienes:

Código SQL:

SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM 
FROM (SELECT RXFAM1, RXFAM2, RXFAM3, RXFAM4, PSPREC, VMVNNM, ROW_NUMBER() OVER (ORDER BY CASE WHEN @marc = 'ok' 
THEN VMVNNM END ASC, CASE WHEN @prc = 'desc' THEN PSPREC END DESC, CASE WHEN @prc = 'asc' THEN PSPREC END ASC) 
AS Recursos
 

aqui estas sacando los valores que necesitas y lo pones como una tabla llamada recursos....y le haces un from recursos??

Código SQL:

FROM Recursos 
WHERE CPA01 LIKE  @clave + '%') AS Resultados WHERE Recursos BETWEEN 8 * @PageNumber  + 1 AND 8 * (@PageNumber + 1)
 

si te fijas tienes 2 clausulas from dentro del mismo query......

[chueke]

Iniciado por Libras

aqui estas sacando los valores que necesitas y lo pones como una tabla llamada recursos....y le haces un from recursos??

si te fijas tienes 2 clausulas from dentro del mismo query......

Con respecto a los FROM esta funcionando bien así en un STORE PROCEDURE tradicional, una se utiliza para la consulta y otro para la paginación.
El problema tal como mencione viene con el string en el sp_executesql.

Use el mismo nombre para el AS que para la tabla, lo cual puede mal entederse. En el SQL a continuacion esta corregido (cambio "Recursos" x "Registros").

Código SQL:

SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM FROM (SELECT RXFAM1, RXFAM2, RXFAM3, RXFAM4, PSPREC, VMVNNM, ROW_NUMBER() OVER (ORDER BY CASE WHEN @marc = ''ok'' THEN VMVNNM END ASC, CASE WHEN @prc = ''DESC'' THEN PSPREC END DESC, CASE WHEN @prc = ''ASC'' THEN PSPREC END ASC) AS Registros FROM Recursos WHERE CPA01 LIKE  @clave + ''%'') AS Resultados WHERE Registros BETWEEN 8 * @PageNumber  + 1 AND 8 * (@PageNumber + 1)
 

[Libras]

si hago esto con un query:

Código SQL:

SELECT * FROM (SELECT * FROM #temp) AS recursos FROM #temp
 

que es lo que estas haciendo me marca un error......vuelvo a preguntar estas seguro que el procedure tiene 2 from en un solo query?

[chueke]

Iniciado por Libras

select * from (select * from #temp) as recursos from #temp

Libras, en esa consulta hay tres FROM y en la que yo puse son solo 2 (uno para traer los registros en base al LIKE de la DB y el otro para la paginación).

[Libras]

ok mi error :P pero de todos modos yo veo 2 from en tu consulta......

[chueke]

Libras, a modo de ejemplo y extraido de una web -en el SP hay dos FROM-:

Código SQL:

CREATE PROCEDURE [dbo].[spPruebaPaginacion]
@Pagina INT,
@RegistrosporPagina INT,
 
AS
 
BEGIN
 
SET NOCOUNT ON;
 
SELECT * FROM (
SELECT ROW_NUMBER()OVER(ORDER BY dbo.PruebaPaginacion.Nombre ASC) AS RowNum, dbo.PruebaPaginacion.Id_Usuario ,
dbo.PruebaPaginacion.Nombre
FROM dbo.PruebaPaginacion
)
AS ResultadoPaginado
WHERE RowNum BETWEEN (@Pagina – 1) * @RegistrosporPagina + 1 AND @Pagina * @RegistrosporPagina
 
END
 

De hecho justo caí acá, donde vos respondes en una consulta similar que también usa dos FROM:

http://www.forosdelweb.com/f87/consulta-row_number-879121/

[Libras]

y me das la razon poniendo el codigo de tu procedure :)

Esta es tu cadena:


SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM
FROM
(SELECT RXFAM1, RXFAM2, RXFAM3, RXFAM4, PSPREC, VMVNNM, ROW_NUMBER() OVER (ORDER BY CASE WHEN @marc = ''ok'' THEN VMVNNM END ASC, CASE WHEN @prc = ''DESC'' THEN PSPREC END DESC, CASE WHEN @prc = ''ASC'' THEN PSPREC END ASC
--aqui no hay from
) AS Registros
FROM Recursos
WHERE CPA01 LIKE @clave + ''%'') AS Resultados WHERE Registros BETWEEN 8 * @PageNumber + 1 AND 8 * (@PageNumber + 1)

y este es el codigo del procedure:

SELECT * FROM (
SELECT ROW_NUMBER()Over(ORDER BY dbo.PruebaPaginacion.Nombre ASC) AS RowNum, dbo.PruebaPaginacion.Id_Usuario ,
dbo.PruebaPaginacion.Nombre
FROM dbo.PruebaPaginacion

--aqui si hay un from
)
AS ResultadoPaginado
WHERE RowNum BETWEEN (@Pagina – 1) * @RegistrosporPagina + 1 AND @Pagina * @RegistrosporPagina


Notas una diferencia? Yo si, que esta usando un from dentro de la paginacion ;), a eso me referia con 2 from en un solo query, en el ejemplo que pones tiene 2 from pero en subquerys ;)

[chueke]

Iniciado por Libras

Notas una diferencia?

Haber Libras, tal vez yo me este volviendo loco con esto y no vea la obviedad, jaj.
Estructuralmente ambas sentencias me parecen iguales:

La que uso:

Código SQL:

SELECT * FROM (
SELECT *, ROW_NUMBER() OVER 
(ORDER BY CASE WHEN @marc = ''ok'' THEN VMVNNM END ASC, CASE WHEN @prc = ''DESC'' THEN PSPREC END DESC, CASE WHEN @prc = ''ASC'' THEN PSPREC END ASC) 
AS Registros FROM Recursos WHERE CPA01 LIKE  @clave + ''%'') 
AS Resultados WHERE Registros BETWEEN 8 * @PageNumber  + 1 AND 8 * (@PageNumber + 1)
 

La que puse de ejemplo:

Código SQL:

SELECT * FROM (
SELECT ROW_NUMBER()OVER
(ORDER BY dbo.PruebaPaginacion.Nombre ASC) AS RowNum, dbo.PruebaPaginacion.Id_Usuario ,
dbo.PruebaPaginacion.Nombre FROM dbo.PruebaPaginacion)
AS ResultadoPaginado
WHERE RowNum BETWEEN (@Pagina – 1) * @RegistrosporPagina + 1 AND @Pagina * @RegistrosporPagina
 

La ubicación de la clausula FROM me parece la misma en ambas.

[Libras]

prueba con algo como esto:

Código SQL:

DECLARE @query Nvarchar(MAX)
SET @query=
'SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM 
    FROM 
    (SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM, ROW_NUMBER() 
    OVER 
    (ORDER BY'  
    -- Orden x marca
    IF @marc='ok'
     SET @query=@query + 'VMVNNM)'
    IF @prc='desc'
     SET @query=query + 'PSPREC)'
    IF @prc='asc'
     SET @query=query + 'PSPREC)' 
 
   SET @query=@query + ' AS Recursos 
    FROM dbo.Recursos WHERE CPA01 LIKE  ''%'' + @palabra + ''%'') 
    AS Resultados 
    WHERE Recursos BETWEEN 8 * @PageNumber  + 1 AND 8 * (@PageNumber + 1)'
 

saludos!

[chueke]

Libras, finalmente logre que funcione con la siguiente sentencia:

Código SQL:

SELECT * FROM (SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM, ROW_NUMBER() 
OVER (ORDER BY CASE WHEN @marc = ''ok'' THEN VMVNNM END ASC, CASE WHEN @prc = ''DESC'' THEN PSPREC END DESC, CASE WHEN @prc = ''ASC'' THEN PSPREC END ASC) AS RowNum FROM Recursos 
WHERE 
CPA01 LIKE @clave + ''%'') 
AS Paginacion WHERE RowNum BETWEEN 8 * @PageNumber  + 1 AND 8 * (@PageNumber + 1)
 

Utilizando la misma estructura que funciono (lo que acabo de poner) modifique el LIKE por varios AND en otro SQL, e increiblemente vuelvo a mostrar el mismo error de antes.

Se especificó una expresión no booleana en un contexto donde se esperaba una condición, cerca de 'BETWEE'.

El otro SQL:

Código SQL:

SELECT * FROM (SELECT RXFAM1, RXFAM2, RXFAM3, PSPREC, VMVNNM, ROW_NUMBER() 
OVER (ORDER BY CASE WHEN @marc = ''ok'' THEN VMVNNM END ASC, CASE WHEN @prc = ''DESC'' THEN PSPREC END DESC, CASE WHEN @prc = ''ASC'' THEN PSPREC END ASC) AS RowNum FROM Recursos 
WHERE 
(RXFAM1 = COALESCE(NULLIF(@ref1, ''''), RXFAM1) 
AND 
RXFAM2 = COALESCE(NULLIF(@ref2, ''''), RXFAM2) 
AND 
RXFAM3 = COALESCE(NULLIF(@ref3, ''''), RXFAM3))
AS Paginacion WHERE RowNum BETWEEN 8 * @PageNumber  + 1 AND 8 * (@PageNumber + 1)
 

Veo que el problema surge cuando pongo el tercer AND (tengo que poner seis en total). Esto me desorienta por completo, alguna idea?? desde ya, gracias.

[gnzsoloyo]

alguna idea

Usa un editor con corrector sintáctico. Siempre es una buena opción cuando tienes consultas muy largas.