Oracle存储过程实现分页

Question

几乎每一个WEB应用都会用到分页，因此，将其做得通用高效就变得非常重要了，根据自己的想法用存储过程做了一个分页的存储过程，与大家分享，希望能够通过讨论得到更好的解决方案。          之所以用存储过程，是因为以后需要修改的话不需要修改程序代码，只需要修改存储过程的代码。但这个例子是在存储过程里动态生成的SQL语句，不知道会不会因此失去存储过程一次编译和快速的特点。代码如下： 1、首先建立一个包，用户创建一个游标类型create or replace package pkg_query as
  type cur_query is ref cursor;
end pkg_query;2、创建存储过程CREATE OR REPLACE PROCEDURE prc_query
       (p_tableName        in  varchar2,   --表名
        p_strWhere         in  varchar2,   --查询条件
        p_orderColumn      in  varchar2,   --排序的列
        p_orderStyle       in  varchar2,   --排序方式
        p_curPage          in out Number,  --当前页
        p_pageSize         in out Number,  --每页显示记录条数
        p_totalRecords     out Number,     --总记录数
        p_totalPages       out Number,     --总页数
        v_cur              out pkg_query.cur_query)   --返回的结果集
IS
   v_sql VARCHAR2(1000) := '';      --sql语句
   v_startRecord Number(4);         --开始显示的记录条数
   v_endRecord Number(4);           --结束显示的记录条数
BEGIN
   --记录中总记录条数
   v_sql := 'SELECT TO_NUMBER(COUNT(*)) FROM ' || p_tableName || ' WHERE 1=1';
   IF p_strWhere IS NOT NULL or p_strWhere <> '' THEN
       v_sql := v_sql || p_strWhere;
   END IF;
   EXECUTE IMMEDIATE v_sql INTO p_totalRecords;
   
   --验证页面记录大小
   IF p_pageSize < 0 THEN
       p_pageSize := 0;
   END IF;
   
   --根据页大小计算总页数
   IF MOD(p_totalRecords,p_pageSize) = 0 THEN
       p_totalPages := p_totalRecords / p_pageSize;
   ELSE
       p_totalPages := p_totalRecords / p_pageSize + 1;
   END IF;
   
   --验证页号
   IF p_curPage < 1 THEN
       p_curPage := 1;
   END IF;
   IF p_curPage > p_totalPages THEN
       p_curPage := p_totalPages;
   END IF;
   
   --实现分页查询
   v_startRecord := (p_curPage - 1) * p_pageSize + 1;
   v_endRecord := p_curPage * p_pageSize;
   v_sql := 'SELECT * FROM (SELECT A.*, rownum r FROM ' ||
            '(SELECT * FROM ' || p_tableName;
   IF p_strWhere IS NOT NULL or p_strWhere <> '' THEN
       v_sql := v_sql || ' WHERE 1=1' || p_strWhere;
   END IF;
   IF p_orderColumn IS NOT NULL or p_orderColumn <> '' THEN
       v_sql := v_sql || ' ORDER BY ' || p_orderColumn || ' ' || p_orderStyle;
   END IF;
   v_sql := v_sql || ') A WHERE rownum <= ' || v_endRecord || ') B WHERE r >= '
            || v_startRecord;
   DBMS_OUTPUT.put_line(v_sql);
   OPEN v_cur FOR v_sql;
END prc_query;3、JAVA代码里取出结果集String sql= "{ call prc_query(?,?,?,?,?,?,?,?,?) }";
CallableStatement call = con.prepareCall(sql);// ……中间数据设置及注册省略call.registerOutParameter(9, OracleTypes.CURSOR);// 取出结果集(ResultSet) call.getObject(9);

Answer 1

我看了你的代码，建议你做一下改动，我看到你使用的是动态sql的方式查询数据库以返回游标的形式实现sql查询，但是希望你使用动态sql的时候不要使用字符串拼接的低级策略，要使用预编译，或者说是用绑定变量。这样做首先节省内存，因为生成新的sql语句串需要消耗内存，再者，如果每次查询都生成新sql语句那么每次查询oracle就不得不再次编译sql语句，这样非常影响速度，还有，用字符串拼接，别人传入的参数可能是sql攻击语句，拼接以后会影响sql语句原形，这样就会影响执行结果，最后，字符串拼接sql语句不利于代码管理，因为变量注入与sql语句杂糅在一起再加上字符串拼接符和过多的引号，让人感觉代码很繁琐，难以理解，如果sql语句很长，那么改动代码时就不得不去sql语句中查找。应该尽量的吧sql语句与注入变量分开。

Answer 2

有个问题想问下:
   --验证页面记录大小 
   IF p_pageSize  < 0 THEN 
       p_pageSize := 0; 
   END IF; 
=====================
这里如果传进来的每页显示的记录条数若小于0的话,应该默认指定一个大于0的数字吧,而不能指定为0吧,否则后面计算总页数会出现异常(除0)操作.