此处不允许列

Question

CREATE OR REPLACE TRIGGER update_QOH
BEFORE INSERT ON ORDERLINE
FOR EACH ROW
DECLARE
    QOH_PRODUCT PRODUCT.QOH%TYPE;
    ORD_NO ORDERS.ORDER_NO%TYPE;
BEGIN
    SELECT QOH INTO QOH_PRODUCT FROM PRODUCT
    WHERE :old.product_no = :new.product_no;

    SELECT ORDER_NO INTO ORD_NO FROM ORDERLINE
    WHERE :old.order_no = :new.order_no;

    IF (:new.QTY <= QOH_PRODUCT) THEN   
        UPDATE PRODUCT SET QOH = QOH_PRODUCT - :new.QTY;
    ELSE
        send_email(ord_no, 'Backorder');

        INSERT INTO BACKORDER
        VALUES (backorder_no_seq.NEXTVAL, :new.product_no, :new.qty, SYSDATE);

        INSERT INTO PRODVENDOR
        VALUES (po_no_seq.NEXTVAL, vendor_no, :new.product_no, vend_qty,
            shipping_method, SYSDATE, NULL, NULL, NULL);
    END IF; 
END;
/
-------------------------------------------------------------------------------
Error(13,3): PL/SQL: SQL Statement ignored

Error(13,91): PL/SQL: ORA-00984: column not allowed here
--------------------------------------------------------------------------------

产品表（P_no、QOH 等）

订单表（OL_no、QTY 等）

延期交货表（B_no、B_QTY 等）

供应商表（V_no 等）

我需要确保当客户购买产品时，有足够的库存产品表中的QOH，如果有，则应降低（更新）产品中的QOH。如果没有，请向客户发送电子邮件，更新延期交货表，然后应从供应商处订购产品。

Answer 1

将此行更改为：

QOH = :old.QOH - :new.QTY

在

QOH := :old.QOH - :new.QTY

PL/SQL 中 := 是赋值运算符，因此在设置 PL/SQL 变量时使用它。

无论编译错误的来源是什么，从触发器发送电子邮件似乎都是一个坏主意。

Answer 2

首先，考虑将插入放入过程而不是触发器中。 （也许还有一个更有用的名字）。如果其他表稍后获得自己的触发器，则从此处插入或更新其他表可能会变得混乱，并且很难跟踪何时何地发生了什么。

其次，这看起来不能很好地处理并发插入 - 同一产品的两个新 ORDERLINE 插入同时会尝试更新产品 QOH，结果可能是意外或不需要的结果 - 例如，QOH 可能会变为负数。您还可能会向供应商收到多个订单；每个请求缺货产品的订单行都将向供应商发出新订单，即使每个订单行数量为 1 并且您一次向供应商订购 100 个订单也是如此。

第三，各种代码错误；我将从几个更明显的开始：

a) 您使用 WHERE :old.product_no = :new.product_no 从 PRODUCT 和 ORDERLINE 中进行选择。我不确定 :OLD 是否在插入前触发器中设置，但如果是，它将与 :NEW 或 null 相同，所以你可能会出现 ORA-02112 或 ORA-01403 错误，因为它会找到所有行，也可能找不到。

b) 您从 ORDERLINE 中进行的选择将不会在第一个订单行上返回任何行，而从第三个订单行开始将返回多行，因此您将再次收到 ORA-01403 和 ORA-02112 错误。但这毫无意义，因为您只是选择要查询的值。您可以仅在电子邮件调用中使用 :NEW 值。

c) 您对 PRODUCT 的更新没有 WHERE 子句，因此所有 QOH 值都将更新。

CREATE OR REPLACE TRIGGER update_QOH
BEFORE INSERT ON ORDERLINE
FOR EACH ROW
DECLARE
    QOH_PRODUCT PRODUCT.QOH%TYPE;
BEGIN
    SELECT QOH INTO QOH_PRODUCT FROM PRODUCT
    WHERE product_no = :new.product_no;

    IF (:new.QTY <= QOH_PRODUCT) THEN   
        UPDATE PRODUCT SET QOH = QOH_PRODUCT - :new.QTY;
        WHERE product_no = :new.product_no;
    ELSE
        send_email(:new.order_no, 'Backorder');

        INSERT INTO BACKORDER
        VALUES (backorder_no_seq.NEXTVAL, :new.product_no, :new.qty, SYSDATE);

        INSERT INTO PRODVENDOR
        VALUES (po_no_seq.NEXTVAL, vendor_no, :new.product_no, vend_qty,
            shipping_method, SYSDATE, NULL, NULL, NULL);
    END IF; 
END;
/

d) vend_no、vend_qty 和 shipping_method 在 PRODVENDOR 的插入中来自哪里？这是跳出的唯一明显的编译错误。

e) 您没有在插入中指定表列。如果您的值顺序错误或有任何缺失，这将导致编译错误，但仅通过查看代码无法判断。 （您还没有说 @WW 更改后的“so may”错误是什么，所以不知道这是否相关）。如果稍后添加另一列，则此触发器将变得无效，因此通常最好显式列出这些列。

从功能上讲，您似乎正在发送一封电子邮件，说明整个订单都处于延期交货状态，而不仅仅是该产品；并且当前订单与您在 BACKORDER 和 PRODVENDOR 中输入的内容之间似乎没有任何链接。

Answer 3

您确定 QOH 是 ORDERLINE 表中的字段吗？ ：旧。和：新的。关键字仅适用于安装了触发器的表。