0

Quisiera saber que estoy haciendo mal en el código, resulta que intento conseguir los datos de la sentencia en el JFrame. les dejo el codigo lo que llevo

package Formulario;


import Formulario.Menu;
import Clases.*;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.logging.Level;
import java.util.logging.Logger;

public class Login extends javax.swing.JFrame {

   public static Controlador controller;
   public static ResultSet rs;

    public Login() {
        initComponents();
    }

    /**
     * This method is called from within the constructor to initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is always
     * regenerated by the Form Editor.
     */
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">                          
    private void initComponents() {

        jPanel1 = new javax.swing.JPanel();
        jLabel1 = new javax.swing.JLabel();
        jLabel2 = new javax.swing.JLabel();
        btIngresar = new javax.swing.JButton();
        tfUsuario = new javax.swing.JTextField();
        lbMsj = new javax.swing.JLabel();
        psPSW = new javax.swing.JPasswordField();
        jLabel3 = new javax.swing.JLabel();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        jLabel1.setText(" Usuario :");

        jLabel2.setText("Clave :");

        btIngresar.setText("INGRESAR");
        btIngresar.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                btIngresarActionPerformed(evt);
            }
        });

        tfUsuario.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                tfUsuarioActionPerformed(evt);
            }
        });

        jLabel3.setFont(new java.awt.Font("Tahoma", 0, 18)); // NOI18N
        jLabel3.setText("Sistema de Inventario de Activos Fijos");

        javax.swing.GroupLayout jPanel1Layout = new javax.swing.GroupLayout(jPanel1);
        jPanel1.setLayout(jPanel1Layout);
        jPanel1Layout.setHorizontalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(jPanel1Layout.createSequentialGroup()
                .addGap(91, 91, 91)
                .addComponent(lbMsj, javax.swing.GroupLayout.PREFERRED_SIZE, 83, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                .addComponent(btIngresar)
                .addGap(45, 45, 45))
            .addGroup(jPanel1Layout.createSequentialGroup()
                .addGap(66, 66, 66)
                .addGroup(jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                    .addGroup(jPanel1Layout.createSequentialGroup()
                        .addGroup(jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false)
                            .addComponent(jLabel1, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                            .addComponent(jLabel2, javax.swing.GroupLayout.PREFERRED_SIZE, 82, javax.swing.GroupLayout.PREFERRED_SIZE))
                        .addGap(18, 18, 18)
                        .addGroup(jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false)
                            .addComponent(tfUsuario)
                            .addComponent(psPSW, javax.swing.GroupLayout.PREFERRED_SIZE, 88, javax.swing.GroupLayout.PREFERRED_SIZE)))
                    .addComponent(jLabel3))
                .addContainerGap(68, Short.MAX_VALUE))
        );
        jPanel1Layout.setVerticalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(jPanel1Layout.createSequentialGroup()
                .addGap(40, 40, 40)
                .addComponent(jLabel3)
                .addGroup(jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                    .addGroup(jPanel1Layout.createSequentialGroup()
                        .addGap(138, 138, 138)
                        .addGroup(jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                            .addComponent(jLabel1)
                            .addComponent(tfUsuario, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                        .addGap(18, 18, 18)
                        .addGroup(jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                            .addComponent(jLabel2)
                            .addComponent(psPSW, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                        .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                        .addComponent(lbMsj, javax.swing.GroupLayout.DEFAULT_SIZE, 131, Short.MAX_VALUE)
                        .addContainerGap())
                    .addGroup(javax.swing.GroupLayout.Alignment.TRAILING, jPanel1Layout.createSequentialGroup()
                        .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                        .addComponent(btIngresar)
                        .addGap(30, 30, 30))))
        );

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addGap(58, 58, 58)
                .addComponent(jPanel1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addContainerGap(142, Short.MAX_VALUE))
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(jPanel1, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                .addContainerGap())
        );

        pack();
    }// </editor-fold>                        

    private void tfUsuarioActionPerformed(java.awt.event.ActionEvent evt) {                                          
        // TODO add your handling code here:
    }                                         

    private void btIngresarActionPerformed(java.awt.event.ActionEvent evt) {                                           

        String Passw = new String(psPSW.getPassword());
        try{
            controller =new Controlador();
            controller.crearConexion();
           }catch (SQLException ex){

               Logger.getLogger(Login.class.getName()).
               log(Level.SEVERE, null, ex);
               }

        String sql="Select * from Usuario WHERE Us_Nom='"+tfUsuario.getText()+"' and Us_Pwd='"+Passw+"'";
            try {
                 rs=controller.mandarQUERY(sql);
            while (rs.next()){

                    Menu vent= new Menu();
                    vent.setVisible(true);
                    this.setVisible(false);
                    }
                } catch (SQLException ex) {
            Logger.getLogger(Login.class.getName()).
            log(Level.SEVERE, null, ex);
            }
    }                                          

    /**
     * @param args the command line arguments
     */
    public static void main(String args[]) {
        /* Set the Nimbus look and feel */
        //<editor-fold defaultstate="collapsed" desc=" Look and feel setting code (optional) ">
        /* If Nimbus (introduced in Java SE 6) is not available, stay with the default look and feel.
         * For details see http://download.oracle.com/javase/tutorial/uiswing/lookandfeel/plaf.html 
         */
        try {
            for (javax.swing.UIManager.LookAndFeelInfo info : javax.swing.UIManager.getInstalledLookAndFeels()) {
                if ("Nimbus".equals(info.getName())) {
                    javax.swing.UIManager.setLookAndFeel(info.getClassName());
                    break;
                }
            }
        } catch (ClassNotFoundException ex) {
            java.util.logging.Logger.getLogger(Login.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (InstantiationException ex) {
            java.util.logging.Logger.getLogger(Login.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (IllegalAccessException ex) {
            java.util.logging.Logger.getLogger(Login.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (javax.swing.UnsupportedLookAndFeelException ex) {
            java.util.logging.Logger.getLogger(Login.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        }
        //</editor-fold>

        /* Create and display the form */
        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new Login().setVisible(true);
            }
        });
    }

    // Variables declaration - do not modify                     
    private javax.swing.JButton btIngresar;
    private javax.swing.JLabel jLabel1;
    private javax.swing.JLabel jLabel2;
    private javax.swing.JLabel jLabel3;
    private javax.swing.JPanel jPanel1;
    private javax.swing.JLabel lbMsj;
    private javax.swing.JPasswordField psPSW;
    private javax.swing.JTextField tfUsuario;
    // End of variables declaration                   
}
1
  • Tu pregunta no es muy clara, o no esta bien definida, a lo que veo, estas generando una ventana Menu por cada row que tenga el ResultSet, lo que puedes hacer, es que si vas a mostrar en una nueva ventana los datos del usuario puedes almacenar cada campo de la fila en su respectiva variable, para después esos datos mandarlos a un label de la ventana Menu,
    – jonathan
    Commented el 26 jun. 2016 a las 7:09

3 respuestas 3

1
 String nombreUsuario=null;
 Menu mnVista;
 while (rs.next()){
  nombreUsuario=rs.getString(0); //ejemplo
 }
mnVista=new mnVista(nombreUsuario);//opcion 1 pase por constructor
mnVista.jLabel1.setText(nombreUsuario);// componente static

Estas son algunas sugerencias adaptandose a lo que tienes de código, por que puede que no sea los más óptimo

1

Para mostrar el resultado de una sentencia SQL lo mejor suele ser mostrarlo mediante un JTable.

Te defines un JTable con el número de columnas igual al número de campos que quieres que se muestren del resultado de la select y sin filas.

Dentro del bucle que tienes:

while(rs.next){
...
}

Vas incrementando el tamaño del JTable mientras haya registros y le asignas a cada columna el valor que quieres

0

Te daré dos buenas prácticas cuando trabajes con Swing/JavaFX.

  1. Utiliza el patrón VO para mapear tus tablas a objetos.
  2. Aprovecha siempre que puedas las clases genéricas de controles en Swing.
  3. Nunca mezcles lógica de negocio con vistas.

Utilizar el patrón VO

Es muy recomendable utilizar el patrón VO cuando trabajes con bases de datos. Al mapear una tabla a un objeto, encapsulas toda esta información en objetos Java, de tal manera que puedes acceder a la información de manera clara, limpia y concisa.

while(rs.next()) {
    Persona persona = new Persona();
    persona.setNombre(rs.getString("nombre"));
    persona.setApellido(rs.getString("apellido"));
    persona.setDireccion(rs.getString("direccion"));
    persona.setTelefono(rs.getString("telefono"));
}

Aprovechar las clases genéricas de Swing/JavaFX

En JTable no hay modo utilizar Generics porque no JTable no es genérica, sin embargo puedes hacerlo haciendo un DefaultTableModel. Sin embargo, en algunos controles como JList o JComboBox es posible asociar un determinado tipo de dato que puede ser un objeto al control.

Para mostrar una List de objetos en tu JTable, es más fácil usando el patrón VO.

DefaultTableModel model = (DefaultTableModel) tblPersonas.getModel();
for(Persona persona : personas) {
  model.addRow(new Object[] {
    persona.getNombre(),
    persona.getApellido(),
    persona.getDireccion(),
    persona.getTelefono()
  });
}

Nunca mezcles código de negocio con vistas

Esto es quizás el punto más importante de los tres. Mezclar código de negocio con vistas es sentenciar tu aplicación al fracaso inminente. Una aplicación que no establece capas simplemente no puede escalar (crecer en un futuro, añadiéndose funcionalidad). Además, llegará un punto en que es una tortura mantenerlo, porque todo está enredado. Una auténtica maraña.

Links de interés:

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.