嵌套PreparedStatement的最佳实践是什么？

Question

我有几个实例，其中一部分旧版 sql 语句基于依赖项。 例如。

if (x !=null)
{
  SQL = "SELECT z WHERE x > y";
}
else
{
  SQL = "SELECT z WHERE x <= y";
} 

SQL2 = SQL + " JOIN a ON b";

我正在根据这个遗留代码创建PreparedStatements。 这里的最佳实践是什么。 我应该为var SQL 创建一个PreparedStatement 并将其嵌套在SQL2 中，还是应该有多个基于SQL2 且没有嵌套的PreparedStatement，还是完全不同的东西？

该代码比示例复杂得多，因为 SQL var 在许多长且复杂的 SQL 查询中重用。

编辑：项目设计需要使用PreparedStatements，我目前没有使用库的选择。

Answer 1

>我应该为 var SQL 创建一个PreparedStatement 并将其嵌套在 SQL2 中吗

否

>或者是否应该有多个基于 SQL2 且不嵌套的PreparedStatement

是

此外：如果您可以为每个查询创建一个字符串，这样会更好。 我真的不喜欢将 SQL 与代码混合在一起。 它使调试和理解变得更加困难，您无法复制/粘贴到 SQL 工具来轻松测试它。 通过将 SQL 与代码分离，您可以将查询与操作（实际的获取）隔离开来，并且更容易维护。 另外，如果代码不是您的，它会更容易理解。

看起来重复字符串并不重要，重点是尽可能简化语句。

我会做这样的事情：

final class DatabaseQueries {
    public final static String SOME_SCENARIO       = "SELECT z WHERE x > y JOIN A, B ";
    public final static String SOME_OTHER_SCENARIO = "SELECT z WHERE x <= y JOIN A, B";
 }

然后从你的类中使用它：

 PreparedStatement pstmt = getCon().prepareStatement( getQuery() );


 private String getQuery() { 
     if( x != null ) { 
          return DatabaseQueries.SOME_SCENARIO;
     } else { 
           return DatabaseQueries.SOME_OTHER_SCENARIO;
     }
 }

在创建类“DatabaseQueries”时，你会发现你重复了很多字符串，我认为用其他常量替换某些部分会很好。

final class DataBaseQueries { 
    // this one is private
    private final static String JOIN_A_B = " join A, B ";
    public final static String SOME_SCENARIO       = "SELECT z WHERE x > y " + JOIN_A_B ;
    public final static String SOME_OTHER_SCENARIO = "SELECT z WHERE x <= y " + JOIN_A_B ;

}

这里的重点是让事情变得更简单。 这是第一步。 在第二步中，您可以创建一个类来创建那些非常非常复杂的查询，但可能是 YAGNI。

如果查询太多，您可以替换它以从 ResourceBundle 加载它们，如下所示 问题

我希望这有帮助。

Answer 2

Ibatis 非常擅长这一点。

<select id="queryName" parameterClass="com.blah.X"><!<[CDATA[
  SELECT z
  FROM a
  JOIN b ON a.id = b.foreign_key
  WHERE

  <isNotNull property="value">
    x > y
  </isNotNull>

  <isNull property="value">
    x <= y
  </isNull>

]]></select>

这只是 Ibatis 功能的一小部分，但它非常轻量级。 技术精湛。

Answer 3

这不是准备好的语句参数的正确使用方式。 参数只能用于代替 SQL 表达式中的文字值。 不是表名、列名或其他 SQL 语法。

您可以使用一些库来构建 SQL 查询的一部分。 我在 PHP 中开发了一个类似的库，名为 Zend_Db_Select< /代码>。

编辑：我在 google 上搜索了一些类似的 Java 库，发现这个选项可能会有所帮助：

• Squiggle 是一个用于动态生成 SQL SELECT 语句的小型 Java 库。 [它的]最佳点是需要使用在运行时更改的条件来构建复杂查询的应用程序。 通常，弄清楚如何构建这个字符串可能会非常痛苦。 Squiggle 消除了大部分这种痛苦。

它是免费的，并根据 Apache 许可证提供，这是一个非常灵活的开源许可证。

谷歌搜索“java query builder”发现了许多其他选项，但有些不是免费的。 有些是可视化查询构建器，而不是编程式查询构建器。

另一种选择是使用复杂的对象关系映射框架（如 Hibernate），但这对于您当前的任务来说似乎有些过分了。

Answer 4

这是类似的问题

简短的回答 - 没有最好的办法。 你可能会得到这样的结果：

String selectQuery =
  (new SelectQuery())
  .addColumns(t1Col1, t1Col2, t2Col1)
  .addJoin(SelectQuery.JoinType.INNER_JOIN, joinOfT1AndT2)
  .addOrderings(t1Col1)
  .validate().toString();

但对我来说，情况更糟。

Answer 5

我想一方面有一种纯粹的对象方法，如果您试图理解遗留代码，这可能对您没有太大帮助。 我发现，在重构真正令人讨厌的遗留代码时，而不是追求“完美”，简化小片段通常更好、更容易，尽可能模块化和有据可查，而无需立即重写整个应用程序。 我发现重构坏代码对我来说最大的障碍是，如果我采取太大的步骤，我就不能再确信我没有破坏任何东西 - 如果它那么糟糕，可能就没有单元测试，并且可能存在未定义或未记录的行为。

我至少会首先分解逻辑和 sql。 您不想要的事情是这样的：

String sql = "yadda yadda yadda ? yadda yadda WHERE ";
if (mystery condition 1){
   sql = sql + " page=?"
}
else if (mystery condition 2)
{
 sql = sql + " ORDER BY ? "
}

一段时间后您将无法知道正在构建哪些语句。 节省复制初始 sql 的时间是不值得的。 它可能更容易理解为：

private static final String FIND_PAGE_QUERY = "...."
private static final String ORDER_BY_QUERY =" ..."

if (mystery condition 1){
   return process(FIND_PAGE_QUERY, parameters);
}
else if (mystery condition 2)
{
  return process(ORDER_BY_QUERY, parameters);
}

然后创建一些东西来包装您的查询和传递的参数，如 Spring JDBC Row Mappers 或类似的东西。 这仍然很丑陋，但是作为您所拥有的增量步骤，它很容易做到，并且至少会解决查询生成的一些混乱。

Answer 6

另一种方法是将条件移至 SQL 语句本身，这样您只需要一个语句。 它会是这样的：

SELECT z WHERE (? IS NOT NULL AND x > y) OR (? IS NULL AND x <= y)

然后为参数绑定一个适当的值。

不确定这是最好的方法......人们可能会发现代码不太清晰。 但这是一种可能性。