sp_execute - Error de SQL dinámico al convertir nvarchar a int

sp_executesql output

Question

Creé un procedimiento en SQL dinámico que tiene una instrucción select y el código se ve así:

ALTER PROCEDURE cagroup    (
    @DataID INT ,
    @days INT ,
    @GName VARCHAR(50) ,
    @T_ID INT ,
    @Act BIT ,
    @Key VARBINARY(16)
)
AS
BEGIN
    DECLARE @SQL NVARCHAR(MAX)
    DECLARE @SchemaName SYSNAME
    DECLARE @TableName SYSNAME
    DECLARE @DatabaseName SYSNAME
    DECLARE @BR CHAR(2)
    SET @BR = CHAR(13) + CHAR(10)

    SELECT  @SchemaName = Source_Schema ,
            @TableName = Source_Table ,
            @DatabaseName = Source_Database
    FROM    Source
    WHERE   ID = @DataID

SET @SQL =  ''SELECT '' + @GName + '' AS GrName ,'' + @BR
                + @T_ID + '' AS To_ID ,'' + @BR
                + @DataID + '' AS DataSoID ,'' + @BR
                + @Act + '' AS Active ,'' + @BR
                + Key + '' AS key'' + @BR
                + ''R_ID AS S_R_ID'' + @BR
                + ''FROM'' + @DatabaseName + ''.''
                + @SchemaName + ''.''
                + @TableName + '' t'' + @BR
                + ''LEFT OUTER JOIN Gro g ON g.GName = ''
                    + @GName + @BR + ''AND g.Data_ID] ='' + @DataID + @BR
                    + ''t.[I_DATE] > GETDATE() -'' + @days + @BR
                    + ''g.GName IS NULL
                        AND '' + @GName + '' IS NOT NULL
                        AND t.[Act] = 1'' + @BR

    PRINT (@SQL)
END

Cuando estoy ejecutando este procedimiento con esta afirmación:

Exec  dbo.cagroup  1,10,''[Gro]'',1,1,NULL

Estoy teniendo el siguiente error.

Msg 245, nivel 16, estado 1, Grupo de grupo de procedimientos, línea 33 La conversión falló al convertir el valor de nvarchar ''SELECT [Gro] AS GName'' a tipo de datos int.

¿Dónde estoy haciendo mal?

Answer 1

Necesita REPARTIR todos los números a nvarchar en la concatenación.

No hay una conversión de estilo VBA implícita a la cadena. En el tipo de datos de SQL Server, la precedencia significa que las entradas son más altas que nvarchar: por lo que toda la cadena intenta ser CAST a int.

SET @SQL =  ''SELECT '' + @GName + '' AS GrName ,'' + @BR
              + CAST(@T_ID AS nvarchar(10)) + '' AS To_ID ,'' ...

Editar: Will A tiene un buen punto: ¡mira por NULLs!

Answer 2

Si tiene que crear este tipo de SQL dinámico, es mejor obtener la información de la columna de los metadatos que pasarla.

Select * from Information_Schema.Columns Where Table_name=@TableName

Debes escribir un cursor feo para construir el SQL. Esperar problemas de rendimiento. Hago mucho de esto durante el desarrollo para escribir código para mí, pero no me atrevo a ejecutarlo en producción.