[SOLUCIONADO] Procedimiento almacenado con errores ayudita

adryan310 · #1 (**permalink**) 06/11/2013, 12:11

Hola a todos...

Tengo el siguiente procedimiento almacenado...

Cita:

CREATE PROCEDURE [dbo].[spTABLE_TEMPORARY_PLAYER]
@Player VARCHAR(10),
@Name VARCHAR(50),
@LastName VARCHAR(50),
@Email VARCHAR(50),
@Password VARCHAR(50),
@Phone VARCHAR(50),
@Zip VARCHAR(50),
@Status CHAR(1),
@LoginDateStart VARCHAR(25),
@LoginDateEnd VARCHAR(25),
@Ip VARCHAR(15),
@WebSite INT,
@Success CHAR(1)
AS
BEGIN
IF OBJECT_ID('tempDB..#PlayerTemp','U') IS NOT NULL
DROP TABLE #PlayerTemp;

DECLARE @StatusAux VARCHAR
DECLARE @WebsiteAux INT
DECLARE @SuccessAux CHAR(1)

IF @Status <> ''
SET @StatusAux = 'AND Status = ' + @Status
ELSE
SET @StatusAux = ''

SELECT IdPlayer, Player, Name, LastName, Email, Password, IdBook
INTO #PlayerTemp
FROM PLAYER
WHERE Player LIKE @Player AND Name LIKE @Name AND LastName LIKE @LastName AND
Email LIKE @Email AND Password LIKE @Password AND Phone LIKE @Phone AND
Zip LIKE @Zip + '' + @StatusAux

IF @WebSite <> 0
SET @WebsiteAux = 'AND B.[id] = ' + @WebSite
ELSE
SET @WebsiteAux = ''

IF @Success <> 'Y' AND @Success <> 'N'
SET @SuccessAux = ''
ELSE
SET @SuccessAux = ' AND L.[success] = '+ @Success

SELECT TOP 10000 P.[IdPlayer], P.[Player], L.[username], L.[password], L.[ip_address], L.[login_date],
L.[success],D.[Description], B.[host_name],
P.[Name] + ' ' + P.[LastName] AS 'Name', P.Email
FROM [BackOffice_Dev].[dbo].[login] L, [dbo].[#PlayerTemp] P, BOOK D, [BackOffice_Dev].[dbo].[book] B
WHERE D.[IdBook] = P.[IdBook] AND L.[book_id] = B.[book_id] AND D.[IdBook] = L.[book_id]
AND P.[player] = L.[username] AND B.[id] = L.[website_id]
AND L.[login_date] >= convert(datetime,(@LoginDateStart))
AND L.[login_date] <= convert(datetime,(@LoginDateEnd))
AND L.[ip_address] LIKE @Ip + @WebsiteAux + @SuccessAux
ORDER BY L.[login_date] DESC
END

El problema que tengo es que los IF que estoy haciendo para validar algunas cosas no esta funcionando y al ejecutar el procedimiento me da el siguiente error:

Cita:

(31 row(s) affected)
Msg 245, Level 16, State 1, Procedure spTABLE_TEMPORARY_PLAYER, Line 47
Conversion failed when converting the varchar value '%%' to data type int.

Eso quiere decir que si me crea la tabla temporal pero el segundo select me esta fallando, alguna ayuda al respecto ?

Libras · #2 (**permalink**) 06/11/2013, 12:17

no se que datos te regrese tu query ni que columna es la que cause el error pero lo que pasa puede ser lo siguiente:

* Quieres comparar un valor string contra un entero
* se esta haciendo una conversion explicita de un campo string a entero

verifica bien los valores que regresa tu query y ve en donde estas usando valores enteros

adryan310 · #3 (**permalink**) 06/11/2013, 12:19

Gracias...

Creo que es en esta parte:

Cita:

IF @WebSite <> 0
SET @WebsiteAux = 'AND B.[id] = ' + @WebSite
ELSE
SET @WebsiteAux = ''

Puedes decirme como puedo declarar a @WebsiteAux de la forma que deseo hacerlo?

Libras · #4 (**permalink**) 06/11/2013, 12:23

porque declaras websiteaux como entero??? no tendria que ser de tipo varchar???

adryan310 · #5 (**permalink**) 06/11/2013, 15:58

Amigo...
Ahora este es mi procedimiento almacenado...

Cita:

CREATE PROCEDURE [dbo].[spTABLE_TEMPORARY_PLAYER]
@Player VARCHAR(10),
@Name VARCHAR(50),
@LastName VARCHAR(50),
@Email VARCHAR(50),
@Password VARCHAR(50),
@Phone VARCHAR(50),
@Zip VARCHAR(50),
@Status CHAR(1),
@LoginDateStart VARCHAR(25),
@LoginDateEnd VARCHAR(25),
@Ip VARCHAR(15),
@WebSite INT,
@Success CHAR(1)
AS
BEGIN
IF OBJECT_ID('tempDB..#PlayerTemp','U') IS NOT NULL
DROP TABLE #PlayerTemp;

DECLARE @StatusAux VARCHAR
DECLARE @WebsiteAux VARCHAR
DECLARE @SuccessAux CHAR(1)

IF @Status <> ''
SET @StatusAux = 'AND Status = ' + @Status
ELSE
SET @StatusAux = ''

SELECT IdPlayer, Player, Name, LastName, Email, Password, IdBook
INTO #PlayerTemp
FROM PLAYER
WHERE Player LIKE @Player AND Name LIKE @Name AND LastName LIKE @LastName AND
Email LIKE @Email AND Password LIKE @Password AND Phone LIKE @Phone AND
Zip LIKE @Zip + '' + @StatusAux

IF @WebSite <> 0
SET @WebsiteAux = 'AND B.[id] = ' + @WebSite
ELSE
SET @WebsiteAux = ''

IF @Success <> 'Y' AND @Success <> 'N'
SET @SuccessAux = ''
ELSE
SET @SuccessAux = ' AND L.[success] = '+ @Success

SELECT TOP 10000 P.[IdPlayer], P.[Player], L.[username], L.[password], L.[ip_address], L.[login_date],
L.[success],D.[Description], B.[host_name],
P.[Name] + ' ' + P.[LastName] AS 'Name', P.Email
FROM [BackOffice_Dev].[dbo].[login] L, [dbo].[#PlayerTemp] P, BOOK D, [BackOffice_Dev].[dbo].[book] B
WHERE D.[IdBook] = P.[IdBook] AND L.[book_id] = B.[book_id] AND D.[IdBook] = L.[book_id]
AND P.[player] = L.[username] AND B.[id] = L.[website_id]
AND L.[login_date] >= convert(datetime,(@LoginDateStart))
AND L.[login_date] <= convert(datetime,(@LoginDateEnd))
AND L.[ip_address] LIKE @Ip + @WebsiteAux + @SuccessAux
ORDER BY L.[login_date] DESC

END

Pero al parecer los if no me quieren funcionar por que la consulta no me arroja datos y si deberia !

Libras · #6 (**permalink**) 06/11/2013, 16:07

estas usando sql server?? si sabias que en sql server no se puede hacer un query dinamico como lo estas haciendo? tendrias que hacer algo como esto:

Código SQL:

Ver originalDECLARE @query VARCHAR(MAX)
SET @query='select * from tabla'
IF @valor=1 THEN
 SET @query=@query + ' where valor=20'
ELSE
 SET @query=@query + ' where valor=50'
 
 EXEC sp_executesql @query

adryan310 · #7 (**permalink**) 06/11/2013, 16:16

Gracias eso no lo sabia y pues ahora ya aprendi algo

Seguire intentanto y te aviso

adryan310 · #8 (**permalink**) 06/11/2013, 16:26

Amigo entonces quiere decir que si yo tengo 10 diferentes opciones o tipos de consultas que se pueden dar debo de hacerlas asi, osea almacenar la consulta en una varables y al final del procedimiento ejecutar dicha variable ?

Libras · #9 (**permalink**) 06/11/2013, 16:38

si son como el ejemplo que mencionas SI

adryan310 · #10 (**permalink**) 06/11/2013, 17:57

Amigo actualmente lo tengo asi el procedimiento:

Cita:

CREATE PROCEDURE [dbo].spSEARCH_MAS
@Player VARCHAR(10),
@Name VARCHAR(50),
@LastName VARCHAR(50),
@Email VARCHAR(50),
@Password VARCHAR(50),
@Phone VARCHAR(50),
@Zip VARCHAR(50),
@Status CHAR(1),
@LoginDateStart VARCHAR(25),
@LoginDateEnd VARCHAR(25),
@Ip VARCHAR(15),
@WebSite INT,
@Success CHAR(1)
AS
BEGIN
DECLARE @CREATE_TLB_TEMP NVARCHAR(MAX) -- QUERY TO CREATE THE TEMPORARY TABLE
DECLARE @QUERY_SEARCH NVARCHAR(MAX) -- QUERY THAT PERFORMED THE SEARCHING

IF OBJECT_ID('tempDB..#PlayerTemp','U') IS NOT NULL
DROP TABLE #PlayerTemp;

SET @CREATE_TLB_TEMP = 'SELECT IdPlayer, Player, Name, LastName, Email, Password, IdBook INTO #PlayerTemp FROM PLAYER '

IF @Status <> 'E' OR @Status <> 'D' BEGIN
SET @CREATE_TLB_TEMP = @CREATE_TLB_TEMP + 'WHERE Player LIKE '+@Player+' AND Name LIKE '+@Name+'
AND LastName LIKE '+@LastName+' AND Email LIKE '+@Email+'
AND Password LIKE '+@Password+' AND Phone LIKE '+@Phone+'
AND Zip LIKE '+@Zip
END
ELSE BEGIN
SET @CREATE_TLB_TEMP = @CREATE_TLB_TEMP + 'WHERE Player LIKE '+@Player+' AND Name LIKE '+@Name+'
AND LastName LIKE '+@LastName+' AND Email LIKE '+@Email+'
AND Password LIKE '+@Password+' AND Phone LIKE '+@Phone+'
AND Zip LIKE '+@Zip+' AND Status = '+@Status
END

EXEC sp_executesql @CREATE_TLB_TEMP
END

Al ejecutarlo de esta forma:

Cita:

EXEC [dbo].spSEARCH_MAS '%%','%test%','%%','%%','%%','%%','%%','D','2013-09-01 00:00:00','2013-11-06 15:45:00','%%',0,'Y'

Me da este error:

Msg 156, Level 15, State 1, Line 1
Incorrect syntax near the keyword 'AND'.

iislas · #11 (**permalink**) 07/11/2013, 11:20

¿Ya revistaste en la línea especifica lo que tienes?

Msg 156, Level 15, State 1, Line 1