0

Tengo este PreparedStatement, me gustaría saber si esta correcto, también me gustaría saber si se tiene que crear un método para hacer el EXECUTE, o en el mismo método del PreparedStatement del UPDATE.

public void Modify(int id_estado_operacion, Date fechaHora_publicacion,  Date audit_fechaHora_ultima_modificacion, int id_publicacion) {
  Connection connection = null;
  PreparedStatement prepareStmt = null;

  try{
      String query = "update publicaciones set id_estado_operacion,"
            + "fechaHora_publicacion=?, "
            + "adudit_fechaHora_ultima_modificacion=GETDATE() "
            + "where id_publicacion = ? ";

      prepareStmt.setInt(0,id_estado_operacion);
      prepareStmt.setDate(1,fechaHora_publicacion);
      prepareStmt.setDate(2,audit_fechaHora_ultima_modificacion );
      prepareStmt.setInt(3,id_publicacion);
      prepareStmt.executeUpdate();
  } catch (SQLException e) {
      System.out.println(e.getMessage());
  }
}
3
  • Este código te dará al menos dos errores: 1º. No estás creando en ningún momento el objeto prepareStmt. Según el código siempre será nulo. Para crearlo, debes hacer algo así: prepareStmt = connection.prepareStatement(query);. Luego, cuando seteas cada valor, como dice Luiggi, debes empezar por el uno (1). 2º. Ahí mismo tienes otro error, en la consulta preparada tienes sólo dos ? y estás seteando cuatro valores. La sintaxis de la consulta no parece ser correcta... set id_estado_operacion, ¿? y luego esto tampoco tiene mucho sentido: adudit_fechaHora_ultima_modificacion=GETDATE()
    – A. Cedano
    Commented el 15 ago. 2017 a las 21:57
  • También tu objeto connection debe ser válido. En el código no se ve en qué momento se establece la conexión. Quizá deberías revisar la documentación de Java sobre el asunto.
    – A. Cedano
    Commented el 15 ago. 2017 a las 22:02
  • a ok si de eso ya lo hize no lo puse en el codigo pero si lo ralize muchas gracias
    – NymHaRI
    Commented el 15 ago. 2017 a las 22:22

1 respuesta 1

1

El problema es que JDBC trabaja con los índices en base a 1, no a 0. Los parámetros deberías colocarlos desde el índice 1.

Tu código debería cambiar a esto:

prepareStmt.setInt(1,id_estado_operacion);
prepareStmt.setDate(2,fechaHora_publicacion);
prepareStmt.setDate(3,audit_fechaHora_ultima_modificacion );
prepareStmt.setInt(4,id_publicacion);
prepareStmt.executeUpdate();
2
  • gracias , mi pregunta ahora es como se hace el EXute de este SP
    – NymHaRI
    Commented el 15 ago. 2017 a las 20:42
  • @NymHaRI usando prepareStmt#executeUpdate debería ejecutarse tu sentencia de lado de la base de datos. Eso debería ser suficiente.
    – user227
    Commented el 15 ago. 2017 a las 21:11

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.