前触发器的语法错误

Question

我有一张桌子BOOKING(hotelID, roomNo, guestID, startDate, endDate) 我想通过比较startDate 和endDate 来创建一个触发器来进行验证（在插入之前），以查看该时隙是否已被其他客人占用。

CREATE OR REPLACE TRIGGER MYTRIGGER 
BEFORE insert ON BOOKING 

referencing new as newTuple
for each row

declare
  t boolean;
  cursor c is 
    select startDate,endDate from ROOM where hotelID = newTuple.hotelID and ROOMNO = newTuple.roomNo;
BEGIN
  t := true;
  open c;
  loop
    fetch c into mStartDate, mEndDate;
    exit when c%NOTFOUND;

    if (NOT((newTuple.startDate >= mEndDate and newTuple.endDate >= mEndDate)
        or(newTuple.startDate <= mStartDate and newTuple.endDate <= mStartDate))) then
      t := false;
    end if;

  end loop;
  close c;

  WHEN (t=true) then
  INSERT INTO BOOKING(hotelID,roomNo,guestID,startDate,endDate) 
  values(newTuple.hotelID, newTuple.roomNo, newTuple.guestID, newTuple.startDate,
         newTyple.endDate);
END;

但它给了我不知道如何解决的语法错误消息（我是 Oracle 新手）：

Error(26,3): PLS-00103: Encountered the symbol "WHEN" when expecting one of the following:
( begin case declare end exception exit for goto if loop mod
null pragma raise return select update while with
<an identifier> <a double-quoted delimited-identifier>
<a bind variable> << continue close current delete fetch lock
insert open rollback savepoint set sql execute commit forall
merge pipe purge
The symbol "case" was substituted for "WHEN" to continue. 
Error(30,4): PLS-00103: Encountered the symbol ";" when expecting one of the following:
case   

Answer 1

直接的问题是您正在使用WHEN，它是CASE 构造的一部分。你需要在这里使用IF：

IF t then
    INSERT INTO BOOKING(hotelID,roomNo,guestID,startDate,endDate) ...
END IF;

但这不是触发器的工作方式 - 您不会从其中再次插入，您会尝试插入与您正在插入的行完全相同的副本，这会导致您的触发器再次触发。可能只会再一次 - 希望第二个插入会在光标中看到第一个并停止。但是两者实际上都会插入 - 当日期发生冲突时，您实际上并没有阻止插入，您只是没有尝试重复；第二个仍然会插入，所以你会得到两个相同的行。防止插入发生的正常方法是让触发器引发异常；概念上：

IF NOT t THEN
    RAISE_APPLICATION_ERROR(-20001, 'Overlapping dates');
END IF;

但这仍然行不通-您至少还有三个其他问题。首先，您不能（轻松）查询要插入的表；你会得到一个 ORA-04091 'table is mutating' 错误。其次，当您引用新值时，您需要在它们前面加上冒号，例如:newTuple.hotelID 等。第三，您遇到了并发问题；同时插入具有重叠日期的两行不会互相看到，两者都会成功。 （严格来说不是错误，但循环遍历所有记录以查找匹配项将是低效的 - 为什么不只查找与插入日期冲突的现有行？）

触发器似乎不是强制执行此约束的合适方式。

---

好的，这实际上并没有出现变异表错误：

create table booking(hotelid number, roomno number, guestid number,
    startdate date, enddate date);

create or replace trigger mytrigger
before insert on booking 
referencing new as new
for each row
declare
    cnt number;
begin
    select count(*) into cnt from booking
    where hotelid = :new.hotelid
    and roomno = :new.roomno
    and not (enddate < :new.startdate or startdate > :new.enddate);

    if cnt > 0 then
        raise_application_error(-20001, 'Overlapping dates');
    end if;
end;
/
TRIGGER MYTRIGGER compiled

插入一些数据：

insert into booking values (1, 1, 1, date '2013-02-28', date '2013-03-05');

1 rows inserted.

insert into booking values (1, 1, 2, date '2013-02-27', date '2013-03-01');

Error starting at line 24 in command:
insert into booking values (1, 1, 2, date '2013-02-27', date '2013-03-01')
Error report:
SQL Error: ORA-20001: Overlapping dates
ORA-06512: at "STACKOVERFLOW.MYTRIGGER", line 10
ORA-04088: error during execution of trigger 'STACKOVERFLOW.MYTRIGGER'

insert into booking values (1, 1, 3, date '2013-03-05', date '2013-03-06');

Error starting at line 25 in command:
insert into booking values (1, 1, 3, date '2013-03-05', date '2013-03-06')
Error report:
SQL Error: ORA-20001: Overlapping dates
ORA-06512: at "STACKOVERFLOW.MYTRIGGER", line 10
ORA-04088: error during execution of trigger 'STACKOVERFLOW.MYTRIGGER'

insert into booking values (1, 1, 4, date '2013-03-06', date '2013-03-07');

1 rows inserted.

两次输入重叠日期的尝试得到了我们的 -20001 异常，并且只插入了两个不重叠的行：

select * from booking;

   HOTELID     ROOMNO    GUESTID STARTDATE  ENDDATE  
---------- ---------- ---------- ---------- ----------
         1          1          1 28/02/2013 05/03/2013 
         1          1          4 06/03/2013 07/03/2013 

但是您仍然遇到并发问题，因为两个会话可能同时插入重叠数据。由于它们都未提交，每个触发器实例中的 select count(*) 不会看到另一个，因此它们都将报告零，也不会引发异常，并且都将被插入。

Answer 2

您在 PL/SQL 上错误地使用了CASE..WHEN 结构。查看 Oracle 文档站点上的 PL/SQL Control Structures 手册页。这是 CASE..WHEN 的另一个示例，用于 SO question 及其答案。

基本上你需要改变它

WHEN (t=true) then
INSERT INTO BOOKING(hotelID,roomNo,guestID,startDate,endDate) 
VALUES (newTuple.hotelID, newTuple.roomNo, newTuple.guestID, newTuple.startDate,
        newTyple.endDate);

通过这个：

CASE
  WHEN (t=true) THEN INSERT INTO BOOKING(hotelID,roomNo,guestID,startDate,endDate) 
                     VALUES (newTuple.hotelID, newTuple.roomNo, newTuple.guestID, 
                             newTuple.startDate, newTyple.endDate);
END CASE;

或者正如对您问题的评论所建议的那样，如果您只检查一个条件，为什么不使用和IF 结构？它会更干净，更容易理解/维护。