[SOLUCIONADO] Problemas al salvar un Array en una BD JDBC

[spybroke]

Hola a todos,

estoy haciendo un ejercicio tonto que seria una especie de programa para controlar altar y bajas de una academia (alumnos y cursos)

Existen X cursos ya creados y se pueden seleccionar en un combobox y segun lo que selecciones, muestra en un jList los alumnos que hay en ese curso, permite editar sus datos, etc...

Tengo el codigo ya acabado y el programa funciona perfecto salvo un detalle que me esta volviendo loco.

El programa tiene que poder cargar y salvar los datos tanto de un fichero (xml) como de una BD con JDBC... pero no se pueden tener las dos formas a la vez (lo que solucione con un if/else y un simple booleano)

La parte de cargar y guardar datos a un fichero xml funciona perfecto, pero me da problemas la parte de la BD, concretamente el salvar los datos (el cargarlos tambien me funciona bien)

Ahora os pondre los codigos, pero basicamente, el programa manda un arraylist de alumnos al metodo, el cual conecta con la BD y, por medio de un bucle for, va creando filas en la BD con los datos de cada alumno.

El codigo del metodo es este:

Código Java:

public void setAlumnos(List<Alumno> lista) {
        try (Connection conn = this.getConexion()) {
            String sqlAlumnos = "INSERT INTO ALUMNOS (DNI, NOMBRE, APELLIDO, FECHA) VALUES (?, ?, ?, ?)";
            PreparedStatement com = conn.prepareStatement(sqlAlumnos);
           
              /*
             Antes del codigo de guardado, deberiamos de comprobar si existe la BD y en caso afirmativo, vaciarla primero.
             if () {
 
             } 
             */
            for (Alumno alumno : lista) {
                com.setString(1, alumno.getDniAlumno());
                com.setString(2, alumno.getNombreAlumno());
                com.setString(3, alumno.getApellidoAlumno());
                com.setDate(4, (Date) alumno.getFechaMatriculacion());
                com.executeUpdate();
                com.close();
            JOptionPane.showMessageDialog(null, "Datos cargados correctamente.");
            }
 
        } catch (Exception ex) {
            JOptionPane.showMessageDialog(null, "No se han podido guardar los datos.");
        }
    }
 

EL fallo es que siempre entra en catch. He podido comprobar que el codigo llega correctamente hasta el com.executeUpdate(); ya que si comento esa parte y justo antes hago un System.out del alumno, me lo muestra correctamente.

Los datos de la BD, conexion y demas funciona bien ya que el metodo que carga los datos a un ArrayList si que funciona bien.

Alguien sabe que puedo estar haciendo mal?

Ademas, como veis tengo un if comentado, ya que hay quiero meter un codigo que detecte si hay alguna fila en la tabla y de ser asi, limpie la tabla entera pero aun estoy con ello xD

[Xerelo]

Lo primero que tienes mal es no tener en cuenta el error, ¿de qué te sirve capturarlo si no sabes qué problema es?

Una cosa es que al usuario no le muestres la traza porque a él no le dice nada, y otra es que tú como programador ignores la excepción que te dice cuál es el error.

El error es bastante evidente, pero mejor que lo veas por ti mismo en la excepción, aprender a tratar correctamente errores y logs también es programar.

[spybroke]

Iniciado por Xerelo

Lo primero que tienes mal es no tener en cuenta el error, ¿de qué te sirve capturarlo si no sabes qué problema es?

Una cosa es que al usuario no le muestres la traza porque a él no le dice nada, y otra es que tú como programador ignores la excepción que te dice cuál es el error.

El error es bastante evidente, pero mejor que lo veas por ti mismo en la excepción, aprender a tratar correctamente errores y logs también es programar.

Tambien a traves de un ex.getMessage() capture el error en su momento, perdon por no ponerlo...

EL error es :

Se ha abortado la sentencia porque habría causado un valor de clave duplicado en una restricción de clave única o primaria o en un índice único identificado por 'SQL150218101140550' definido en 'ALUMNOS'.

Pero debe ser que no entiendo bien la logica de lo que creo que estoy haciendo, porque no encuentro ese valor duplicado xD

EN teoria, carga la lista, y por cada elemento alumno coge sus parametros y los inserta en una fila...
El unico parametro que daría error de ese tipo es el del DNI ya que es la clave primaria, pero por como se construye la lista, es IMPOSIBLE dos alumnos con el mismo DNI por lo que no entiendo porque repite valores, algo se me esta escapando...

[Profesor_Falken]

Se ha abortado la sentencia porque habría causado un valor de clave duplicado en una restricción de clave única o primaria o en un índice único identificado por 'SQL150218101140550' definido en 'ALUMNOS'.

Nunca habia visto ese error sql tan detallado. Si pones un ex.printStackTrace() te pintara por consola mas infomacion.

En cualquier caso lo que te esta diciendo es, basicamente, que estas insertanto un alumno con un DNI que ya existe.
El alumno con el DNI existe ya previamente en tu base de datos, ya que el programa que tienes es imposible que pueda hacer mas de una iteracion del bucle, ya que estas cerrando el statement y cuando intente introducir el segundo alumno te va a fallar con otra excepciùon diferente.


Un saludo

[Xerelo]

Pienso lo mismo que Profesor_Falken, yo había visto el cierre del statement, pero para el problema de la clave duplicada, puede ser

1. Que ya hayas creado ese registro en alguna prueba anterior
2. Que no estés creando correctamente el List<Alumno>, por ejemplo no creando un objeto Alumno nuevo cada vez que añades un alumno a la lista.
3. Improbable, pero que las restricciones en la BBDD no sean las que crees

http://docs.oracle.com/javase/tutori.../prepared.html

[spybroke]

Vale, soy imbécil... XD
Veis esa parte comentada de vaciar la tabla?

Y use efectivamente lo que decís, como estoy haciendo pruebas no se me ha ocurrido otra cosa que hacer la prueba primero cargando el ArrayList a partir de la BD y luego intentar guardarlo...
Creo que no necesita más explicación la tontería que estaba haciendo jajajaja

Solo tendría que limpiar la BD primero y luego ya salvar los datos, que tengo que ver también como hacerlo jajaja

Gracias por abrirme los ojos, hay veces que te ciegas buscando un error de código y miras y miras, y al final es un error tontisimo como este