3

deseo realizar 3 queries dentro de un método en JAVA con el fin de asignar un número de serie a mi registro que lleve como consecutivo el número de ID, este es mi código.

public void guardar()
{
   String erp = jComboBox1.getSelectedItem().toString();
   String cant = jComboBox2.getSelectedItem().toString();
   String estpack = jComboBox3.getSelectedItem().toString();
   Date fecha = jDateChooser1.getDate();
   String serie;
   int aidi;

   int cantpc = Integer.parseInt(cant) * Integer.parseInt(estpack); 
       try
       {
         Connection s = Conexiones.getConexion();
        Statement st = s.createStatement();
        st.executeQuery("insert into Embarques (ERP,CantCont,EstandarPack,FechaEmbarque,CantPiezas) values ('"+erp+"','1','"+estpack+"','"+new SimpleDateFormat("yyyy-MM-dd").format(fecha)+"','"+cantpc+"')");     
        }catch(SQLException e)
        {

        }
       try{
        Connection s2 = Conexiones.getConexion(); 

        Statement st2=s2.createStatement();
        ResultSet last_id = st2.executeQuery("SELECT MAX(ID) FROM Embarques");


        while(last_id.next())
        {
            aidi = last_id.getInt("ID");
            serie = "SMX-"+new SimpleDateFormat("yyyyMMdd").format(fecha)+"-"+aidi;

        Statement st3 = s2.createStatement();
        st3.executeQuery("UPDATE Embarques SET No_Serie ='"+serie+"' WHERE id='"+aidi+"'");
          }      
        }catch(SQLException e)
        {

        }      
   }
2
  • ¿Y el problema cuál es?
    – A. Cedano
    Commented el 13 sept. 2018 a las 3:10
  • Primero, tu código está propenso a SQLInjection segundo las excepciones no sirven mucho si son omitidas, por lo que no sabremos ayudarte en caso de que tengas un error, mínimo haz un e.printStacktace(); en cada catch para saber si es por culpa de un apóstrofe mal escapado (revisa preparedStatement)
    – Ajeno
    Commented el 13 sept. 2018 a las 13:10

1 respuesta 1

0

ya lo solucioné: Dejemos que mi hardware se encargue de la seguridad. Mi problema fue un error en el código, que resultó en la consulta SQL (asignación de nombre a la columna de mi consulta). No deseo modificar este código aunque presente vulnerabilidades ya que las subsano con el hardware utilizado. Lo solucioné de la siguiente manera:

try //EJECUTAMOS LA INCERSIÓN A LA BD
       {
        Connection s = Conexiones.getConexion();// CREAMOS LA CONEXIÓN PARA LA INSERCIÓN DE DATOS
        Statement st = s.createStatement();
        st.executeQuery("insert into Embarques (ERP,CantCont,EstandarPack,FechaEmbarque,CantPiezas) values ('"+erp+"','1','"+estpack+"','"+new SimpleDateFormat("yyyy-MM-dd").format(fecha)+"','"+estpack+"')");     
       //EJECUTAMOS LA INCERCIÓN DEL NUEVO EMBARQUE SIN EL NÚMERO DE SERIE 
       }catch(SQLException e)
        {

        }
       try{//EJECUTAMOS LA CONSULTA Y ACTUALIZACIÓN EN LA BD
        Connection s2 = Conexiones.getConexion(); // CREAMOS CONEXIÓN PARA LA CONSULTA
        Statement st2 = s2.createStatement();
        // SE CONSULTA EL VALOR MÁXIMO DE ID EN LA TABLA EMBARQUE
        // CON EL PROPOSITO DE ASIGNAR EL NÚMERO DE SERIE CORRESPONDIENTE
        // POR LÓGICA SE OBTIENE EL ULTIMO VALOR INSERTADO
        // EL CUAL SE REALIZÓ EN LA SENTENCIA ANTERIOR
        // Y SE CONCATENA EL VALOR OBTENIDO EN LA PARTE FINAL DEL NÚMERO DE SERIE GENERADO

        rs = st2.executeQuery("SELECT MAX(ID) AS ID FROM Embarques");// CONSULTA DEL MÁXIMO ID SE LE ASIGNA NOMBRE AL RESULTADO PARA PODER SER ASIGNADO A UNA VARIABLE
        while(rs.next())// ENTRA EN EL CICLO MIENTRAS SE EJECUTA LA CONSULTA
        {

            Connection s3 = Conexiones.getConexion();// CREAMOS LA CONEXIÓN PARA LA ACTUALIZACIÓN DEL REGISTRO
            Statement st3 = s3.createStatement();
            serie = "SMX-"+date_full+"-"+rs.getInt("ID");   // SE CONCATENAN LOS DATOS PARA GENERAR EL NÚMERO DE SERIE
            st3.executeQuery("UPDATE Embarques SET No_Serie ='"+serie+"' WHERE id='"+rs.getInt("ID")+"'");// SE EJECUTA LA ACTUALIZACIÓN DEL 
        }//FIN WHILE

        }catch(SQLException e)//CATCH3
        {

        }//FIN CATCH3  

¡Saludos amigos!

Tu Respuesta

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

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