帮我看看这个触发器是否有问题呀？

[dm1cyg]

库存里有商品编码、数量等字段

目的是，如果插入到销售表的记录和库存表里的编号相同就从库存里减少库存
CREATE TRIGGER xshj ON dbo.门诊销售表
FOR INSERT
/*
write by cyg
2007-4-15
处理划价销售部分


*/
AS
begin tran
if exists(select * from inserted a , 库存表 b where (a.商品编码=b.商品编码))
begin
update 库存表 set 数量=b.数量-a.数量 from 库存表 b join inserted a on ( b.商品编码=a.商品编码)
end

commit tran

[hpx021131]

沙发?

[wzy_love_sly]

应该没问题吧

[wzy_love_sly]

CREATE   TRIGGER   xshj   ON   dbo.门诊销售表  
FOR   INSERT

AS
  begin   tran
    if   exists(select     1     from   inserted   a   ,   库存表   b   where   a.商品编码=b.商品编码)
          begin
                update   库存表     set     数量=b.数量-a.数量     from     库存表   b     join   inserted   a     on      b.商品编码=a.商品编码
        end

commit   tran 


create table 门诊销售表 (商品编码 int,商品名称 varchar(20),数量 int)


create table 库存表 (商品编码 int,商品名称 varchar(20),数量 int)

insert into 库存表 select 1,'电脑',14
insert into 库存表 select 2,'机箱',20
insert into 库存表 select 3,'硬盘',10
select * from 库存表
insert into 门诊销售表 select 1,'电脑',5
select * from 库存表

商品编码 商品名称 数量
1 电脑 14
2 机箱 20
3 硬盘 10

-------------------------------------
商品编码 商品名称 数量
1 电脑 9
2 机箱 20
3 硬盘 10

[areswang]

where (a.商品编码=b.商品编码)
on ( b.商品编码=a.商品编码)
---------------------------------
不用括号

[dawugui]

没看出什么地方错了?

难道有其他的条件?

[dm1cyg]

要是用游标杂处理呀

[dawugui]

括号要与不要都没有问题.

[JL99000]

没有问题的
看看是不是业务逻辑上有问题

[wzy_love_sly]

游标和这一样，还不如这个

[dm1cyg]

这个是不是只要插入到销售表里库存就肯定能减少呀

[wzy_love_sly]

是，你键触发器就是执行更新库存

[w2jc]

要是用游标杂处理呀
------------------------------
能不用游标就最好不要用，
游标是对一个记录操作，SQL是对一组记录操作，
一定是后者快。