Ошибка Выполнение хранимой процедуры

Question

Когда я попытался выполнить эту хранимую процедуру, это дало мне недействительную ошибку идентификатора, может ли кто-нибудь мне помочь?

CREATE Proc [dbo].[AddLogtrail] 
    @cmodule varchar(100), 
    @caudittype varchar(15),  
    @ctable varchar(100),  
    @cfield varchar(100),  
    @cfieldtype varchar(100), 
    @coriginalval varchar(100),  
    @cnewval varchar(100),  
    @cuser varchar(100),  
    @creason varchar(100)  AS  

    DECLARE @SQL varchar (400)  
    set @SQL = 'Insert into AudittrailLog (ddatetime,cmodulename,caudittype,ctablename,cfieldname,cfieldtype,coriginalval,cnewval,cuser,creason) values ' + 
    '(convert(datetime,GETDATE(),105),'+@cmodule+','+@caudittype+','+@ctable+','+@cfield+','+@cfieldtype+','+@coriginalval+','+@cnewval+','+@cuser+','+@creason+')'; 
    PRINT @SQL 
    Exec @SQL 

Answer 1

Вам нужно добавить дополнительные кавычки для VARCHAR значения

set @SQL = 'Insert into AudittrailLog (ddatetime,cmodulename,caudittype,ctablename,cfieldname,cfieldtype,coriginalval,cnewval,cuser,creason) values ' + 
    '(convert(datetime,GETDATE(),105),'''+@cmodule+''','''+@caudittype+''','''+@ctable+''','''+@cfield+''','''+@cfieldtype+''','''+@coriginalval+''','''+@cnewval+''','''+@cuser+''','''+@creason+''')'; 

При печати он должен показать, как это.

Insert into AudittrailLog (ddatetime ,cmodulename ,caudittype ,ctablename ,cfieldname ,cfieldtype ,coriginalval ,cnewval ,cuser ,creason) values (convert(datetime,GETDATE(),105),'test','test','test','test','test','test','test','test','test') 

также выполнить его, как показано ниже

EXEC (@SQL) 

Answer 2

Вам нужно обернуть каждый varchar в '':

CREATE Proc [dbo].[AddLogtrail] 
    @cmodule varchar(100), 
    @caudittype varchar(15),  
    @ctable varchar(100),  
    @cfield varchar(100),  
    @cfieldtype varchar(100), 
    @coriginalval varchar(100),  
    @cnewval varchar(100),  
    @cuser varchar(100),  
    @creason varchar(100)  AS  

    DECLARE @SQL varchar (400)  
    set @SQL = 'Insert into AudittrailLog (ddatetime,cmodulename,caudittype,ctablename,cfieldname,cfieldtype,coriginalval,cnewval,cuser,creason) values ' + 
    '(convert(datetime,GETDATE(),105),'''+@cmodule+''','''+@caudittype+''','''+@ctable+''','''+@cfield+''','''+@cfieldtype+''','''+@coriginalval+''','''+@cnewval+''','''+@cuser+''','''+@creason+''')'; 
    PRINT @SQL 
    Exec @SQL 

Я надеюсь, что я не пропустите ни одного из ''.

Answer 3

Является ли ваш вопрос просто образцом или вашей фактической процедурой?

Поскольку нет никакой причины использовать динамический SQL для этой процедуры. Ваш запрос статичен. Это простая вставка, где известны все поля. Почему вы просто не используете инструкцию INSERT напрямую?