在mysql的存储过程中我执行了下面的一段 OPEN C_COUNT_LIST; LOOP FETCH C_COUNT_LIST INTO CUSTOMER_PROV, C_COUNT; EXIT WHEN C_COUNT_LIST%NOTFOUND; set @RPT_SQL :='UPDATE RPT_BL_LINE_COUNT T SET T.N_MONTH = ' || C_COUNT ||' ' ' WHERE T.PROV_CODE= ' || CUSTOMER_PROV || ' AND T.PAY_CDDE=''01'' AND T.FEETYPE=''0'' ' ' AND T.VERSION= ' || G_VERSION || ' AND T.SETTLEMONTH= ' || RPT_SETTLEMONTH || ' '; SELECT @RPT_SQL; PREPARE STMT FROM @RPT_SQL; EXECUTE STMT; DEALLOCATE PREPARE STMT; END LOOP; CLOSE C_COUNT_LIST; 我发现FETCH C_COUNT_LIST INTO CUSTOMER_PROV, C_COUNT;这段并没有给CUSTOMER_PROV和C_COUNT赋值 导致两个变量的值为空 当然我在存储过程开始已经定义了CUSTOMER_PROV和C_COUNT变量 帮我分析并解决
时间: 2024-04-29 14:19:19 浏览: 83
根据你提供的代码,我看到在存储过程开始时定义了CUSTOMER_PROV和C_COUNT变量,但是在循环开始前并没有给它们赋初值。因此,在第一次进入循环时,FETCH语句无法将结果集中的值赋给这两个变量,导致它们的值为空。
解决这个问题的方法是在循环开始前给它们赋一个初始值。例如,可以将定义变量的语句改成:
DECLARE CUSTOMER_PROV VARCHAR(20) DEFAULT '';
DECLARE C_COUNT INT DEFAULT 0;
这样,在循环开始前,CUSTOMER_PROV和C_COUNT就会有一个默认值,避免了空值的问题。
另外,你在拼接SQL语句时,建议使用参数绑定的方式,而不是直接拼接字符串。这可以避免SQL注入攻击,并且可以提高性能和可读性。例如:
SET @RPT_SQL = 'UPDATE RPT_BL_LINE_COUNT T SET T.N_MONTH = ? WHERE T.PROV_CODE= ? AND T.PAY_CDDE=''01'' AND T.FEETYPE=''0'' AND T.VERSION= ? AND T.SETTLEMONTH= ?';
然后,使用PREPARE语句和EXECUTE语句执行动态SQL,并使用参数绑定将变量传递给它。例如:
PREPARE STMT FROM @RPT_SQL;
EXECUTE STMT USING C_COUNT, CUSTOMER_PROV, G_VERSION, RPT_SETTLEMONTH;
DEALLOCATE PREPARE STMT;
这样可以更加安全和高效地执行动态SQL语句。
阅读全文