1

Pongo en contexto, en mi sistema de agendamiento de citas quiero registrar un array en una tabla, esta tabla aloja los id de una cita y los id de los diversos servicios que el usuario puede elegir para esta cita, hay otra columna que puede ser null sin ningún problema aquella tabla tiene como nombre cita_Servicio, las columnas que contiene esta tabla son idCita y idServicio quiere decir que una cita puedo tener muchos servicios por este motivo necesito registrar varios registros a la vez a continuación agrego código

public class CitaServicioDAO extends Conexion implements Crud {
//1.Declarar objetos y variables

private Connection conexion = null;
private PreparedStatement puente = null;
private ResultSet mensajero = null;

//1.1.Declarar variable para cualquier metodo
public boolean operaciones = false;
public String sql;

//1.2.Declarar las variables del VO
public String idCita="", idServicio[];
public String cisCosto = "";

public CitaServicioDAO() {
}

//2.
public CitaServicioDAO(CitaServicioVO cisVO) {
    super();

    try {
        conexion = this.obtenerConexion();
        idCita = cisVO.getIdCita();
        idServicio = cisVO.getIdServicio();
        cisCosto = cisVO.getCisCosto();
    } catch (Exception e) {
        Logger.getLogger(CitaServicioDAO.class.getName()).log(Level.SEVERE, null, e);
    }

}

@Override
public boolean agregarRegistro() {
    try {
        sql = "INSERT INTO cita_servicio VALUES(?,?)";
            puente = conexion.prepareStatement(sql);            
        for (int i = 0; i < idServicio.length; i++) {
            puente.setString(1, idCita);
            puente.setString(2, idServicio[i]);
            puente.addBatch();
            puente.executeBatch();
        }
    } catch (SQLException e) {
        Logger.getLogger(CitaServicioDAO.class.getName()).log(Level.SEVERE, null, e);
    } finally {
        try {
            this.cerrarConexion();
        } catch (SQLException e) {
            Logger.getLogger(CitaServicioDAO.class.getName()).log(Level.SEVERE, null, e);
        }
    }
    return operaciones;
}

Realmente no se cual es el problema aqui, no se por que no me registra en la base de datos hago debug y continuación los pantalla introducir la descripción de la imagen aquí

Al momento de yo hacer debug todo correcto pero me salta el try catch a la expeption y cada variable toma su valor correspondiente quisiera saber cual es el error no logro encontrarlo,Agradeceria respuesta introducir la descripción de la imagen aquí

Linea 63

introducir la descripción de la imagen aquí Linea 64

introducir la descripción de la imagen aquí Linea 68 lo que dice el exception e

10
  • 1
    Cual es el error que te tira?
    – alanfcm
    Commented el 17 sept. 2020 a las 18:45
  • Por eso digo salta a la exception como esta amrcado en la imagen en verde lo que tiene que pasar es que debe saltar al metodo cerrarConexion y no me registra en la base de datos Commented el 17 sept. 2020 a las 18:48
  • 2
    Si salta a la exception es porque hay un error, mi pregunta es cual es ese error?
    – alanfcm
    Commented el 17 sept. 2020 a las 18:53
  • Amigo por eso estoy preguntando jajaja es que no se que pasa, no se si falta algo o que Commented el 17 sept. 2020 a las 18:54
  • Amigo mira las imágenes que acabo de colocar de lo que pasa en cada linea de código realizando debug tal vez ahi encuentres el error Commented el 17 sept. 2020 a las 23:21

2 respuestas 2

1

La instrucción executeBatch() se ejecuta UNA vez, cuando terminas de añadir todos los conjuntos de valores.

...
for (int i = 0; i < idServicio.length; i++) {
    puente.setString(1, idCita);
    puente.setString(2, idServicio[i]);
    puente.addBatch();
}
puente.executeBatch();
...

Lo otro: yo revisaría (no sé si lo haces en otro lugar) que el elemento en idServicio no esté vacío para ningún caso.

6
  • Mira hize eso y pues ahora si me recorre correctamente el bucle for pero sigue con el mismo problema, exception e me manda el mismo error Commented el 18 sept. 2020 a las 0:27
  • @DavidSepulveda Ya cerraste la pregunta duplicada?
    – Alfabravo
    Commented el 18 sept. 2020 a las 0:31
  • 1
    ya la borre amigo Commented el 18 sept. 2020 a las 0:33
  • amigo ahora que hago, no entiendo por que pasa eso Commented el 18 sept. 2020 a las 1:41
  • Añade la definición completa de la tabla a la pregunta, porfa. Estoy casi seguro que por ahí va la cosa (falla en el primer registro teniendo datos).
    – Alfabravo
    Commented el 18 sept. 2020 a las 1:50
0

Esta es la solucion del problema

 public boolean agregarRegistro() {
    try {
        sql = "INSERT INTO cita_servicio (idCita, idServicio) VALUES(?,?)";
            puente = conexion.prepareStatement(sql);            
        for (int i = 0; i < idServicio.length; i++) {
            puente.setString(1, idCita);
            puente.setString(2, idServicio[i]);
            puente.addBatch();               
        }
            puente.executeBatch();
            operaciones = true;
    } catch (SQLException e) {
        Logger.getLogger(CitaServicioDAO.class.getName()).log(Level.SEVERE, null, e);
    } finally {
        try {
            this.cerrarConexion();
        } catch (SQLException e) {
            Logger.getLogger(CitaServicioDAO.class.getName()).log(Level.SEVERE, null, e);
        }
    }
    return operaciones;
}

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.