0

No consigo que mi aplicación en Java borre datos de una base de datos.

Adjunto código de mi método para borrar:

int rowIndex = this.tabla.getSelectedRow();

    try{
       String dni = String.valueOf( tabla.getValueAt(rowIndex, 0));
       st = con.createStatement(); 
       int fila = tabla.getSelectedRow();
       int col = tabla.getSelectedColumn();
       System.out.println(fila);
       System.out.println(col);
       String sql = "DELETE FROM clientes where dni="+dni;          

       st.executeUpdate(sql);
       modelo.removeRow(fila);

           JOptionPane.showMessageDialog(null, "Datos Borrados");

    }catch(SQLException e){
        JOptionPane.showMessageDialog(null, "Error al borrar");
    }

ese código es llamado desde un botón en su actionListener. He probado con prepareStatement, y tampoco. No me interesa esta forma, ya que no sé cómo pasarle por parámetro al método el dni sacado de un JTable.

1
  • Puedes mostrar el error que te da? el 2 jun. 2017 a las 9:36

2 respuestas 2

2

Creo que el problema es el string del DNI, deberías de ponerlo entre comillas simples

 String sql = "DELETE FROM clientes where dni='"+dni+"'";       
4
  • siiiiiiiiii era eso... gracias joder....
    – scorpions
    el 2 jun. 2017 a las 9:24
  • @scorpions, menos XXXXX y mas cerrar y valorar la pregunta...
    – Charly
    el 2 jun. 2017 a las 9:31
  • y como se hace eso?? no lo veo
    – scorpions
    el 2 jun. 2017 a las 9:35
  • Añadiendo la respuesta como respuesta del hilo, y cerrándolo. Para que en futuras búsquedas se vea que está cerrado, y así ayudar a la comunidad el 2 jun. 2017 a las 9:45
2

Aunque ya haya una respuesta admitida como correcta, considero que incluye una mala práctica, así que añado otra.

Deberías utilizar PreparedStament y no concatenar los parámetros de entrada para evitar posibles ataques por inyección de sql.

Así, el código quedaría:

try (PreparedStatement st=con.prepareStatement("DELETE FROM clientes where dni=?")){
    st.setString(1,dni);
    st.executeUpdate();
}catch(SQLException e){
    JOptionPane.showMessageDialog(null, "Error al borrar");
}

Tu Respuesta

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

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