Question

我想问一下优化这三个查询的最佳方法。该功能将在国家项目中被数百万用户使用，我希望它尽可能优化（快速）。

请添加任何有用的评论或回答:) 并提前谢谢你。

 try {
    //so we don't execute empty batches
    boolean flag=false;
    conn = dataSource.getConnection();
    autoCommitValue = conn.getAutoCommit();
    conn.setAutoCommit(false);
    stmt = conn.prepareStatement("DELETE FROM table1 WHERE input = ?");
    stmt.setLong(1, input);
    stmt.executeUpdate();
    stmt = conn
            .prepareStatement("INSERT INTO table1 (c1)  values (?)");

    for (Object Info : Info List) {
        if (Info .getType() == 0) {
            flag=true;
            stmt.setInt(1, Info.getC1());
            stmt.addBatch();
        }
    }
    if(flag){
    result = stmt.executeBatch();
    flag=false;
    }
    stmt = conn.prepareStatement("INSERT INTO table1 (c1,c2,c3) values (?,?,?)");

    for (Object Info : Info List) {
        if (Info .getType() != 0) {
            flag=true;
            stmt.setLong(1, input);
            stmt.setInt(2, Info.getC1());
            stmt.setInt(3, resInfo.getC2());
            stmt.addBatch();
         }
    }
    if(flag){
    result = stmt.executeBatch();
    }
    conn.commit();

} catch (SQLException e) {
     conn.rollback();
    throw new SQLException();
} finally {
    if (conn != null) {
        // now we return the connection to its original state
        conn.setAutoCommit(autoCommitValue);
    }
    closeDBObjects(conn, stmt);
}

Answer 1

在尝试优化代码之前，我要做的第一件事就是进行编译。以下行不编译：

for (Object Info : Info List) {
    if (Info .getType() != 0) {
    ...

然后，我会通过尊重Java命名约定，通过将这个长方法分成几个，并使用比flag更好的名称来使代码更清晰。

然后我会测量，测试并查看是否存在性能问题。如果我有一个，我会试着找到它的来源。只有当我确定它来自这种方法（这不太可能）时，我才会尝试优化它。

我们假设这个方法需要优化。我要做的第一件事是确保存在适当的索引，以加快delete语句。

如果列表为空，我会避免准备这两个语句。你可以使用flag变量来避免它们的执行，但你不能避免它们的准备。

然后我会尝试查看是否需要使用批处理，或者使用多个批处理是否有用，具体取决于列表的大小，我的数据库建议的内容，以及测量和测试所说的内容。

但正如我所说。做到这一切是过早的优化，这是所有邪恶的根源。尝试使代码正确，清洁和可维护之前。

在一个事务中执行两个不同的批处理查询和删除查询（Java）

1 个答案: