Ayuda agregando objetos a un ArrayList

[Casvill]

Hola!

Un amigo va a abrir un local de pizzas y yo para ir poniendo en práctica lo que he aprendido en la u, quise tratar de hacer un software que le sirviera (aún no sé bases de datos pero cuando tenga listo el código en java, empezaré a aprender bases de datos para implementarlo).

(En Colombia las universidades públicas están en paro desde hace meses, así que sólo alcancé a ver unas cuantas pocas clases, todo lo demás lo he leído por cuenta própia, disculpen las malas prácticas que de seguro habrán u.u)

El caso es que tengo una clase que es la interfaz, otra clase Control, Categoría y Producto (y otras dos que aún no uso).

Cotrol tiene una lista de Categorías, por otra parte Categoría tiene una lista de Productos.

Control ----> Categorías -----> Productos

Estoy tratando de añadir un Producto a una categoría pero por alguna razón me sale un error, y no sé cuál sea el error, ya lo traté de hacer por aparte en otros proyectos y me funciona, no sé que estaré haciendo mal aquí -.-"

Si alguien pudiera ayudarme se lo agradecería mucho, me tiene con dolor de cabeza pero no quiero abandonar esto, pues es el primer proyecto fuera de la u que me propongo a hacer.

PD: El código compila, el error sale cuando añado un producto a una categoría (NO LA AÑADE !! :S)

Aquí les dejo el código:

https://github.com/Casvill/Software-Local

GUI: (la mayor parte de este código está hecho por netBeans, yo hice los actionPerformed)
El botón que se encarga de añadir el producto (habiendo creado previamente una categoría) es  jbCrearPAñadir.

Estos son los errores que aparecen cuando le doy al boton de añadir productos:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
   at control.Categoria.añadirP(Categoria.java:26)
   at control.Control.añadirPro(Control.java:30)
   at gui.Inventario_Productos.jbCrearPAñadirActionPerformed(Inventario_Productos.java:401)
   at gui.Inventario_Productos.access$100(Inventario_Productos.java:6)
   at gui.Inventario_Productos$3.actionPerformed(Inventario_Productos.java:127)
   at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
   at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2348)
   at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
   at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
   at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
   at java.awt.Component.processMouseEvent(Component.java:6539)
   at javax.swing.JComponent.processMouseEvent(JComponent.java:3324)
   at java.awt.Component.processEvent(Component.java:6304)
   at java.awt.Container.processEvent(Container.java:2239)
   at java.awt.Component.dispatchEventImpl(Component.java:4889)
   at java.awt.Container.dispatchEventImpl(Container.java:2297)
   at java.awt.Component.dispatchEvent(Component.java:4711)
   at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4904)
   at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4535)
   at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4476)
   at java.awt.Container.dispatchEventImpl(Container.java:2283)
   at java.awt.Window.dispatchEventImpl(Window.java:2746)
   at java.awt.Component.dispatchEvent(Component.java:4711)
   at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:760)
   at java.awt.EventQueue.access$500(EventQueue.java:97)
   at java.awt.EventQueue$3.run(EventQueue.java:709)
   at java.awt.EventQueue$3.run(EventQueue.java:703)
   at java.security.AccessController.doPrivileged(Native Method)
   at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
   at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:84)
   at java.awt.EventQueue$4.run(EventQueue.java:733)
   at java.awt.EventQueue$4.run(EventQueue.java:731)
   at java.security.AccessController.doPrivileged(Native Method)
   at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
   at java.awt.EventQueue.dispatchEvent(EventQueue.java:730)
   at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:205)
   at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
   at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
   at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
   at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
   at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)


Gracias de antemano!  

[rub'n]

Que tal dog,

Estoy tratando de añadir un Producto a una categoría pero por alguna razón me sale un error, y no sé cuál sea el error, ya lo traté de hacer por aparte en otros proyectos y me funciona, no sé que estaré haciendo mal aquí -.-"

Si alguien pudiera ayudarme se lo agradecería mucho, me tiene con dolor de cabeza pero no quiero abandonar esto, pues es el primer proyecto fuera de la u que me propongo a hacer.

wtf dog, o el compilador te arroja excepción o no te arroja? una de dos, ese log con la traza del error puedes publicarlo aquí también, fíjate también

Que hacer eso ayuda, a que te ayuden mas rápidamente, porque tu clase es algo larga y muchos no cuentan con el tiempo necesario para reproducir tu error  

[Casvill]

wtf dog, o el compilador te arroja excepción o no te arroja? una de dos, ese log con la traza del error puedes publicarlo aquí también, fíjate también

Que hacer eso ayuda, a que te ayuden mas rápidamente, porque tu clase es algo larga y muchos no cuentan con el tiempo necesario para reproducir tu error  

Gracias por responder, dog! 

Edité el post, subí el código a github y pusé aquí lo que aparece en el log.

Feliz día! 

[seupcdm]

Estoy leyéndolo directamente desde github, pero en la clase Categoría tienes dos constructores que hacen cosas distintas.

Código (java) [Seleccionar] Expandir


   public Categoria()
   {
       codigo = "";
       nombre = "Otros";
       productos = new ArrayList();
   }

   public Categoria(String codigo, String nombre)
   {
       this.codigo = codigo;
       this.nombre = nombre;
   }

No me preocupan los valores de las variables porque no sé para qué lo utilizas después, pero lo que sí me llama la atención es que solo instancies tu ArrayList en uno de los constructores, y más aún que la única instancia que hagas de las categorías sea a través del constructor que NO instancia el ArrayList:

Código (java) [Seleccionar] Expandir

private void jbCrearCatAñadirActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_jbCrearCatAñadirActionPerformed
       
       Categoria categoria = new Categoria("", jtfCrearCatNombre.getText());
       control.añadirC(categoria);
       jcbCrearPCategoria.addItem(jtfCrearCatNombre.getText());
       jcbEditarCategoria.addItem(jtfCrearCatNombre.getText());
       
       jtfCrearCatNombre.setText("");
       
       /*BorrarEsto:----------------------------------*/
       for (int i = 0; i < control.getCategorias().size(); i++)
       {
           System.out.println(control.getCategorias().get(i).getNombre());
       }
       /*FinBorrar.-----------------------------------*/
       

       //actulizarTablaProductos();
   }//GEN-LAST:event_jbCrearCatAñadirActionPerformed

Vamos, igual me estoy inventando las cosas, pero según lo que veo y sin haber ejecutado tu programa, tienes dos opciones: usar el otro constructor o añadir la instancia del ArrayList al constructor que sí usas.

Código (java) [Seleccionar] Expandir


   public Categoria()
   {
       codigo = "";
       nombre = "Otros";
       productos = new ArrayList();
   }

   public Categoria(String codigo, String nombre)
   {
       this.codigo = codigo;
       this.nombre = nombre;
       productos = new ArrayList();
   }

También te diría que deberías especificar el tipo de variable que tu ArrayList contiene a través del operador <>. Si usas Java 7 o posterior, puedes hacer una instancia genérica tal que así new ArrayList<>();, si no, especifica el tipo new ArrayList<Producto>();

Y bueno, no voy a decir mucho más, pero aquí se debate qué es mejor, si utilizar la clase List o Arraylist, por si te interesa:
https://stackoverflow.com/questions/2279030/type-list-vs-type-arraylist-in-java

También te recomendaría leer bien las trazas de los errores.

Código [Seleccionar] Expandir

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
  at control.Categoria.añadirP(Categoria.java:26)
  at control.Control.añadirPro(Control.java:30)
  at gui.Inventario_Productos.jbCrearPAñadirActionPerformed(Inventario_Productos.java:401)

Es un puntero a null, lo que por lo general quiere decir que estás intentando acceder a las propiedades de un objeto que aunque esté declarado, no está instanciado. Tienes que ir de abajo arriba, y el último punto donde te casca es en la línea 26 de la clase Categoría. Justo donde intentas añadir a tu ArrayList el objeto producto, que por lo que veo sí está instanciado, lo cual me hace corroborar lo que te he comentado antes.

Código (java) [Seleccionar] Expandir

public void añadirP(Producto producto)
    {
        productos.add(producto);
    }

Un saludo.

[Casvill]

Hola!
Hey muchas gracias por responder, hayaste mi error y aparte ahora aprendí a estar más pendiente de los constructores, siempre se aprende algo nuevo, muchas gracias de nuevo!

También te diría que deberías especificar el tipo de variable que tu ArrayList contiene a través del operador <>. Si usas Java 7 o posterior, puedes hacer una instancia genérica tal que así new ArrayList<>();, si no, especifica el tipo new ArrayList<Producto>();

Esto no lo sabía, pensaba que sólo debía especificarse cuando se declara (aunque hasta ahora no me ha dado problema por hacerlo así), si es por buenas prácticas, pues lo voy a implementar!

Y bueno, no voy a decir mucho más, pero aquí se debate qué es mejor, si utilizar la clase List o Arraylist, por si te interesa:
https://stackoverflow.com/questions/2279030/type-list-vs-type-arraylist-in-java

Gracias! siempre he usado ArrayList, pero al parecer recomiendan usar más List, no lo he leído completamente aún, pero me interesa mucho, ya lo dejé pendiente para leer!

También te recomendaría leer bien las trazas de los errores.

Código [Seleccionar] Expandir

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
  at control.Categoria.añadirP(Categoria.java:26)
  at control.Control.añadirPro(Control.java:30)
  at gui.Inventario_Productos.jbCrearPAñadirActionPerformed(Inventario_Productos.java:401)

Es un puntero a null, lo que por lo general quiere decir que estás intentando acceder a las propiedades de un objeto que aunque esté declarado, no está instanciado. Tienes que ir de abajo arriba, y el último punto donde te casca es en la línea 26 de la clase Categoría. Justo donde intentas añadir a tu ArrayList el objeto producto, que por lo que veo sí está instanciado, lo cual me hace corroborar lo que te he comentado antes.

Código (java) [Seleccionar] Expandir

public void añadirP(Producto producto)
    {
        productos.add(producto);
    }

Un saludo.

Como dije, siempre se aprende algo nuevo, esto tampoco lo sabía, muchas gracias!
Saludos, te debo una! 

[seupcdm]

¡No se debe nada! Suerte con tu proyecto y un saludo. 

[antopixel]

Si necesitas alguna otra ayuda, me puedes mandar mensaje privado.
Pdd: Tambien soy de Colombia y estoy en paro.