Question

Hola buenas noches hoy vengo haciendo una pregunta de codigo :)

Bueno el codigo de abajo es el codigo de registrar datos en una BD algo sencillo pero yo me e preguntado esta bien ?es que yo e visto otros tipo de codigos,mas pequeños a estos y me e preguntado si esta bien que use este tipo de codigo o que deberia mejor :)

ESPERO Y ALGUIEN ME CONTESTE :)

-------------------------Este codigo fue unos de los primero pero el que siempre e usado------------------
Connection reg = conexionConBaseDatos.getConexion();

       String sql = "INSERT INTO faltas(codigo,profesor,hr,clase,dia,motivo)VALUES(?,?,?,?,?,?)";
    try{
        PreparedStatement pst= reg.prepareStatement(sql);
        pst.setString(1, txtcodigo.getText());
        pst.setString(2, txtprofesor.getText());
        pst.setString(3, txthrfaltada.getText());
        pst.setString(4, txtclasefaltada.getText());
        pst.setString(5,txtdiafaltada.getText());
       pst.setString(6, txtmotivo.getText());

        int n = pst.executeUpdate();
          if (n>0){
              JOptionPane.showMessageDialog(null,"FALTA REGISTRADA");
          }

    }catch (Exception e){

         JOptionPane.showMessageDialog(null,"DISCULPE LA FALTA NO A SIDO REGISTRADA/INGRESE LOS DATOS NUEVAMENTE");   
    }    

}     

Comments

¿Esta bien para que? ¿Que quieres hacer?

---

Hola peter bueno creo que el que me puso el "punto negativo"no le gusto mi pregunta?,okey peter lo que yo quiero sabber :) fa forma de obtimizar & escribir codigo limpio o leible
(EL CODIGO QUE ESTOY USANDO ES LEIBRE,SORRY SOY NUEVO Y ME GUSTARIA SABER UN CONSEJO Y SEGUN DICE EL TITULO SON EXPERTO EN ESTA COMUNIDAD Y ME GUSTARIA SABER DE LOS EXPERTO EN PROGRAMACION)

me doy a enteder?

---

A lo que me refiero es a que tienes que plantear una pregunta que se refiere a un problema para buscar una solución y aquí no parece que tengas ningún problema, ni error, ni nada.

¿Me explico?

Por eso pedía que explicaras que quieres hacer de forma genera y si tienes un problema o error.

Saludos.

---

Tienes toda la razon :)

Answer 1

Hay un pequeño error creo. Esa parte de código iria en la capa de persist dentro de model. El modelo no puede mostrar mensajes, de eso se debe encargar la vista, y el modelo no puede emplear la vista. Deberias retornar el valor de n en esa función, al modelo y desde el controlador escribir un mensaje a través la vista, ésta con una función de showMessage donde le pases el mensaje al JOptionPane. El controlador llamará a la función del modelo que devuelve el valor, por ejemplo "model.insert() filas insertadas".