Eliminar fila de una base de datos

[HackmanC]

Hola,

En este caso, cuando estas ingresando los datos, estás poniendo comillas simples para denotar en la instrucción SQL que los valores son constantes alfanúmericas:

Iniciado por sterlingdiazd

Código Java:

stmt.execute("insert into Usuarios(Nombre, Apellido, Usuario, Contraseña, Perfil) values
(     '"+ usuario.getNombre() +"', '"+ usuario.getApellido() +"', '" + usuario.getUsuario() +
"', '"+ usuario.getContraseña() +"', '"+ usuario.getPerfil() +"') ");
 

Pero acá no lo estás haciendo, aún cuando deben ser constantes alfanuméricas.

Iniciado por sterlingdiazd

Código Java:

stmt.execute(" delete from Usuarios where Usuario = " + usuarios.getUsuario() );
 

En este caso yo te aconsejaría que en lugar de 'intentar' mantener sincronizada la base de datos con tu lista, realices una nueva consulta que traiga 'todos' los datos cada vez que insertas, eliminas o actualizas un registro, no solo es más facil darle mantinimiento a una aplicación sino que eliminas varios problemas.

Suponiendo que la cantidad de usuarios es un valor moderadamente pequeño, sino fuera así, es decir, si fueran cientos o miles de usuarios tampoco es aconsejable que los metas en un List todos al mismo tiempo, entonces tendrías que desarrollar una interface gráfica (una pantalla) donde la lógica sea mas eficiente.

Iniciado por sterlingdiazd

Código Java:

public void eliminar(int fila){
        con.eliminarUsuario(usuarios.get(fila));
        // usuarios.remove(fila);
        Volver a cargar todos los usuarios.
        fireTableDataChanged();
        // fireTableRowsDeleted(fila, fila);
    }
 

Aunque lograste un concepto bastante definido, donde desacoplaste la capa de acceso a datos claramente, tus entidades, etc., creo, en mi opinión personal, que la parte gráfica se podría mejorar bastante, principalmente usando un Layout.

Saludos,