Alguien dispuesto a revisar codigo?

[juanito1712]

Me compre un libro sobre java hace unos dias y desde entonces ando dándole fuerte a esto.

Acabo de crear mi primera aplicación de prueba y funcionar funciona, pero no si si lo he realizado correctamente.

Ya se que lo que pido es una faena relativamente pesada así que si nadie esta dispuesto a hacerlo es comprensible y tampoco voy a mosquearme.

Lo que decía, he creado mi primera aplicación, esta tiene 3 calses

• main
• pantalla
• verclientes

lo que no se es si lo he realizado correctamente o si he cometido una aberración a la programación orientada a objetos, eso es basicamente lo que quiero que se mire y se proponga la solución correcta (No creo que lo haya hecho correcto al 100% ni mucho menos)

clase main

Código java:

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.Statement;
 
public class Main {
    public static void main(String[] args)
    {
        Connection conn = null;  
        Statement  st   = null;  
        ResultSet  rst1 = null; 
        try {  
            // Cargamos el controlador JDBC  
             Class.forName("org.hsqldb.jdbcDriver"); 
            conn = DriverManager.getConnection("jdbc:hsqldb:file:/usuarios");   
            try {  
                st  = conn.createStatement();  
                rst1 = st.executeQuery("SELECT * FROM empleados");  
                System.out.println("las tablas ya estan creadas");
                st.close();  
                conn.close(); 
                Pantalla ventana;
                ventana= new Pantalla();
                ventana.setVisible(true);
            } catch (Exception ex){  
                System.out.println(ex);
                //crear las tablas ya que no existen
            }  
        } catch (Exception ex){  
            System.err.println("Se pa producido un error al cargar el controlador JDBC");  
            return;  
        } 
        
    }
}
 

clase pantalla

Código java:

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;
import java.sql.SQLException;
import java.sql.Statement;
 
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame; 
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JPanel;
 
 
public class Pantalla extends JFrame
{
    JPanel marco;
    public Pantalla()
    {
        setBounds(500,300,400,300);
        setTitle("Facturas mini");
        setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
        addWindowListener (new WindowAdapter()
        {
            public void windowClosing(WindowEvent arg0)
            {
                System.exit(0);
            }
        }
        );
        
        JMenuBar barraMenu;
        barraMenu=new JMenuBar();
        SeleccionarMenubar sm=new SeleccionarMenubar();
        
        
        JMenu clientes;
        JMenuItem nuevoCliente,buscarCliente;
        clientes=new JMenu("Clientes");
        
        buscarCliente = new JMenuItem("Buscar cliente");
        buscarCliente.addActionListener(sm);
        buscarCliente.setActionCommand("buscarcliente");
        clientes.add(buscarCliente);
        nuevoCliente = new JMenuItem("Nuevo cliente");
        clientes.add(nuevoCliente);
        barraMenu.add(clientes);
        
        getContentPane().add(barraMenu,BorderLayout.NORTH);
        marco = new JPanel();
        marco.setBackground(Color.GREEN);
        getContentPane().add(marco,BorderLayout.CENTER);
        
    }
    
    public class SeleccionarMenubar implements ActionListener
    {
        @Override
        public void actionPerformed(ActionEvent e) 
        {
            String comando;
            comando=e.getActionCommand();
            if (comando.equals("buscarcliente")){
                getContentPane().remove(marco);
                try {
                    marco=new Verclientes();
                } catch (SQLException e1) {
                    
                    e1.printStackTrace();
                }
                getContentPane().add(marco,BorderLayout.CENTER);
                marco.updateUI();
            }       
        }
    }   
}
 

clase verclientes

Código java:

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
 
 
import javax.swing.JPanel;
import javax.swing.JTable;
import javax.swing.table.DefaultTableModel;
 
public class Verclientes extends JPanel
{ 
    public Verclientes()throws SQLException 
    {
        Connection conn2 = null;  
        Statement  st2   = null;  
        String     sql2  = null;
        ResultSet  rst2 = null; 
        
        try {  
            // Cargamos el controlador JDBC  
             Class.forName("org.hsqldb.jdbcDriver");  
        } catch (Exception ex){  
            System.err.println("Se pa producido un error al cargar el controlador JDBC");  
            return;  
        }  
        conn2 = DriverManager.getConnection("jdbc:hsqldb:file:/usuarios");  
        st2  = conn2.createStatement();  
        rst2 = st2.executeQuery("SELECT * FROM empleados");  
        
        
        DefaultTableModel modelo = new DefaultTableModel();
        JTable tabla = new JTable(modelo);
        
        
     // Creamos las columnas.
        modelo.addColumn("id");
        modelo.addColumn("nombre");
        modelo.addColumn("nacimiento");
 
        // Bucle para cada resultado en la consulta
        while (rst2.next())
        {
           // Se crea un array que será una de las filas de la tabla. 
           Object [] fila = new Object[3]; // Hay tres columnas en la tabla
 
           // Se rellena cada posición del array con una de las columnas de la tabla en base de datos.
           for (int i=0;i<3;i++)
              fila[i] = rst2.getObject(i+1); // El primer indice en rs es el 1, no el cero, por eso se suma 1.
 
           // Se añade al modelo la fila completa.
           modelo.addRow(fila); 
        }
        tabla = new JTable();
        tabla.setModel(modelo);
        add(tabla);
        st2.close();  
        conn2.close(); 
        
    }
}
 

[Heimish2000]

Para ver si hace lo que quieres, deberías primero decir que es lo que quieres xD

[Fuzzylog]

La primera burrada es anidar bloques try.
Se hace un único try.
Se incluyen tantos catch como errores distintos quieras manejar.

Busca por internet ejemplos de try-catch pq te veo hecho un lío

[francopai]

Mi humilde aporte: una clase no se puede llamar "verClientes", una clase es una representacion de una entidad, gralmente es un sustantivo como: VisorDeClientes, Automovil, Inmueble, etc.
verCliente, hace referencia a un metodo, ya que es un verbo.

Espero que te sea de ayuda
Saludos

[juanito1712]

fuzzylog-gracias esos son los consejos que andaba buscando

heimish-no es ver si hace lo que quiero, lo que quiero lo hace perfectamente, lo que quiero que reviseis son las burradas que pueda haber cometidio como bien ha hecho fuzzylog o ya decir que el programa esta escrito correctamente

[juanito1712]

Iniciado por francopai

Mi humilde aporte: una clase no se puede llamar "verClientes", una clase es una representacion de una entidad, gralmente es un sustantivo como: VisorDeClientes, Automovil, Inmueble, etc.
verCliente, hace referencia a un metodo, ya que es un verbo.

Espero que te sea de ayuda
Saludos

vale que lo que me quieres decir no es que tengo mal la clase en si, sino simplemente el nombre de la clase que tendria que ser VisorcClientes o CreadorClientes y ya si lo necesitara crearia un metodo dentro que se podria llamar verClientes o crearClientes.
he comprendido bien?

[juanito1712]

fuzzylog, de nuevo gracias

arreglado try/catch

Código java:

try {  
            // Cargamos el controlador JDBC  
            Class.forName("org.hsqldb.jdbcDriver"); 
            conn = DriverManager.getConnection("jdbc:hsqldb:file:/usuarios");   
            st  = conn.createStatement();  
            rst1 = st.executeQuery("SELECT * FROM empleados");  
            System.out.println("las tablas ya estan creadas");
            st.close();  
            conn.close(); 
            Pantalla ventana;
            ventana= new Pantalla();
            ventana.setVisible(true);
            
        } catch (SQLException ex){ 
            //System.out.println(ex);
            System.out.println("Las base de datos no existe");  
            System.out.println("Creando bases de datos");  
            //codigo para crear bases de datos
            return;  
        } 
        catch (ClassNotFoundException ex){ 
            //System.out.println(ex);
            System.out .println("Se pa producido un error al cargar el controlador JDBC");  
            return;  
        }
 

[francopai]

Si, asi es. Aqui te dejo un link para que te orientes un poco sobre nombres y demas asuntos.
http://java.sun.com/docs/books/jls/t...tml/j3TOC.html

Saludos

[CarlosAP]

Quizás te interese leerte algo sobre el principio de ocultación de la información, ahora es una aplicación pequeña hecha por ti pero cuando otra persona tenga que hacer uso de las clases que tú has generado te interesará evitar aliasing y que toquen atributos que no deben...

[megajandro]

yo veo lo siguiente:
1- No existe un esquema de objetos de dominio. Por ej: cliente. (Aunque los atributos parecen mas de una persona que de cliente). No existe ninguno de los conceptos de POO. En un simple ABMC(CRUD) ya los podes aplicar.
2- No hay una separacion de la presentacion (Swing) con tu capa de manejo de BD(JDBC).
Para ver mejor esto deberias entender como maneja swing el modelo MVC.
3- Es cierto lo del try, catch que dijo juanito1712. Aunque para un mejor manejo seria que entiendas que es una excepcion chequeada y cual no.
4- Tenes codigo repetido para obtener la conexion de la BD.

Y yo agregaria utilizar algun plugin para verificar bugs como (FindBugs) y un checkstyle creo que
te aniquilaria.

Saludos.

[juanito1712]

megajandro:
juanito1712 soy yo :P eso es una modificacion que me sugirió fuzzylog.

Pero bueno tu post tiene mucho contenido y no entiendo nada de lo que me quieres hacer ver
es la primera vez que me meto en esto de programación orientada a objetos y el libro sobre el que me baso, me parecía muy bueno al comprarlo y me ha parecido muy pobre al terminarlo

si no te importase ir poco a poco...

1- No existe un esquema de objetos de dominio. Por ej: cliente. (Aunque los atributos parecen mas de una persona que de cliente). No existe ninguno de los conceptos de POO. En un simple ABMC(CRUD) ya los podes aplicar.

que es un esquema de objetos de dominio?

[CarlosAP]

Creo que lo que te quiere decir es que en ningún momento has creado una clase Persona o Cliente (Recuerda, nombres de clases empiezan en mayúsculas cada una de sus primeras letras de palabra), como sería lo lógico, está claro que eso funciona, si, pero si te fijas la consulta de la base de datos te devuelve en cada fila (conjunto de datos perteneciente a la misma entrada) un objeto, ¿no sería lógico que cada entrada de una misma tabla sea un objeto de la misma clase? Es decir, tu tabla Clientes tiene, por ejemplo 5 entradas (Filas distintas) de las cuales cada una tiene 5 columnas, o lo que es lo mismo, tienes cinco objetos Cliente de los cuales cada uno tiene 5 atributos... Quizás eso sería más lógico que construir un array de Object del cual extraes la información índice a índice. En general no debe darte miedo crear clases, al fin y al cabo son para eso, mejoran la extensibilidad y legibilidad del código, es más facil que cuando tengas que agregar, por ejemplo, a los clientes un campo nuevo, lo hagas tocando únicamente la clase Cliente. En resumen, utiliza los objetos y las ventajas que éstos te aportan!

No se si me he explicado lo más mínimo pero lo he intentado... :D

[juanito1712]

explicarte , lohas hecho muy bien y creo que he entendido lo que quieres decir
ahora voy a ver si soy capaz de plasmarlo en el programa
gracias