2017-03-25 118 views
0

我有一个mysql函数将相应地更改记录。但循环只执行一次,并留下循环与此条件。 “IF v_finished = 1 THEN LEAVE get_stock; END IF;”MYSQL循环在功能不能正常工作

但是它应该执行多次。就像在我的测试情况下,3次

BEGIN 

DECLARE P_stock int(11); 
DECLARE P_product int(11); 
DECLARE V_From_warehouse int(11); 
DECLARE V_To_warehouse int(11); 
DECLARE v_finished INTEGER DEFAULT 0; 
DECLARE V_To_warehouse_stock int(11); 
DECLARE V_From_warehouse_stock int(11); 



declare cur1 cursor for 
     SELECT material_transfer_details.product_id , material_transfer_details.quantity FROM 
      material_transfers, 
      material_transfer_details 
     WHERE 
      material_transfers.id = material_transfer_details.mtm_id 
     AND 
      material_transfers.status = 'Y' 
     AND 
      material_transfers.id = V_MTM_id; 

DECLARE CONTINUE HANDLER 
     FOR NOT FOUND SET v_finished = 1; 

SELECT warehouse_from INTO V_From_warehouse FROM material_transfers WHERE id =V_MTM_id; 

SELECT warehouse_to INTO V_To_warehouse FROM material_transfers WHERE id =V_MTM_id; 


OPEN cur1; 

get_stock: LOOP 

IF v_finished =1 THEN 
     LEAVE get_stock; 
    END IF; 

fetch cur1 into P_product , P_stock; 

SELECT quantity INTO V_To_warehouse_stock from stocks where warehouse_id = V_To_warehouse and product_id = P_product; 

SELECT quantity INTO V_From_warehouse_stock from stocks where warehouse_id = V_From_warehouse and product_id = P_product; 



IF (V_To_warehouse_stock IS NOT NULL) 
THEN 
    UPDATE 
     stocks SET quantity = quantity - P_stock 
    WHERE 
     warehouse_id = V_to_warehouse 
     AND 
      product_id = P_product; 
ELSE 

    INSERT INTO stocks(product_id , warehouse_id , quantity ,status, created_datetime , updated_datetime) values 
     (P_product , V_to_warehouse , 0-P_stock , 'Y', sysdate() , sysdate()); 


END IF; 



IF (V_From_warehouse_stock IS NOT NULL) 
THEN 
    UPDATE 
     stocks SET quantity = quantity + P_stock 
    WHERE 
     warehouse_id = V_from_warehouse 
     AND 
      product_id = P_product; 
ELSE 

    INSERT INTO stocks(product_id , warehouse_id , quantity ,status, created_datetime , updated_datetime) values 
     (P_product , V_from_warehouse , P_stock , 'Y', sysdate() , sysdate()); 


END IF; 

SET P_stock = 0; 
SET P_product = 0; 

END LOOP get_stock; 

CLOSE cur1; 




UPDATE material_transfers SET Status = 'N' WHERE id= V_MTM_id; 

UPDATE material_transfer_details SET Status = 'N' WHERE mtm_id = V_MTM_id; 

return '00000'; 

END 
+0

cur1有多少行,我的意思是选择scrict返回任何行或不。首先检查一次。 – Rams

+0

它返回3行。我已经手动检查过 –

+0

“未找到的继续处理程序”不仅会对光标起作用,而且还会在例如'SELECT warehouse_to INTO V_To_warehouse ...'没有找到任何行 - 你显然期望得到,因为你在'IF(V_To_warehouse_stock IS NOT NULL)'中测试了这个行。您可以添加其他处理程序;您可以在获取新行之前设置一个变量(这样您可以在处理程序中检查该变量);或其他类似的方式。 – Solarflare

回答

0

两件事情:

首先,改变你的代码。

get_stock: LOOP 

    SET v_finished = FALSE; 

    fetch cur1 into P_product , P_stock; 

    IF v_finished =1 THEN 
    LEAVE get_stock; 
    END IF; 

既然你在做其他的事情,可能会跳闸处理程序,重置v_finished,然后从游标提取,并才把测试是否退出循环。按照书面的说法,如果你没有提前处理处理程序,你应该在完全错误的地方进行测试,并且会留在循环中太长时间。

接下来...请确定您了解SELECT ... INTO。我不认为它完全符合你的想法。

标量子查询是一种更安全的解决方案:

SET V_To_warehouse_stock = (SELECT quantity from stocks where warehouse_id = V_To_warehouse and product_id = P_product); 

如果SELECT ... INTO未返回行,变量的值不会改变。它保留了它以前的价值,如果存在的话,那很少是你期望的。

最好避免在这样的距离上进行鬼怪行为,这是一个容易陷入陷阱的陷阱。

查看https://dba.stackexchange.com/a/35207/11651的效果示例。