Error

[ignatar20]

Muy buenas.
Tengo un problema y es el siguiente:
java.lang.NumberFormatException for input string: 123.

Se que es por incompatibilidad del dato enviado a la base de datos, pero el caso es que hago bien (o creo hacerlo bien xD) la conversión de string a int y me sigue saliendo.
También sé que hay muchos post relacionados con el mío, pero me he leído unos cuántos y no he podido solucionarlo.
Os dejo el código para que lo entendáis mejor.

Código Java:

private void btGuardarActionPerformed(java.awt.event.ActionEvent evt) {                                          
             
       try
       {
            String nombre,direccion,telefonoS,ciudad,sueldoS,idS,puesto;
            int id,telefono;
            double sueldo;
            nombre = this.tfNombre.getText();
            direccion = this.tfDireccion.getText();
            telefonoS = this.tfTelefono.getText();
            ciudad = (String)this.cbCiudad.getSelectedItem();
            sueldoS = this.tfSueldo.getText();
            idS = this.tfID.getText();
            puesto = this.tfPuesto.getText();
            mostrarMensaje(idS);
           if(nombre.equals("")|| puesto.equals("")|| direccion.equals("")||telefonoS.equals("")|| ciudad.equals("")|| sueldoS.equals("")|| idS.equals(""))
           {
               mostrarMensaje("Error, todos los campos deben estar escritos.");
           }
           else
           {
               
               
               try
               {
                id = Integer.parseInt(idS);
                telefono = Integer.parseInt(telefonoS);
                sueldo = Double.parseDouble(sueldoS);
                Class.forName("com.mysql.jdbc.Driver");
               Connection con = DriverManager.getConnection("jdbc:mysql://localhost:3306/bdempleados","","");
               Statement stmt=(Statement)con.createStatement(); 
               String sql = "insert into trabajadores(ID,Nombre,Direccion,Ciudad,Telefono,Sueldo,Puesto)values"+"("+id+",'"+nombre+"','"+direccion+"','"+ciudad+"',"+telefono+","+sueldo+",'"+puesto+"')";
               stmt.executeUpdate(sql);
               mostrarMensaje("Empleado guardado correctamente");  
               }catch(NumberFormatException e2)
               {
                   mostrarMensaje("Error de conexion\n"+e2);
               }
        
           }
           
       }catch(Exception e)
       {
           mostrarMensaje("Debe introducir todos los campos\n"+e);
       }
 

Gracias de antemano.

Saludos

[Xerelo]

Busca la línea 123.

Un numberFormatException es que estás intentando convertir a número algo que no es número, y eso no lo controlas en ningún momento. Si en telefono, sueldo o id hay un espacio en blanco o una letra dará error, y si en sueldo no pones el separador decimal correcto también. Dependiendo del Locale será "." o ",", en español es ","

[ignatar20]

En las líneas 10, 12 y 13 lo cojo como string para poder validarlo.
Después en las líneas 26, 27 y 28 lo convierto en int y double.

[Xerelo]

Lo vuelvo a repetir, tú no validas que algo sea un número, sólo validas que los campos no sean "". Si en el campo telefono escribieses "555-4567" ibas a tener un numberformatexception.

Así que según tu mensaje de error no acepta el valor "123." Como ya te he dicho la opción más lógica es que esté fallando en el double ya que para Locale español los decimales se marcan con ",", pero sólo me puedo basar en que veo un 123. en el error que has puesto. Tú sabrás de donde sale.

De hecho, teniendo en cuenta que la excepción la recoges con un método que aparentemente no pinta la excepción entera, ni siquiera se puede decir que esa sea la parte del código donde está el problema. Si dejas que pinte la traza del error entera podrás ver exactamente en que línea está el error.

Exception in thread "main" java.lang.NumberFormatException: For input string: "xxx"
at java.lang.NumberFormatException.forInputString(Num berFormatException.java:65)
at java.lang.Integer.parseInt(Integer.java:504)
at java.lang.Integer.parseInt(Integer.java:527)
at VRP.main(VRP.java:43)

[Profesor_Falken]

Buenas,

Aparte de lo que bien dice Xerelo, quería hacer un comentario sobre ésto:

telefono = Integer.parseInt(telefonoS);
sueldo = Double.parseDouble(sueldoS);

El número de teléfono no debería ser un Integer, ya que no se va a tratar nunca como un número. No vas a hacer operaciones con él como sumas o restas. Lo que si que probablemente vas a hacer es aplicarle patrones de búsqueda para buscar los que contienen cierto número o comienzan por un prefijo determinado.
Por eso debería ser un String.

Double no es útil para trabajar con monedas ya que tiene problemas de redondeo, al igual que Float. Para cantidades de dinero se debe utilizar BigDecimal o, mejor aún, un Integer/Long de céntimos.


Un saludo