【问题标题】:Using setTime() or setNull() in PreparedStatement depending on user input根据用户输入在 PreparedStatement 中使用 setTime() 或 setNull()
【发布时间】:2016-04-02 21:59:27
【问题描述】:

我正在我的 JAVA 应用程序和 PostgreSQL 数据库中进行过滤搜索。

我有以下用于选择航班数据的 PostgreSQL 查询:

String query =  "SELECT flight.flight_id, route.from_location, route.to_location, flight.base_cost, flight.departure_date, flight.departure_time, flight.arrival_date, flight.arrival_time, aircraft.manufacturer, aircraft.seats " +
                "FROM flight " +
                "INNER JOIN route ON flight.route_id = route.route_id " +
                "INNER JOIN aircraft ON flight.aircraft_id = aircraft.aircraft_id " +
                "WHERE " +
                "(from_location = CASE WHEN ? IS NULL THEN from_location ELSE ? END) AND " +
                "(to_location = CASE WHEN ? IS NULL THEN to_location ELSE ? END) AND " +
                "(base_cost = CASE WHEN ? IS NULL THEN base_cost ELSE ? END) AND " +
                "(departure_date = CASE WHEN ? IS NULL THEN departure_date ELSE ? END) AND " +
                "(departure_time = CASE WHEN ? IS NULL THEN departure_time ELSE ? END) AND " +
                "(arrival_date = CASE WHEN ? IS NULL THEN arrival_date ELSE ? END) AND " +
                "(arrival_time = CASE WHEN ? IS NULL THEN arrival_time ELSE ? END)";

然后我有一个这样的Java代码:

    ResultSet rs = null;
    PreparedStatement prepStatement = connection.prepareStatement(query);
    try {
                if (!showFromTxtField.getText().trim().isEmpty()) {
                    prepStatement.setString(1, showFromTxtField.getText());
                    prepStatement.setString(2, showFromTxtField.getText());
                } else {
                    prepStatement.setNull(1, java.sql.Types.VARCHAR );
                    prepStatement.setNull(2, java.sql.Types.VARCHAR );
                }
                if (!showToTxtField.getText().trim().isEmpty()) {
                    prepStatement.setString(3, showToTxtField.getText());
                    prepStatement.setString(4, showToTxtField.getText());
                } else {
                    prepStatement.setNull(3, java.sql.Types.VARCHAR );
                    prepStatement.setNull(4, java.sql.Types.VARCHAR );
                }
                if (!showCostTxtField.getText().trim().isEmpty()) {
                    prepStatement.setFloat(5, Float.parseFloat(showCostTxtField.getText()));
                    prepStatement.setFloat(6, Float.parseFloat(showCostTxtField.getText()));
                } else {
                    prepStatement.setNull(5, java.sql.Types.FLOAT );
                    prepStatement.setNull(6, java.sql.Types.FLOAT);
                }
                LocalDate depDate = showDepDateDatePicker.getValue();
                if (depDate != null) {
                    prepStatement.setDate(7, Date.valueOf(depDate));
                    prepStatement.setDate(8, Date.valueOf(depDate));
                } else {
                    prepStatement.setNull(7, java.sql.Types.DATE );
                    prepStatement.setNull(8, java.sql.Types.DATE);
                }
                if (!showDepTimeTxtField.getText().trim().isEmpty()) {
                    DateFormat formatter = new SimpleDateFormat("HH:mm:ss");
                    try {
                        Time timeValue = new Time(formatter.parse(showDepTimeTxtField.getText()).getTime());
                        prepStatement.setTime(9, timeValue);
                        prepStatement.setTime(10, timeValue);
                    } catch (ParseException e) {
                        e.printStackTrace();
                    }
                } else {
                    prepStatement.setNull(9, java.sql.Types.TIME );
                    prepStatement.setNull(10, java.sql.Types.TIME );
                }
                LocalDate arrDate = showArrDatePicker.getValue();
                if (arrDate != null) {
                    prepStatement.setDate(11, Date.valueOf(arrDate));
                    prepStatement.setDate(12, Date.valueOf(arrDate));
                } else {
                    prepStatement.setNull(11, java.sql.Types.DATE );
                    prepStatement.setNull(12, java.sql.Types.DATE);
                }
                if (!showArrTimeTxtField.getText().trim().isEmpty()) {
                    DateFormat formatter = new SimpleDateFormat("HH:mm:ss");
                    try {
                        Time timeValue = new Time(formatter.parse(showArrTimeTxtField.getText()).getTime());
                        prepStatement.setTime(13, timeValue);
                        prepStatement.setTime(14, timeValue);
                    } catch (ParseException e) {
                        e.printStackTrace();
                    }
                } else {
                    prepStatement.setNull(13, java.sql.Types.TIME );
                    prepStatement.setNull(14, java.sql.Types.TIME );
                }

                rs = prepStatement.executeQuery();
      } catch (SQLException e) {
                e.printStackTrace();
            }

因此,重点是仅返回符合文本字段条件的航班。如果任何文本字段留空,则查询应返回符合该特定条件的所有航班,但会使用其他非空条件进行过滤。

我在参数 $9 处出错。 当我将输入showDepTimeTxtFieldString(例如“18:00:00”)转换为java.sql.Time 对象时,我可以将其用作prepStatement 方法setTime() 的输入参数,我的查询应该只返回航班(已满足其他标准)departure_time 与在文本字段中输入的时间相同。如果文本字段为空且没有 String 输入,我的查询应该从我的数据库中返回带有 departure_time 的航班。

我的问题是,当我运行我的代码时,出现以下错误: org.postgresql.util.PSQLException: ERROR: could not determine data type of parameter $9 无论我用字符串填充departureTimeTxtField 还是将其留空,所以参数departure_time 应该设置为NULL 并且我的查询应该返回所有 departure_time 的航班。

所以我假设我对 setTime() 方法参数 timeValue 以及从 Stringjava.sql.Time 的转换有问题。

如何进行正确的转换以消除该错误?

编辑:SQL 查询和 Java 源代码已更新。

【问题讨论】:

  • 假设参数不为 null 并执行 SELECT * FROM flights WHERE departure_time = ? 会发生什么?无论如何我都会这样做(如果您的参数为空,请使用select * from flights)。
  • 您好,先生。当我只使用SELECT * FROM flights WHERE departure_time = ? 执行代码时,它已经工作了,但是我的查询要复杂得多。我做了修改,请看一下。
  • 我将动态生成 SQL 查询,而不是使用处理所有可选参数的单个查询。例如,使用 QueryDSL。
  • 您应该只对原始类型使用setNull,对于所有非原始类型,您应该能够只使用setXXX(null)。检查您是否使用最新的 PostgreSQL 驱动程序,如果是,请提交错误报告,因为使用 setTime(null) 应该适用于兼容的 JDBC 驱动程序。

标签: java sql postgresql jdbc prepared-statement


【解决方案1】:

我能够通过使用COALESCE() 而不是CASE WHEN ... 来解决这个问题:

String query = "SELECT * FROM flights WHERE departure_time = COALESCE(?, departure_time)";

【讨论】:

  • 谢谢先生,您救了我的命...您能否解释一下为什么我的查询不适用于 CASE WHEN 方法?
  • “您能解释一下为什么我的查询不适用于 CASE WHEN 方法吗?” - 我可以提供一个猜测,但仅此而已。如果 CASE WHEN 方法在其他地方对您有用,并且仅对 java.sql.Time 值失败,那么它可能是一个错误,您可能需要考虑提交票证,但我会留下它由你决定。
  • @PeterBabinec:我想这与如何评估表达式的数据类型有关。可能casecoalesce() 在这里工作不同。我已经看到 Oracle 的 nvl()case 的类似问题
【解决方案2】:

该错误是由于 SQL 中无效的CASE 语法(而不是因为时间格式)而产生的,请查看CASE 语法here

此外,从示例中,您似乎正在尝试获取起飞时间为空或文本字段中存在时间的所有航班,在这种情况下,我建议采用以下方法:

ResultSet rs = null;
PreparedStatement prepStatement = null;
if (!departureTimeTxtField.getText().trim().isEmpty()) {
    DateFormat formatter = new SimpleDateFormat("HH:mm:ss");
    Time timeValue = new Time(formatter.parse(departureTimeTxtField.getText()).getTime());
    prepStatement  = connection.prepareStatement("SELECT * FROM flights WHERE departure_time = ?");
    prepStatement.setTime(1, timeValue);
}else{
    prepStatement  = connection.prepareStatement("SELECT * FROM flights WHERE departure_time IS NULL");
}
rs = prepStatement.executeQuery();

【讨论】:

  • 嗨。查询语法应该没有问题。我在另一个类中使用了这种“CASE”方法,但没有 JOIN,只有 Integer 和 Float 数据类型,一切正常。仅当我开始弄乱 Time 时才发生此错误。此外,我更新了我的问题,因为我的查询更加困难,并且您发布的方法会有很多 if-else 语句。
猜你喜欢
  • 2010-11-24
  • 2013-04-10
  • 1970-01-01
  • 2018-10-30
  • 1970-01-01
  • 1970-01-01
  • 2012-03-14
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多