Ошибка выполнения SQL Trigger

Question

У меня есть следующий триггер. При вставке новой строки Хранимая процедура не может получить значение параметра переменной @ItemID. Я пытаюсь передать значение ItemID колонны вновь вставленной строки хранимой процедуры CalculateCurrentStock

ALTER TRIGGER UpdateCurrentStock 
    ON StockIn 
    AFTER INSERT 
AS 
BEGIN 
    SET NOCOUNT ON; 
    EXEC CalculateCurrentStock (SELECT ItemID From INSERTED) 
END 

Текст ошибки читает

Процедура или функция «CalculateCurrentStock» ожидает параметр «@ItemID», который не был поставлен. Заявление было прекращено.

Благодарим за помощь.

EDIT: Ответ

Я изменил триггер на Derek Кромм-х и предложение КМ в

ALTER TRIGGER UpdateCurrentStock 
    ON StockIn 
    AFTER INSERT 
AS 
BEGIN 
    SET NOCOUNT ON; 

    DECLARE @CSV varchar(max) 
    SELECT @CSV=ISNULL(@CSV+',','')+ItemID From INSERTED 
    EXEC CalculateCurrentStock @CSV 


END 

Спасибо за помощь :)

Answer 1

Я лично высунул свой глаз перед тем, как использовать курсор, особенно когда вы можете передать набор значений (параметр табличных значений или CSV) за один раз за один вызов процедуры.

Если вы используете SQL Server 2008, вы можете создать свою процедуру с помощью Table-Valued Parameters и должны быть в состоянии пройти в таблице INSERTED.

Если вы используете SQL Server 2005, вы можете создать список значений с запятой (csv) в переменной varchar (max) и передать это в вашу процедуру.

EDIT, вот как пройти в CSV

ALTER TRIGGER UpdateCurrentStock 
    ON StockIn 
    AFTER INSERT 
AS 
BEGIN 
    SET NOCOUNT ON; 
    DECLARE @CSV varchar(max) 
    SELECT @CSV=ISNULL(@CSV+',','')+ItemID From INSERTED 
    EXEC CalculateCurrentStock @CSV 
END 

теперь в пределах вам нужно разделить на части значения @CSV, так что смотрите здесь: Pass a list-structure as an argument to a stored procedure

Answer 2

Edit: Как Мартин отметил, вы можете передать параметр таблицы, если используете SQL Server 2008. Если вы используете 2005 или ранее, вы не можете этого сделать. В этом случае KM предлагает использовать значение с разделителями-запятыми, с чем я лично не согласен.

На самом деле, вероятно, вы должны скопировать логику хранимой процедуры непосредственно в триггер и полностью избежать всего беспорядка.