SQL in vorbereitete Anweisung SQL-Ausnahme

Question

werfen Ich versuche herauszufinden, warum dieser Code eine SQL-Ausnahme auslöst. Wenn ich diesen Code ausführe, druckt es "Bad SQL in Kundeneinfügung ps", die die Nachricht in diesem inneren catch-Block ist. Ich habe mehrere vorbereitete Anweisungen mit solchen SQL-Einfügungen sowohl in dieser Klasse als auch anderswo in meiner Anwendung. Sie arbeiten alle gut. Ich habe das immer wieder durchgesehen und ich kann mir nicht vorstellen, warum dieser hier eine Ausnahme macht.

try { 
       Connection conn = DBconnection.getConnection(); 
       PreparedStatement ps = conn.prepareStatement("SELECT customerId FROM customer WHERE customerName=\"" + name + "\";"); 
       System.out.println(ps.toString()); 
       ResultSet rs = ps.executeQuery(); 

       if (rs.next()) { 
        customerId = rs.getString("customerId"); 
       } 
       try { 

        PreparedStatement customerInsert = DBconnection.getConnection().prepareStatement("INSERT " 
          + "INTO customer (customerName, addressId, active, createDate, createdBy, lastUpdate, lastUpdateBy)" 
          + "VALUES(\"" + name + "\", " + addressId + ", " + active + ", UTC_TIMESTAMP(), \"" + LogInController.getUserName() + "\", UTC_TIMESTAMP(), \"" + LogInController.getUserName() + "\");"); 

        customerInsert.executeUpdate(); 

        System.out.println(customerInsert.toString()); 
        System.out.println(rs.toString()); 

       } catch (SQLException sq) { 
       System.out.println("Bad SQL in customer insert ps"); 
       } 

      } catch (SQLException customerIdException) { 
       System.out.println("Bad SQL in customer ps"); 
      } 

Answer 1

Sie verwenden PreparedStatement, als ob Sie Statement verwendet haben. Setzen Sie die Parameter nicht in SQL, setzen Sie Platzhalter ? Markierungen. Dann nutzen Sie die verschiedenen setXyz Methoden (setString, setInt, etc.) in den Parametern zu füllen:

PreparedStatement customerInsert = DBconnection.getConnection().prepareStatement(
    "INSERT INTO customer (customerName, addressId, active, createDate, createdBy, lastUpdate, lastUpdateBy)" + 
        "VALUES(?, ?, ?, ?, ?, ?, ?);" 
); 
customerInsert.setString(1, name); 
customerInsert.setInt(2, addressId); 
// ...etc. Notice that the parameter indexes start with 1 rather than 0 as you might expect