Eliminar fila de una base de datos

[sterlingdiazd]

Saludos.

Estoy haciendo una aplicacion que elimine un usuario a una base de datos, de un arrayList y al mismo tiempo del JFrame. Los datos se insertan bien pero cuando le voy a eliminar, me dice que el nombre de la columna no es valido, y me trae el nombre del usuario, que invoco en el metodo. El error que me da es el siguiente:

com.microsoft.sqlserver.jdbc.SQLServerException: El nombre de columna 'Z' no es válido.

Este es el codigo:

Código Java:

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.ArrayList;
import javax.swing.JFrame;
 
public class Conexion {
 
    Connection con;
    Statement stmt;
    ResultSet rs;
    
    public Conexion(){
        
        try {
Class.forName("com.microsoft.sqlserver.jdbc.SQLServerDriver").newInstance();
            con = DriverManager.getConnection( "jdbc:sqlserver://localhost;databaseName=Eventos;user=Sterling;password=123456" );
            stmt = con.createStatement();
        } catch (InstantiationException e) {
            e.printStackTrace();
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        } catch (ClassNotFoundException e) {
            e.printStackTrace();
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }
    
    public void listarUsuario(){
        try {
            rs = stmt.executeQuery( "select * from Usuarios order by Usuario ");
            while ( rs.next() ){
                System.out.println( 
                        rs.getString("Nombre")      + "\t" + 
                        rs.getString("Apellido")    + "\t" + 
                        rs.getString("Usuario")     + "\t" + 
                        rs.getString("Contraseña")  + "\t" + 
                        rs.getString("Perfil") 
                );
            }
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
    }
    
    public ArrayList<Usuario> getUsuario(){
        ArrayList<Usuario> usuarios = null;
        try {
            rs = stmt.executeQuery("select * from Usuarios order by Usuario ");
            usuarios = new ArrayList<Usuario>();
            while ( rs.next() ){
                usuarios.add( new Usuario( rs.getString("Nombre"), rs.getString("Apellido"), rs.getString("Usuario"), rs.getString("Contraseña"), rs.getString("Perfil")  ) );
            }
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
        return usuarios;
    }
    
    public void insertarUsuario(Usuario usuario){
        try {
            stmt.execute("insert into Usuarios(Nombre, Apellido, Usuario, Contraseña, Perfil) values (     '"+ usuario.getNombre() +"', '"+ usuario.getApellido() +"', '" + usuario.getUsuario() +"', '"+ usuario.getContraseña() +"', '"+ usuario.getPerfil() +"') ");
        } catch (SQLException e) {
            
            System.out.println("No se puede insertar valor repetido");
            e.printStackTrace();
        }
    }
    
    public void eliminarUsuario( Usuario usuarios ){
        try {
            [SIZE="5"]stmt.execute(" delete from Usuarios where Usuario = " + usuarios.getUsuario() );[/SIZE]
            
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
    }
    
    public static void main(String[] args) {
        Conexion conecta = new Conexion();
        conecta.listarUsuario();
    }
}
 

Y aqui está el query que elimina el usuario:

Código Java:

import java.util.ArrayList;
import javax.swing.table.AbstractTableModel;
 
public class Modelo extends AbstractTableModel {
 
    ArrayList<Usuario> usuarios;
    Conexion con;
    
    Modelo(){
        con = new Conexion();
        usuarios = con.getUsuario();
    }
    
    [SIZE="5"]public void eliminar(int fila){
        con.eliminarUsuario(usuarios.get(fila));
        usuarios.remove(fila);
        fireTableRowsDeleted(fila, fila);
    }[/SIZE]
    
    String [] encabezado = {"Nombre", "Apellido", "Usuario", "Contraseña", "Perfil"};
    public int getColumnCount() {
        return encabezado.length;
    }
    
    public int getRowCount() {      
        return usuarios.size();
    }
    
    public String getColumnName(int x) {
        return encabezado[x];
    }
    
    public void añadir(Usuario usuario){
        con.insertarUsuario(usuario);
        usuarios = con.getUsuario();
        fireTableDataChanged();
 
    }
    public Object getValueAt(int x, int y) {
        String retorno = "";
        Usuario usuario = usuarios.get(x);
        
        switch (y){
        case 0: retorno = usuario.getNombre();
        break;
        case 1: retorno = usuario.getApellido();
        break;
        case 2: retorno = usuario.getUsuario();
        break;
        case 3: retorno = usuario.getContraseña();
        break;
        case 4: retorno = usuario.getPerfil();
        }
        return retorno;
    }
}
 

Gracias por cualquier ayuda que me puedan brindar.
Si alguno quisiera probar la aplicacion, les dejo las otras 2 clases.

Código Java:

package mantenimiento;
 
public class Usuario {
 
    private String Nombre;
    private String Apellido;
    private String Usuario;
    private String Contraseña;
    private String Perfil;
 
    public String getNombre() {
        return Nombre;
    }
    public void setNombre(String nombre) {
        Nombre = nombre;
    }
    public String getApellido() {
        return Apellido;
    }
    public void setApellido(String apellido) {
        Apellido = apellido;
    }
    public String getUsuario() {
        return Usuario;
    }
    public void setUsuario(String usuario) {
        Usuario = usuario;
    }
    public String getContraseña() {
        return Contraseña;
    }
    public void setContraseña(String contraseña) {
        Contraseña = contraseña;
    }
    public String getPerfil() {
        return Perfil;
    }
    public void setPerfil(String perfil) {
        Perfil = perfil;
    }
 
    public Usuario(String nombre, String apellido, String usuario,
            String contraseña, String perfil) {
        super();
        Nombre = nombre;
        Apellido = apellido;
        Usuario = usuario;
        Contraseña = contraseña;
        Perfil = perfil;
    }
}
 

[sterlingdiazd]

Esta es la parte visual.

Código Java:

package mantenimiento;
 
import java.awt.BorderLayout;
import javax.swing.JPanel;
import javax.swing.JFrame;
import java.awt.Dimension;
import javax.swing.JScrollPane;
import java.awt.Rectangle;
import javax.swing.JTable;
import javax.swing.JTextField;
import javax.swing.JLabel;
import javax.swing.JButton;
 
public class FrameUsuario extends JFrame {
    private static final long serialVersionUID = 1L;
    private JPanel jContentPane = null;
    private JScrollPane jScrollPane = null;
    private JTable jTableUsuario = null;
    private Modelo modelo = null;
    private JTextField txtNombre = null;
    private JTextField txtApellido = null;
    private JLabel lblNombre = null;
    private JLabel lblApellido = null;
    private JButton btAñadir = null;
    private JLabel lblUsuario = null;
    private JLabel lblContraseña = null;
    private JLabel lblPerfil = null;
    private JTextField txtUsuario = null;
    private JTextField txtContraseña = null;
    private JTextField txtPerfil = null;
    private JButton btEliminar = null;
 
    public FrameUsuario() {
        super();
        initialize();       
    }
 
    private void initialize() {
        modelo = new Modelo();
        this.setSize(691, 470);
        this.setContentPane(getJContentPane());
        this.setTitle("Listado de Usuarios");
    }
 
    private JPanel getJContentPane() {
        if (jContentPane == null) {
            lblPerfil = new JLabel();
            lblPerfil.setBounds(new Rectangle(269, 82, 113, 32));
            lblPerfil.setText("Perfil");
            lblContraseña = new JLabel();
            lblContraseña.setBounds(new Rectangle(269, 29, 113, 32));
            lblContraseña.setText("Contraseña");
            lblUsuario = new JLabel();
            lblUsuario.setBounds(new Rectangle(31, 135, 113, 32));
            lblUsuario.setText("Usuario");
            lblApellido = new JLabel();
            lblApellido.setBounds(new Rectangle(31, 82, 113, 32));
            lblApellido.setText("Apellido");
            lblNombre = new JLabel();
            lblNombre.setBounds(new Rectangle(31, 29, 113, 32));
            lblNombre.setText("Nombre");
            jContentPane = new JPanel();
            jContentPane.setLayout(null);
            jContentPane.add(getJScrollPane(), null);
            jContentPane.add(getTxtNombre(), null);
            jContentPane.add(getTxtApellido(), null);
            jContentPane.add(lblNombre, null);
            jContentPane.add(lblApellido, null);
            jContentPane.add(getBtAñadir(), null);
            jContentPane.add(lblUsuario, null);
            jContentPane.add(lblContraseña, null);
            jContentPane.add(lblPerfil, null);
            jContentPane.add(getTxtUsuario(), null);
            jContentPane.add(getTxtContraseña(), null);
            jContentPane.add(getTxtPerfil(), null);
            jContentPane.add(getBtEliminar(), null);
        }
        return jContentPane;
    }
 
    private JScrollPane getJScrollPane() {
        if (jScrollPane == null) {
            jScrollPane = new JScrollPane();
            jScrollPane.setBounds(new Rectangle(29, 181, 614, 238));
            jScrollPane.setViewportView(getJTableUsuario());
        }
        return jScrollPane;
    }
 
    private JTable getJTableUsuario() {
        if (jTableUsuario == null) {
            jTableUsuario = new JTable();
            jTableUsuario.setModel(modelo);
        }
        return jTableUsuario;
    }
 
    private JTextField getTxtNombre() {
        if (txtNombre == null) {
            txtNombre = new JTextField();
            txtNombre.setBounds(new Rectangle(149, 29, 113, 32));
        }
        return txtNombre;
    }
 
    private JTextField getTxtApellido() {
        if (txtApellido == null) {
            txtApellido = new JTextField();
            txtApellido.setBounds(new Rectangle(149, 82, 113, 32));
        }
        return txtApellido;
    }
 
    private JButton getBtAñadir() {
        if (btAñadir == null) {
            btAñadir = new JButton();
            btAñadir.setBounds(new Rectangle(513, 29, 113, 36));
            btAñadir.setText("Añadir");
            btAñadir.addActionListener(new java.awt.event.ActionListener() {
                public void actionPerformed(java.awt.event.ActionEvent e) {
                    //Insercion de los datos
                    modelo.añadir(new Usuario(txtNombre.getText(), txtApellido.getText(), txtUsuario.getText(), txtContraseña.getText(), txtPerfil.getText()));
                    //Limpiado de los fields
                    txtNombre.setText("");
                    txtApellido.setText("");
                    txtUsuario.setText("");
                    txtContraseña.setText("");
                    txtPerfil.setText("");
                }
            }
            );
        }
        return btAñadir;
    }
 
    private JTextField getTxtUsuario() {
        if (txtUsuario == null) {
            txtUsuario = new JTextField();
            txtUsuario.setBounds(new Rectangle(151, 135, 113, 32));
        }
        return txtUsuario;
    }
 
    private JTextField getTxtContraseña() {
        if (txtContraseña == null) {
            txtContraseña = new JTextField();
            txtContraseña.setBounds(new Rectangle(390, 29, 113, 32));
        }
        return txtContraseña;
    }
 
    private JTextField getTxtPerfil() {
        if (txtPerfil == null) {
            txtPerfil = new JTextField();
            txtPerfil.setBounds(new Rectangle(390, 82, 113, 32));
        }
        return txtPerfil;
    }
 
    private JButton getBtEliminar() {
        if (btEliminar == null) {
            btEliminar = new JButton();
            btEliminar.setBounds(new Rectangle(512, 80, 115, 36));
            btEliminar.setText("Eliminar");
            btEliminar.addActionListener(new java.awt.event.ActionListener() {
                public void actionPerformed(java.awt.event.ActionEvent e) {
                    modelo.eliminar(jTableUsuario.getSelectedRow());
                }
            });
        }
        return btEliminar;
    }
 
}  //  @jve:decl-index=0:visual-constraint="10,10"
 

[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,

[leocente]

Proba cambiando esta linea que tenes

Código Javascript:

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

por esta

Código Javascript:

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

y fijate si es eso.

Lo demas no tengo tiempo para verlo, disculpa. Pero creo que lo que te marque es el problema. Sino avisa!


Saludos maquina!

[sterlingdiazd]

Gracias, mil Gracias! El problema se soluciono.