Question

我正在尝试检查是否已在该日期租用了要插入系统的房间。我虽然计算了与房间号和日期相匹配的行，然后回滚交易。但是我收到了以下错误，即使我已经更改了代码以引发用户定义的异常：

ERROR:  cannot begin/end transactions in PL/pgSQL
HINT: Use a BEGIN block with an EXCEPTION clause instead.
CONTEXT: PL/pgSQL function "checkRoom"() line 17 at SQL statement

CREATE OR REPLACE FUNCTION "checkRoom"() RETURNS TRIGGER AS
$BODY$
DECLARE 
    counter integer;
  BEGIN
    SELECT COUNT("num_sesion")
    FROM "Sesion"
    INTO counter
    WHERE "Room_Name"=NEW."Room_Name" AND "Date"=NEW."Date";

    IF (counter> 0) THEN -- Probably counter>1 as it's triggered after the transaction..
        raise notice 'THERE'S A ROOM ALREADY!!';
        raise exception 'The room is rented at that date';
    END IF;
    RETURN new;
EXCEPTION
    WHEN raise_exception THEN
        ROLLBACK TRANSACTION;
        RETURN new;
END;$BODY$
LANGUAGE plpgsql VOLATILE NOT LEAKPROOF;

然后我创建了触发器：

CREATE TRIGGER "roomOcupied" AFTER INSERT OR UPDATE OF "Room_Name", "Date"
ON "Sesion" FOR EACH ROW
EXECUTE PROCEDURE "checkRoom"();

从我上一次使用SQL开始已经过去2年了，plsql和plpgsql之间的变化让我发疯了。

Answer 1

PostgreSQL处理错误与其他数据库有很大不同。任何未处理的错误都会引发给用户。在PL / pgSQL中，您可以捕获任何异常，或者您可以引发任何异常，但您无法显式控制事务。任何PostgreSQL语句都在事务内部执行（函数也是如此）。当任何未处理的异常到达顶部时，最外部的事务会自动中断。

你能做什么：

引发异常（通常在触发器中）

BEGIN
  IF CURRENT_USER <> 'Admin' THEN
    RAISE EXCEPTION 'missing admin rights';
  END IF;
  RETURN NEW;
END;

捕获异常

BEGIN
  BEGIN -- start of protected section
    -- do some, what can be stopped by exception
  EXCEPTION WHEN divide_by_zero THEN
    -- exception handler;
    RAISE WARNING 'I was here';
    -- should ignore
  EXCEPTION WHEN others THEN
    -- any unexpected exception
    RAISE WARNING 'some unexpected issue';
    RAISE; -- forward exception'
  END;

没有其他可能性 - 所以在PL / pgSQL中编写应用程序非常简单，但与PL / SQL或TSQL不同。

Answer 2

触发器功能的几个问题：

使用IF EXISTS (...) THEN代替计算所有出现并检查counter > 0。更快，更简单。
名为 AFTER INSERT OR UPDATE的触发器功能只能返回NULL。 RETURN NEW仅与名为 BEFORE 的触发器相关。我引用the manual here：

对于在操作之后触发的行级触发器，将忽略返回值，因此它们可以返回NULL。
不平衡的单引号。
作为@Pavel already explained，您无法在plpgsql函数中控制事务。任何未处理的异常都会自动回滚您的事务。因此，只需删除EXCEPTION块。

您的假设触发器被重写：

CREATE OR REPLACE FUNCTION check_room()
  RETURNS TRIGGER AS
$func$
BEGIN
   IF EXISTS (
         SELECT 1
         FROM   "Sesion"               -- sure ist's not "Session"?
         WHERE  "Room_Name" = NEW."Room_Name"
         AND    "Date" = NEW."Date") THEN
     RAISE EXCEPTION 'The room is rented at that date';
   END IF;
   RETURN NULL;
END
$func$ LANGUAGE plpgsql;

您的触发器会更有意义 BEFORE 但是UNIQUE INDEX ON ("Room_Name", "Date")会更有效地做同样的事情。任何违规行都会引发重复键异常并回滚事务（除非被捕获和处理）。

触发器ERROR上的回滚事务

2 个答案: