使用存储过程插入表中

Question

我有这个简单的存储过程，它会在我的Orders表中添加一列

create or replace PROCEDURE ADD_ORDER 
(
  CUSTOMER_ID IN NUMBER 
, NEW_ORDER_ID OUT NUMBER 
) AS 
DECLARE
  NEW_ORDER_ID := MAX(ORDERS.ORDER_NO) + 1;
BEGIN
  INSERT INTO ORDERS(ORDER_NO, REP_NO, CUST_NO, ORDER_DT, STATUS)
  VALUES( NEW_ORDER_ID, 36, CUSTOMER_ID, CURDATE(), 'C')
END ADD_ORDER;

这是说声明部分不在正确的位置（我认为），也不应该在那里结束。 这是错误屏幕上的内容：

错误（6,1）：PLS-00103：在预期以下情况之一时遇到符号“DECLARE”：
begin function pragma procedure subtype type current cursor 删除存在先前的外部语言 在“DECLARE”之前插入符号“begin”以继续。

错误（11,1）：PLS-00103：在预期以下之一时遇到符号“END”：，; 返回 返回

谁能告诉我这里出了什么问题？

Answer 1

正如已经提到的，select 的最大订单号然后使用它来插入一行是一个坏主意。 如果两个进程同时执行此操作，它们会尝试插入具有相同订单号的行。

更好地使用 Oracle 的内置特性SEQUENCE或IDENTITY 。

以下是创建表的方法：

CREATE TABLE orders 
(
  order_no  NUMBER(8) GENERATED ALWAYS AS IDENTITY,
  rep_no    NUMBER(3)    DEFAULT 36       NOT NULL,
  cust_no   NUMBER(8)                     NOT NULL,
  order_dt  DATE         DEFAULT SYSDATE  NOT NULL,
  status    VARCHAR2(1)  DEFAULT 'C'      NOT NULL
);

这就是您的程序的样子：

CREATE OR REPLACE PROCEDURE add_order 
(
  in_cust_no   IN  NUMBER,
  out_order_no OUT NUMBER 
) AS
BEGIN
  INSERT INTO ORDERS(cust_no) VALUES (in_cust_no)
    RETURNING order_no INTO out_order_no;
END add_order;

演示： https://dbfiddle.uk/?rdbms=oracle_18&fiddle=4b49723c15eb810c01077286e171bc95

Answer 2

您的代码中存在语法错误。

 NEW_ORDER_ID := MAX(ORDERS.ORDER_NO) + 1; --not be used liked it.

使用下面的代码

create or replace PROCEDURE ADD_ORDER 
(
  CUSTOMER_ID IN NUMBER 
, NEW_ORDER_ID OUT NUMBER 
) AS 

  V_NEW_ORDER_ID NUMBER;
BEGIN
    SELECT NVL(MAX(ORDER_NO),0)+1 INTO V_NEW_ORDER_ID FROM ORDERS;
    
    INSERT INTO ORDERS(ORDER_NO, REP_NO, CUST_NO, ORDER_DT, STATUS)
    VALUES( V_NEW_ORDER_ID, 36, CUSTOMER_ID, CURDATE(), 'C');
    
    NEW_ORDER_ID:=V_NEW_ORDER_ID;
   
    /* 
        * CURDATE() -> I am assuming it is user defined function. You can also use SYSDATE, CURRENT_DATE istead of CURDATE()
        * OUT Parameter is a write-only parameter. You cannot read value from OUT Parameter
    */
    
END ADD_ORDER;

Answer 3

很少有事情需要是正确的。

如果您希望编写PROCEDURE或FUNCTION ，则不必使用DECLARE关键字。 在编写测试脚本或其他东西时，您应该使用DECLARE关键字来声明变量。 在编写程序时，

1. 所有参数都应该在括号内。
2. 变量应该在AS和BEGIN关键字之间定义，并且应该给出数据类型。

如果您需要获取 ORDERS TAB 的最大数量，您必须为此编写一个 SQL 查询。 因为MAX function 只能在SQL内部使用。 此外，如果您有兴趣，在 ORACLE 中有一个内置功能调用SEQUENCE可用于NEW_ORDER_ID 。 您可以通过下面的链接进行检查。 为 oracle plsql 添加序列

我对您的代码做了一些更改。 希望它现在工作正常。 请访问这里。

CREATE or REPLACE PROCEDURE ADD_ORDER (
    CUSTOMER_ID     IN NUMBER 
    NEW_ORDER_ID    OUT NUMBER
) AS

  CURSOR get_max_order_no IS
    SELECT MAX(order_no)
    FROM ORDERS;
  rec_ NUMBER := 0;  

BEGIN

  OPEN get_max_order_no;
  FETCH get_max_order_no INTO rec_;
  CLOSE get_max_order_no;

  NEW_ORDER_ID := rec_ + 1;

  INSERT INTO ORDERS
    (ORDER_NO, REP_NO, CUST_NO, ORDER_DT, STATUS)
  VALUES 
    (NEW_ORDER_ID, 36, CUSTOMER_ID, SYSDATE, 'C');

END ADD_ORDER;