插入多个记录时触发器不起作用

Question

当我在表Pedidos上插入一条记录时，我有以下触发器正常工作。

但是，当我插入多个记录时，我收到一条512错误消息。我已经搜索过有关插入多个记录和触发器的详细信息，但没有找到我的问题的答案。

触发器读取插入的记录并从其他表中找到值以修改表planificaciones中的列situacion的值。

我完全错误的方式，我试图做到这一点？我的触发器中是否有任何明显的问题？

CREATE TRIGGER TRG_INS_PL_SYNC_STATUS_PLA ON dbo.pedidos after insert as begin if @@ROWCOUNT = 0 
    return 
set nocount on 
declare @v_idpla int, 
     @v_situacion nvarchar(12), 
     @v_nombre nvarchar(50), 
     @v_almacen nvarchar(50), 
     @v_status_pedido nvarchar(4); 

set @v_almacen = (select almacen_pedido from inserted); 
set @v_nombre =(select cliente from inserted); 
set @v_status_pedido = (select status_pedido from inserted); 
set @v_situacion = (select top 1 nombre from dbo.StatusPlanificacion 
        where STATUS_PEDIDO = @v_status_pedido); 
set @v_idpla = (select top 1 id from dbo.Planificaciones 
       where dia_entrega >= GETDATE() and situacion <> 'Departed' 
         and nombre like '%'+@v_almacen +'%'+ @v_nombre); 
if(@v_idpla is not null) 
    begin 
     --select Timespan=SYSDATETIME() from inserted; 
     select @@rowcount; 
     UPDATE DBO.Planificaciones 
     SET situacion = @v_situacion 
     WHERE id = @v_idpla; 
    end 
end 

UPDATE &解决：寻找在坦纳建议我做的下一个更新的代码和作品，但我觉得有些人能发现这更加清晰和有效。在制革商建议，光标不是最好的方式来做到这一点，最好的选择是加入。在我的情况下，这个插入永远不会超过50个插入在同一时间。

CREATE TRIGGER TRG_INS_PL_SYNC_STATUS_PLA 
    ON dbo.pedidos 
    after insert as 
    begin 

declare @v_idpla int,@v_situacion nvarchar(12),@v_nombre nvarchar(50),@v_almacen nvarchar(50), @v_status_pedido nvarchar(4) 
DECLARE c_cursor CURSOR FAST_FORWARD FOR SELECT ALMACEN_PEDIDO, CLIENTE, STATUS_PEDIDO FROM INSERTED; 
OPEN c_cursor 
fetch next from c_cursor into @v_almacen,@v_nombre,@v_status_pedido 
--declared and open cursor chargin values to variables 
while @@fetch_status = 0 
begin 
    -- set values to variables from anothers tables 
    set @v_situacion = (select top 1 nombre from dbo.StatusPlanificacion where STATUS_PEDIDO = @v_status_pedido); 
    set @v_idpla = (select top 1 id from dbo.Planificaciones where dia_entrega >= GETDATE() and 
     situacion <> 'Departed' and nombre like '%'+@v_almacen +'%'+ @v_nombre); 
    --check value not null for assigned variable and do update to the value 
    if(@v_idpla is not null) 
     begin 
      UPDATE DBO.Planificaciones 
      SET situacion = @v_situacion 
      WHERE id = @v_idpla; 
     end 
     --move to the next row of cursor 
     fetch next from c_cursor into @v_almacen,@v_nombre,@v_status_pedido 
end 
CLOSE c_cursor 
DEALLOCATE c_cursor 
end 

Answer 1

不知道，如果代码是100％正确的，但应该给你一个想法..

插入与该批次的所有行的数据集。你只需要考虑基于集合的操作。

CREATE TRIGGER TRG_INS_PL_SYNC_STATUS_PLA 
ON dbo.pedidos 
    AFTER INSERT 
AS 
BEGIN 
    UPDATE p 
     SET 
      situacion = i.nombre 
    FROM DBO.Planificaciones p 
    INNER JOIN (
     SELECT 
      v_idpla.id 
      v_situacion.nombre 
     FROM INSERTED I 
     CROSS APPLY (
      select top 1 
       SP.nombre 
      from dbo.StatusPlanificacion SP 
      where 
       SP.STATUS_PEDIDO = I.STATUS_PEDIDO 
     ) v_situacion 
     CROSS APPLY (
      select top 1 
       Pla.id 
      from dbo.Planificaciones Pla 
      where 
       Pla.dia_entrega >= GETDATE() and 
       Pla.situacion <> 'Departed' and 
       Pla.nombre like '%'+I.ALMACEN_PEDIDO +'%'+ I.CLIENTE 
     ) v_idpla 
    ) I ON 
     P.id = I.id 
END