通过 JDBC 读取“BLOB”需要太长时间

Question

当我从数据库读取图像时，它会立即出现在 JLabel 中，但需要太多时间才能完成从数据库传输 BLOB 的过程。

public byte[] selectImage(int Id) throws SQLException {
    ResultSet res=null;
    int c=0;

    try {
        Class.forName(driver);
        con = DriverManager.getConnection(connectionURL);
    } catch (SQLException ex) {
        Logger.getLogger(Connect.class.getName()).log(Level.SEVERE, null, ex);
    } catch (ClassNotFoundException ex) {
        Logger.getLogger(Connect.class.getName()).log(Level.SEVERE, null, ex);
    }

    System.out.println(con.getAutoCommit());

    statement = con.createStatement() ;
    res = statement.executeQuery("SELECT PHOTO FROM CUSTOMER where ID="+Id) ;

    while(res.next() && (res!=null)) {
        Blob bodyOut = res.getBlob("PHOTO");
        int length = (int) bodyOut.length();
        System.out.println("   Body Size = "+length);
        imageBytes = bodyOut.getBytes(1, length);
        bodyOut.free(); 
    }

    return imageBytes;
}

Answer 1

考虑将图像存储在数据库之外。仅在数据库中存储足够的信息来让您找到该文件（无论是在文件系统上，还是从 HTTP 服务器，或者您现在存储它，因为它不在数据库中）。二进制数据实际上并不是 RDBMS 优化处理的用例。

另外，您的代码存在一些严重的问题：

1. 最大的问题可能是由于未能使用绑定变量而导致的安全缺陷，又名 PreparedStatement。这是 SQL 注入 101。

2. 您正在使用原始 JDBC。原始 JDBC 很乏味且容易搞砸。例如，您没有关闭 ResultSet 或 Connection，更不用说 statement 变量，它绝对应该是本地的。当您开始关闭它们时，您应该在 finally 块中执行此操作，以确保它总是发生，即使存在错误。

3. 如果您碰巧从查询中获得多个结果（我猜您不应该这样做，但以防万一），您只有碰巧查看 STDOUT 才会知道，并且你只会得到最后一张图像。您的 while 循环可能更好地表示为 if ，以表明您只期望和/或关心第一个结果。如果您关心是否有多个结果，则可能应该使用 if 而不是 while，然后添加后续的 if (rs.next()) throw new MyAppropriatelyNamedException; 所以你知道发生了一些意想不到的事情。

4. 对 res 的 null 检查毫无价值。当您执行检查时，rs.next() 语句将已经抛出 NullPointerException。您可能应该删除该检查。

5. 为什么使用日志框架，只是为了转而使用 System.out.println 来输出一些调试信息？