error en SP

[pedraxito]

Hola q tal?

estoy tratando de hacer un SP pero me esta marcando un error, aqui les dejo una copia del Sp para ver si es posible que me ayuden a encontrar el error, espero sus respuestas, gracias!

Código:

SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO

CREATE PROCEDURE spBuscaPacientes
    -- Add the parameters for the stored procedure here
    @NOMBRE nchar(30),
    @MADRE nvarchar(30),
    @DIRECCION nvarchar(60),
    @TEL nvarchar(50)
AS
    DECLARE @criteria nvarchar(1000)
    DECLARE @statement nvarchar(2000)
    SET @criteria = 'WHERE 1=1 '
IF NOT @NOMBRE IS NULL
    SET @criteria = @criteria + 'AND Nombre LIKE @NOMBRE '
IF NOT @MADRE IS NULL
    SET @criteria = @criteria + 'AND Nombre_Madre LIKE @MADRE '
IF NOT @DIRECCION IS NULL
    SET @criteria = @criteria + 'AND Direccion LIKE @DIRECCION'
IF NOT @TEL IS NULL
    SET @criteria = @criteria + 'AND Telefono_1 LIKE @TEL'
BEGIN
    SELECT * FROM Pacientes @criteria
END
GO

[Andres95]

Código:

CREATE PROCEDURE spBuscaPacientes
    -- Add the parameters for the stored procedure here
    @NOMBRE nchar(30),
    @MADRE nvarchar(30),
    @DIRECCION nvarchar(60),
    @TEL nvarchar(50)
AS
BEGIN
    DECLARE @criteria nvarchar(1000)
    DECLARE @statement nvarchar(2000)
    SET @criteria = 'WHERE 1=1 '
IF NOT @NOMBRE IS NULL
    SET @criteria = @criteria + 'AND Nombre LIKE @NOMBRE '
IF NOT @MADRE IS NULL
    SET @criteria = @criteria + 'AND Nombre_Madre LIKE @MADRE '
IF NOT @DIRECCION IS NULL
    SET @criteria = @criteria + 'AND Direccion LIKE @DIRECCION'
IF NOT @TEL IS NULL
    SET @criteria = @criteria + 'AND Telefono_1 LIKE @TEL'



execute ('SELECT * FROM Pacientes' + @criteria)

END
GO

[Thumper]

No soy muy amante del SQL Dinámico o "Código Rojito" como lo llamamos mis patas y yo, asi que te dejo esta propuesta haber q te parece.

SELECT *
FROM Pacientes
WHERE ( NOMBRE LIKE @NOMBRE OR @NOMBRE IS NULL) AND
( MADRE LIKE @MADRE OR @MADRE IS NULL) AND
( DIRECCION LIKE @DIRECCION OR @DIRECCION IS NULL) AND
( TEL LIKE @TEL OR @TEL IS NULL)

Por cierto esas claúsulas LIKE no estan cumpliendo su verdadera labor, en el código que presentas sería lo mismo poner "=".

Para buscar los que coincidan con el mismo nombre

NOMBRE = @NOMBRE OR @NOMBRE IS NULL

ó

Aquellos nombres que tengan la palabra dentro.

NOMBRE LIKE '%' + @NOMBRE + '%' OR @NOMBRE IS NULL

Saludos

[Andres95]

Tienes razon, es preferible en la mayoria de los casos ejecutar el codigo directamente.

Solo que en algunos casos es conveniente evaluar el desempeño de ambos, ya que con like en todos los campos de un catalogo, estarias haciendo un barrido total de la tabla, y en el caso de una tabla con muchos registros, entre menos filtros en el where tipo like es mejor...


Saludos!