使用JSP Servlet上传图片至数据库时出现错误求助

我来帮你梳理下代码里可能存在的问题，以及对应的解决办法，这些都是这类上传场景中常见的坑：

1. 缺少Multipart请求处理配置 #

你的表单用了enctype="multipart/form-data"，但Servlet默认不会处理这种类型的请求，必须在Servlet类上添加@MultipartConfig注解才能让request.getPart()正常工作。如果没加这个注解，getPart()要么返回null，要么直接抛出异常。

解决办法：在你的Servlet类顶部添加注解：

@MultipartConfig(
    maxFileSize = 1024 * 1024 * 5,    // 限制单个文件大小为5MB
    maxRequestSize = 1024 * 1024 * 10 // 限制整个请求大小为10MB
)
public class Slider extends HttpServlet {
    // 你的doPost等方法...
}

2. SQL语句存在注入风险+语法隐患 #

你直接把id拼接到SQL字符串里，这不仅会导致SQL注入漏洞，如果id是数字类型，加单引号还可能触发SQL语法错误。应该用预编译参数来传递id，而不是字符串拼接。

修改SQL部分的代码：

// 替换原来的拼接写法，用?作为占位符
st = con.prepareStatement("update slider set img=? where id=?");
if(inputStream != null) {
    st.setBlob(1, inputStream);
}
// 根据id的实际类型选择set方法：如果是字符串用setString，数字用setInt
st.setString(2, id);
// 如果id是整数类型，改成：st.setInt(2, Integer.parseInt(id));

3. 资源未关闭导致泄漏 #

你的Connection、PreparedStatement、InputStream用完后都没有关闭，这会导致数据库连接泄漏、内存泄漏等问题。推荐使用Java的try-with-resources语法，它会自动关闭实现了AutoCloseable接口的资源。

重构后的核心代码片段：

try (Connection con = rd.getConnection();
     PreparedStatement st = con.prepareStatement("update slider set img=? where id=?")) {
    String id = request.getParameter("id");
    System.out.println("id is " + id);

    Part filePart = request.getPart("img");
    System.out.println(filePart);
    InputStream inputStream = null;

    if (filePart != null) {
        System.out.println("image name: " + filePart.getName());
        System.out.println("file size: " + filePart.getSize());
        System.out.println("content type: " + filePart.getContentType());
        inputStream = filePart.getInputStream();
        System.out.println("input stream: " + inputStream);
    }

    if(inputStream != null) {
        st.setBlob(1, inputStream);
    }
    st.setString(2, id);
    int result = st.executeUpdate();

    if(result > 0) {
        System.out.println("file uploaded successfully...");
    }
} catch(Exception e) {
    // 打印完整堆栈信息，而不是只打印异常对象，这样才能定位具体错误
    e.printStackTrace();
} finally {
    // 手动关闭InputStream（如果用try-with-resources包裹inputStream也可以）
    if (inputStream != null) {
        try {
            inputStream.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

4. 数据库驱动类过时（可选） #

如果你的MySQL版本是8.0及以上，com.mysql.jdbc.Driver已经被弃用，建议换成com.mysql.cj.jdbc.Driver，同时数据库连接URL需要加上时区参数，比如：jdbc:mysql://localhost:3306/your_db?serverTimezone=UTC。

5. 异常信息不详细 #

你当前只打印System.out.println(e);，这只能看到异常的简单描述，无法获取错误发生的具体位置和调用链。改成e.printStackTrace();可以打印完整的堆栈轨迹，帮你快速定位问题。

额外检查点 #

• 确认数据库中slider表的img字段类型是BLOB（如果图片较大，用LONGBLOB）；
• 检查ReadProperty类的getConnection()方法是否正确加载了配置，返回的连接是否有效；
• 如果request.getParameter("id")获取不到值，可以尝试通过Part来读取隐藏字段：

  Part idPart = request.getPart("id");
  String id = new String(idPart.getInputStream().readAllBytes(), StandardCharsets.UTF_8);
  

内容的提问来源于stack exchange，提问作者Anirudha Potdar