Busqueda de datos en un JTextField

[jalm2_3]

Saludos a toda la comunidad,

Estoy realizando una pequeña aplicación en Java en donde yo ingreso un dato para que este sea buscado en mi base de datos y me despliegue cierta información en diferentes JTextField pero al momento de limpiar mis campos y hacer una nueva búsqueda me arroja la excepción de mi bloque Catch diciéndome error en la base de datos y no me despliega mis datos.

Este es el código que tengo dentro de mi botón Buscar:

Código:

private void btnBuscar1ActionPerformed(java.awt.event.ActionEvent evt) {                                           
         
         try
            {
                Connection conn = DriverManager.getConnection("jdbc:mysql://localhost:3306/catalogo","root","root"); //Direccion URL de la BD 
                Statement st = conn.createStatement();
                ResultSet rs1 = st.executeQuery("SELECT Codigo, Nombre, Descripcion, Compatibilidad, Utilidad FROM cat_productos WHERE cat_productos.Codigo = '"+txtCodigo1.getText()+"' OR cat_productos.Nombre = '"+txtNombre1.getText()+"'");
       

                    if (rs1.next()) {
                        cod = rs1.getString(1);
                        nom = rs1.getString(2);
                        des = rs1.getString(3);
                        com = rs1.getString(4);
                        util = rs1.getString(5);
                        txtCodigo1.setText(cod);
                        txtNombre1.setText(nom);
                        jTextDescripcion.setText(des);
                        modelolist.addElement(com);
                        jTextUtilidad.setText(util);
                 
             
                    }
                    
            }
            catch(Exception e)
            {
                JOptionPane.showMessageDialog(null, "Error en la Base de Datos", "Aviso", JOptionPane.WARNING_MESSAGE);  
            }
        btnBuscar1.setCursor(new java.awt.Cursor(java.awt.Cursor.DEFAULT_CURSOR));
        lblFunciones1.setText("En Ejecución....");
    }

Espero y me puedan auxiliar, se los agradeceria mucho.

[nickou]

Hola jalm2_3

Asi a simple vista veo que:

Código Java:

cod = rs1.getString(1); /// estas tratando el cod como un String,  yo siempre uso los cod como enteros, a menos que lo hayas hecho asi a proposito
 

en tal caso que sea un int iria.

Código Java:

cod = rs.getInt(1));
 

Yo el while lo haria asi:

Código Java:

if (rs1.next()) {
   cod = rs.getInt(1));
   nom = rs1.getString(2);
   des = rs1.getString(3);
   com = rs1.getString(4);
   util = rs1.getString(5);
 
   txtCodigo1.setText(Integer.parseInt(cod)); // le hago el casteo
   txtNombre1.setText(nom);
   jTextDescripcion.setText(des);
   jTextUtilidad.setText(util);
   modelolist.addElement(com); 
 
}
 

espero te sirva de algo.

Saludos

[Xerelo]

Una cosa es que al usuario no quieras que le llegue el texto de una excepción, y otra es que no le llegue al desarrollador.

Siempre debes pintar/tratar una excepción para poder saber qué ha ocurrido, tu error puede ser de conexión de base de datos o que com es null o cualquier cosa, pero no lo sabrás porque lo conviertes todo en un mensaje.

Por lo demás el código no está bien, creas una conexión nueva cada vez que pulsas un botón, y ni la cierras ni cierras el Statement.

[jalm2_3]

Te agradezco tu respuesta Xerelo, pero entonces que puedo hacer para resolver este problema porque cada vez que abro el programa y consulto un dato si lo encuentra pero al momento de buscar un nuevo registro no me despliega nada.

[nickou]

Es cierto lo que dices Xerelo :


"Siempre debes pintar/tratar una excepción para poder saber qué ha ocurrido, tu error puede ser de conexión de base de datos o que com es null o cualquier cosa, pero no lo sabrás porque lo conviertes todo en un mensaje."


yo capturaria la excepción asi:

Código Java:

catch (SQLException e) {
 
        JOptionPane.showMessageDialog(null, e.getMessage());
                    
        }
 

y apartir de la exepcion, podes encontrar ideas para resolver los errores.

Saludos

[Xerelo]

Nickou, lo que propones funciona, pero no es la forma correcta de hacer las cosas. Al usuario hay que darle mensajes de error claros, y una excepción no lo es para quien no sabe.

Cuando desarrolléis debéis acostumbrados a guardar los errores en fichero (log) que es lo único que tendréis cuando un usuario lo use, o como mínimo a sacarlos por consola. Los pop up o mensajes mostrados en la aplicación son para el usuario, por lo que sólo tienen que mostrar la información necesario para que lo corrija o sepa que hay un problema.

Usuario/password no válido es para el usuario, ArrayIndexOutOfBounds no.

[jomaruro]

Totalmente de acuerdo, además a los usuarios no les gusta que aparezcan "ventanitas extra".