<%@ page language="java" import="java.util.*,java.sql.*,computer.computer073.*" pageEncoding="GB18030"%> <% request.setCharacterEncoding("GBK"); String username = request.getParameter("username"); String password = request.getParameter("password"); String kind = request.getParameter("kind"); System.out.println(username+"----"+password+"------"+kind); Connection conn = DB.getConn(); String sql = null; if(kind.equals("borrower")){ sql = "select count(*) from borrower where bno = '"+username+"' and bpw = '"+password+"'"; }else sql = "select count(*) from managerlogin where username = '"+username+"' and pw = '"+password+"'"; System.out.println(sql); Statement stmt = conn.createStatement(); ResultSet rs = DB.executeQuery(stmt,sql); rs.next(); int count = rs.getInt(1); DB.close(rs); DB.close(stmt); DB.close(conn); if(count==1){ if(kind.equals("borrower")){ response.sendRedirect("borrower.jsp?bno="+username); }else{ response.sendRedirect("manager.jsp"); } }else{ response.sendRedirect("login.jsp"); } %> 这段代码有什么错误
时间: 2024-03-06 11:48:06 浏览: 168
这段代码有以下几个问题:
1. 在使用数据库连接之前,需要先确保数据库连接已经建立并且可用。代码中没有检查 `DB.getConn()` 方法返回的连接是否为 `null` 或者是否已经关闭,如果连接不可用,会导致 `NullPointerException` 或者 `SQLException` 等异常。
2. 在构造 SQL 查询语句时,使用了字符串拼接的方式,存在 SQL 注入的风险。建议使用预编译的 `PreparedStatement` 对象来执行 SQL 语句,可以提高安全性和执行效率。
3. 在查询结果集之前,应该先判断结果集是否为空,否则调用 `rs.next()` 方法可能会抛出 `SQLException` 异常。
4. 在使用完 `ResultSet` 和 `Statement` 对象之后,需要及时关闭它们以释放相关资源。代码中使用了 `DB.close(rs)`、`DB.close(stmt)` 和 `DB.close(conn)` 方法来关闭这些对象,但是这些方法的实现没有给出。建议自定义一个 `DBUtil` 类,提供静态方法封装关闭对象的操作。
5. 在重定向之前,需要先设置响应头(`response`)中的编码格式,否则可能会出现乱码。例如可以在 JSP 页面中添加以下代码:
```
<% response.setContentType("text/html; charset=UTF-8"); %>
```
6. 在重定向之前,需要先关闭响应输出流(`response.getWriter()`),否则可能会抛出 `IllegalStateException` 异常。可以在 JSP 页面中添加以下代码:
```
<% response.getWriter().close(); %>
```
综上所述,建议对这段代码进行重构,遵循良好的编码规范和安全性原则。
阅读全文