Test Foro de elhacker.net SMF 2.1

No sé cual sea un nombre apropiado para esta consulta ya que no se que sea la causa, pero la idea es esa, enlazar 3 clases. Después de darle varias vueltas al asunto, no pude figurar que es lo que estoy haciendo mal.

La clase principal (InterfacePrincipal , que es la GUI), crea en tiempo de ejecución y en función a una determinada cantidad, componentes para mostrar la información relacionada a un articulo de venta.

Por lo que dependiendo esta cantidad se crearan tantos objetos sean necesarios de otra clase que describo a continuación:


Clase InformacionArticulo



// Package e Imports

public class InformacionArticulo extends JPanel{
   
       Carrito carrito = new Carrito();
   
       private JButton jButtonComprar;
       
       String nombre;
       String precio;

   public InformacionArticulo(
               
               String nombre ,
               String precio
       
       ){
           
               this.nombre = nombre;
               this.precio = precio;
               
               formato(); // Utilizado para medidas y dar formato a los componentes
   }

   private void formato(){
           
           jButtonComprar.addActionListener(new ActionListener(){
               @Override
               public void actionPerformed(ActionEvent evt){
                   carrito.agregar(nombre , precio);
               }
           });
         
           add(jButtonComprar);
   }
}




Full Code: http://pastebin.com/qXrQ0Rf9]http://pastebin.com/qXrQ0Rf9]http://pastebin.com/qXrQ0Rf9 (http://pastebin.com/qXrQ0Rf9)


De lo anterior elimine código irrelevante para no hacer tan largo el tema y confundirlos con otros componentes que no vienen al caso. Entonces de la anterior clase lo más importante es el botón jButtonComprar y su Action Performed, el cual al ser presionado se deberá estar cargando información en una tabla de la clase principal (InferfacePrincipal) a traves del médoto agregar() de la clase Carrito


Clase Carrito


public class Carrito {
   
   InterfacePrincipal gui = new InterfacePrincipal; // *Nota Importante, esta es la clase principal
   
   public Carrito(){}
   
   public void agregar(String nombre , String precio){
       
       String vector[] = {"#", nombre , precio};
       DefaultTableModel modelo = (DefaultTableModel)gui.getModeloTablaCarrito();
       modelo.addRow(vector);

       System.out.println("Termina método agregar");
   }
}




De donde getModeloTablaCarrito() devuelve el modelo de la tabla donde se muestran los articulos agregados al carrito. Posteriormente se agrega una fila con vector como Data al modelo de la tabla que se encuentra en GUI.

Pero el problema está en que no se ve reflejado en la tabla, y he comprobado que el ActionPerformed del jButtonComprar funcionan, las variables nombre y precio son las correctas y pasan correctamente a agregar() de Carrito, e inclusive se imprime "Termina método agregar". Por lo que sé que paso por todo lo anterior dentro de ese método.

*Nota Importante Aquí lo que hice antes fue solo hacer la referencia, es decir, InterfacePrincipal gui; pero obtenia un NullPointerException, que supongo era lógico, por lo que tuve que crear el Objeto, pero de lo que no estoy seguro es si al crear un Objeto InterffacePrincipal, se crea otro totalmente diferente al original de la Clase Principal (GUI)  puesto que dentro del constructor de InferfacePrincipal se inicializan todos los componentes,

Estoy trabjando en NetBeans, y he probado el código para agregar filas con vector como Data dentro de la misma clase principal donde se encuentra la Tabla y funciona correctamente. El problema es que no sé que es lo que está pasando, de la forma que lo estoy haciendo no se refleja en la tabla.

En si, InferfacePrincipal llama a InformacionArticulos el cual llama a Carrito el cual llama a InterfacePrincipal para obtener su Tabla y agregar filas con cierta información..


Notas:
De esta manero creo los Objetos de InformacionArticulo

private void mostrarProductos(
           JPanel contenedor , JComboBox producto
   ){
       String tipo = producto.getSelectedItem().toString();
       String nombre;
       String existencia;
       String precio;
       String imagen;
       File archivo = new File(generarRutaArchivo(tipo));
       
       int c_Height = 0;
       int c_Articulos = 0;
       
       int cantidadArticulos = getCantidadArticulos(archivo);
       for(int i=0 ; i<cantidadArticulos ; i++){
           
           nombre = getInformacionArchivo(archivo , i , 0);
           existencia = getInformacionArchivo(archivo , i , 1);
           precio = getInformacionArchivo(archivo , i , 2);
           imagen = generarRutaImagen(tipo , nombre);
           
           contenedor.add(new InformacionArticulo(
                   tipo , nombre , existencia , precio , imagen));
           
           c_Height += 300;
           c_Articulos++;
       }
       
       contenedor.setPreferredSize(new Dimension(400 , c_Height));
       contenedor.setLayout(new GridLayout(c_Articulos , 1));
   }



De donde getInformaciónArticulo(File , int , int) va a leer un archivo donde está alojada la información de todos los articulos pertenecientes a una categoria, ejemplo Teclado.

La forma de uso sería:
getInformacionArticulo(ArchivoALeer , NumeroDeLinea , IndiceDeInformacion)

Donde en IndiceDeInformacion.
• 0 = Nombre del articulo
• 1 = Existencias
• 2 = Precio del articulo
• 3 = Descripcion

Entonces por ejemplo si del JComboBox selecciono teclado, llamará al método anteriormente expuesto.

(http://k43.kn3.net/7/E/D/0/9/C/3AB.png)


private void jComboBoxProductosActionPerformed(java.awt.event.ActionEvent evt) {                                                  
       cambiarPanel("Informacion"); // Cambia a un panel donde se mostrará la información de todos los articulso de la categoria seelccionada.
       
       JPanel panel = new JPanel();
       mostrarProductos(panel , jComboBoxProductos);
       
       jScrollPane3.setVerticalScrollBarPolicy(VERTICAL_SCROLLBAR_AS_NEEDED);
       jScrollPane3.setViewportView(panel);
       
       
}


Y una vez se tenga algo como la siguiente imagen al dar click en el botón "Agregar al Carrito", recordando a la clase InformaciónArticulo, especificamente el ActionPerformed del Boton:


ButtonComprar.addActionListener(new ActionListener(){
               public void actionPerformed(ActionEvent evt){
                   carrito.agregar(nombre , precio);
               }
           });



Se pasan como argumentos nombre y precio a agregar() de Carrito.


public void agregar(String nombre , String precio){
             
       String vector[] = {"#", nombre , precio};
       DefaultTableModel modelo;
       modelo = (DefaultTableModel)InterfacePrincipal.getInstance().getTabla().getModel();
       
       modelo.addRow(vector);
       
       System.out.println("Termina método agregar");
   }


De lo cual llega a imprimir por consola "Termina método agregar", dando a entender que paso por todo lo anterior a esa intrucción dentro del método. Dicho método debería de reflejar la información en la siguiente tabla, pero ahí está el problema, no lo hace.

Teóricamente, una nueva instancia es un nuevo objeto. Si tienes dudas sobre esto, puedes hacer la clase  InterfazPrincipal como un singleton, es decir que sólo exista una instancia de dicha clase. El patrón de diseño Singleton se aplica a una clase de la siguiente manera:


public class InterfazPrincipal
{
    private static InterfazPrincipal instance;

    // lo hacemos private para evitar que se creen instancias
    private InterfazPrincipal() {
    }

    public static InterfazPrincipal getInstance() {
            if (instance == null) instance = new InterfazPrincipal();
            return instance;
    }
}


Entonces, cuando desees utilizar la GUI en otras clases obtienes su instancia por medio del método estático getInstance(). Prueba con ello y si siguen los inconvenientes trata de pegar todo el código para ver posibles fallos.

Cita de: Migugami en 24 Mayo 2014, 14:25 PM
Teóricamente, una nueva instancia es un nuevo objeto. Si tienes dudas sobre esto, puedes hacer la clase  InterfazPrincipal como un singleton.

Woah, esto no lo sabía, que genial. ¿Cómo se llama el tema relacionado a eso? para buscar mas información.


También de vuelta al problema en si, probé con lo que mencionas y otras combinaciones, pero no se refleja en la tabla, los metodos que utlizé. Cabe mencionar que no hay otra tabla en esa clase por lo que no hay problema en saber cual tabla es.

getModeloTablaCarrito() // Devuelve el modelo de la tabla
getTablaCarrito() // Devuelve la tabla

Los probé dentro de la misma clase, donde se encuentra la tabla y funcionan correctamente, por lo que me lleva a pensar que realmente obtiene la tabla solo que de otro objeto y no del "original". En unos momentos edito el tema y adiciono algunas notas importantes. Gracias por tu tiempo.

Ya te entendí, la tabla está en una GUI y Carrito solo debe guardar datos en esa tabla. No es necesario el patrón Singleton, nada más tienes que pasarle la referencia de la clase InterfazPrincipal a la clase Carrito que guardará la referencia en un objeto InterfazPrincipal. Recuerda que en Java el paso de parámetros es por referencia, así que cualquier cambio que se haga en la referencia, se guardará.

public class Carrito {
InterfazPrincipal gui;

public Carrito (InterfazPrincipal gui) {
this.gui = gui;
}

public void agregar(String nombre , String precio){
       
       Object[] fila = new Object{"#", nombre , precio}; // importante, siempre debes introducir una fila Object
       DefaultTableModel modelo = (DefaultTableModel)gui.getModeloTablaCarrito();
       modelo.addRow(fila);
   }

}


public class InterfazPrincipal extends JFrame {
// le pasamos la referencia de la clase a la clase Carrito
final Carrito CARRITO_DRIVER = new Carrito(this);

// otras cosas
}

O puedes hacer uso de reflection:


public class Carrito {
InterfazPrincipal gui = Class.forName("paquete.Clase").getInstance();

public Carrito () {
}

public void agregar(String nombre , String precio){
       
        Object[] fila = new Object{"#", nombre , precio};
        DefaultTableModel modelo = (DefaultTableModel)gui.getModeloTablaCarrito();
        modelo.addRow(fila);
    }

}

Cita de: Migugami en 25 Mayo 2014, 08:11 AM
Ya te entendí, la tabla está en una GUI y Carrito solo debe guardar datos en esa tabla. No es necesario el patrón Singleton, nada más tienes que pasarle la referencia de la clase InterfazPrincipal a la clase Carrito que guardará la referencia en un objeto InterfazPrincipal. Recuerda que en Java el paso de parámetros es por referencia, así que cualquier cambio que se haga en la referencia, se guardará.

Código (=java) [Seleccionar] Expandir

public class Carrito {
InterfazPrincipal gui;

public Carrito (InterfazPrincipal gui) {
this.gui = gui;
}

public void agregar(String nombre , String precio){
       
       Object[] fila = new Object{"#", nombre , precio}; // importante, siempre debes introducir una fila Object
       DefaultTableModel modelo = (DefaultTableModel)gui.getModeloTablaCarrito();
       modelo.addRow(fila);
   }

}

Código (=java) [Seleccionar] Expandir


public class InterfazPrincipal extends JFrame {
// le pasamos la referencia de la clase a la clase Carrito
final Carrito CARRITO_DRIVER = new Carrito(this);

// otras cosas
}

O puedes hacer uso de reflection:

Código (=java) [Seleccionar] Expandir


public class Carrito {
InterfazPrincipal gui = Class.forName("paquete.Clase").getInstance();

public Carrito () {
}

public void agregar(String nombre , String precio){
       
        Object[] fila = new Object{"#", nombre , precio};
        DefaultTableModel modelo = (DefaultTableModel)gui.getModeloTablaCarrito();
        modelo.addRow(fila);
    }

}

¡Muchas Gracias!

(http://k40.kn3.net/AD2132EBB.png)