Javaを使用した一般的なjdbc更新クエリ

Question

jdbcを使用して更新クエリを作成しています。しかし、条件に基づいて私は列に異なる値を設定する必要があります。このコードをより簡単なコードとして変更できますか？あなたのアイデアを教えてください。

if(allDealsCreated) { 
    System.out.println("Updating the status of the deals as CLOSED"); 
    if(deals != null && !deals.isEmpty()) {    
     for (String dealId : deals) { 
      PreparedStatement closedPreparedStatement = null; 
      try (Connection con = DriverManager.getConnection(
       "jdbc:as400://localhost/BB", 
       "<username>", 
       "<password>")) { 

      String sql = "<Update query to set status as closed>"; 
      closedPreparedStatement = con.prepareStatement(sql); 
      closedPreparedStatement.executeUpdate(); 

     } catch(Exception e) { 
      e.printStackTrace(); 
     } 
    } 
} 
} else { 
System.out.println("Updating the status of the deals as NEW"); 
if(deals != null && !deals.isEmpty()) {    
    for (String dealId : deals) { 
     PreparedStatement newPreparedStatement = null; 
     try (Connection con = DriverManager.getConnection(
       "jdbc:as400://localhost/BB", 
       "<username>", 
       "<password>")) { 

      String sql = "<Update query to set status as new>"; 
      newPreparedStatement = con.prepareStatement(sql); 
      newPreparedStatement.executeUpdate(); 

     } catch(Exception e) { 
      e.printStackTrace(); 
     } 
    } 
} 
} 

Answer 1

あなたのコードに問題があります。あなたはforループでConnectionとPreparedStatementsを作成していますか？それはベストプラクティスではありません。以下のコードに従ってください。

if (deals != null && !deals.isEmpty()) { 
     try { 
      Connection con = DriverManager.getConnection(
        "jdbc:as400://localhost/BB", 
        "<username>", 
        "<password>"); 
      PreparedStatement preparedStatement = null; 
      String sql; 
      if (allDealsCreated) { 
       System.out.println("Updating the status of the deals as CLOSED"); 
       sql = "UPDATE DEALS SET STATUS = 'CLOSED' WHERE DEALNO= ?"; 
      } else { 
       System.out.println("Updating the status of the deals as NEW"); 
       sql = "UPDATE DEALS SET STATUS = 'NEW' WHERE DEALNO= ?"; 
      }      
      preparedStatement = con.prepareStatement(sql); 
      for (String dealId : deals) { 
       preparedStatement.setString(1, dealId); 
       preparedStatement.addBatch(); 
      } 
      preparedStatement.executeBatch(); 
     } catch (Exception e) { 
      e.printStackTrace(); 
     } 
    }