MySQLにJDateChooseを挿入

Question

MySQLデータベースにデータを挿入しようとすると問題が発生します。私はそれがJDateChooserのために起こったと思います。

これは私が取得していますエラーです：

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'May 24 21:06:17 CEST 2017, Fri May 26 00:00:00 CEST 2017, Desayuno, Adrian Poved' at line 1 

そして、これは、データを挿入する方法であるが：

public void nuevaReserva(ReservaVO reserva){ 
      try { 
       Statement st = bd.getConexion().createStatement(); 
       st.executeUpdate("INSERT INTO reserva VALUES(null,"+reserva.getInicio()+", "+reserva.getFin() 
       +", "+reserva.getRegimen()+", "+reserva.getCod_cliente()+", "+reserva.getCod_usuario() 
       +", "+reserva.getCod_habitacion()+");"); 
      } catch (SQLException e) { 
       // TODO Auto-generated catch block 
       e.printStackTrace(); 
      } 
     } 

私もデータを得るために、私のcontrolerでこのメソッドを持っていますJDateChooserとComboBoxから：

private void insertaReserva() { 
     ReservaDAO modeloReserva = new ReservaDAO(); 
     String refEmpeladoS = String.valueOf(refEmpleado); 
     ReservaVO reserva = new ReservaVO("",nrv.getDateChooserLlegada().getDate().toString(), 
       nrv.getDateChooserSalida().getDate().toString(), 
       nrv.getListaPension().getSelectedItem().toString(), 
       nrv.getListaClientes().getSelectedItem().toString(),refEmpleadoS, 
       nrv.getListaHabitaciones().getSelectedItem().toString()); 

     modeloReserva.nuevaReserva(reserva); 
     } 

ありがとうございます。

Answer 1

日付を文字列にフォーマットするには、SimpleDateFormat.format（）メソッドを使用する必要があります。 nrv.getDateChooserSalida（）。getDate（）。toString（）を使用するだけでデータオブジェクトを取得することはできません。

SimpleDateFormat sdf= new SimpleDateFormat("yyyy-MM-dd"); 
String dateFormatted = sdf.format(nrv.getDateChooserSalida().getDate().toString()); 

ReservaVOコンストラクタでDateFormattedを使用します。

Answer 2

ほとんどの場合、あなたの日付価値の周りに引用符がありません。

しかし、それだけであなたを得ることができます。

改善するためのいくつかの点：

a）の使用のPreparedStatement /バインド変数

主にこれはあなたのデータベースに厄介なSQLインジェクションを防ぐ -

public void nuevaReserva(ReservaVO reserva){ 
     try { 
      PreparedStatement st = bd.getConexion().prepareStatement(
    "INSERT INTO reserva VALUES(null, ?, ?, ?, ?, ?, ?)"); 
      st.setString(1, reserva.getInicio()); 
      st.setString(2, reserva.getFin()); 
      st.setString(3, reserva.getRegimen()); 
      st.setString(4, reserva.getCod_cliente()); 
      st.setString(5, reserva.getCod_usuario()); 
      st.setString(6, reserva.getCod_habitacion()); 
      st.executeUpdate(); 
     } catch (SQLException e) { 
      // TODO Auto-generated catch block 
      e.printStackTrace(); 
     } 
    } 

を、読みSQLが少し良くなり、同様

覚えておいてください：あなたのカラム定義についてはわかりません。いくつかのsetInt（）があるかもしれませんし、そうでなければあなたのニーズにはるかに良いかもしれません。

B）のtry-とリソース

閉じられることはありませんあなたのDatabaseconnection bd.getConnexion（）

を使用することによって改善します。それは望んでいるかもしれませんが（1つの接続は長い時間開いています）、あなたのgetConexion（）がデータベースへの接続を開くか（その場合は閉じる必要があります）、接続プール - この場合、あなたはそれを返す必要があります。両方とも、リソースを試してみると簡単に実行できます：

public void nuevaReserva(ReservaVO reserva){ 
     try (Connection con = bd.getConexion(); 
      PreparedStatement st = bd.getConexion().prepareStatement(
    "INSERT INTO reserva VALUES(null, ?, ?, ?, ?, ?, ?)")) { 
      st.setString(1, reserva.getInicio()); 
      st.setString(2, reserva.getFin()); 
      st.setString(3, reserva.getRegimen()); 
      st.setString(4, reserva.getCod_cliente()); 
      st.setString(5, reserva.getCod_usuario()); 
      st.setString(6, reserva.getCod_habitacion()); 
      st.executeUpdate(); 
     //Both statement and connection will be closed and closing curly brace 
     // even in case of an Exception! 
     } catch (SQLException e) { 
      // TODO Auto-generated catch block 
      e.printStackTrace(); 
     } 
    }