Foros aprenderaprogramar.com
Didáctica y divulgación de la programación

Muchas gracias por la explicación dongo. No te preocupes si salen post más largos de la cuenta se leen y se entienden mejor, creo que tengo algo más claro cuando usar static.

Lo que comentas en la edición es listaEnvio de la clase EnvioProducto, esa trabaja bien. El problema está en la listaEnvios de GestionProductos. Perdón por los nombres practicamente iguales.

Como bien comentas, en EnvioProductos voy creando una lista con productos que se van añadiendo a esa lista. Una vez que es envío se valida. En la clase GestionProductos en la línea 65 si el envío está validado este se añade a listaEnvios.
El problema está en que fijate en la ejecución que puse antes.
Empiezo a crear una lista, le añado productos, la valido y salgo. Aparece:
Envio añadido
En la lista tenemos1 envios
Luego hago el mismo procedimiento, creo un nuevo envío, lo valido y cuando salgo aparece de nuevo
Envio añadido
En la lista tenemos1 enviosEs decir, o no se añade este último elemento a listaEnvios(GestionProductos) o este último elemento sobreescribe al anterior que estaba. Con el debugger veo que lo que hace es la segunda opción se sobreescribe el elemento.

PD: No se si al modificar los post se pierden los archivos adjunto o volví a olvidar adjuntar los ficheros corrigiendo los static

Edito: encontre el gazapo, en la línea 38 estaba definiendo
EnvioDeProductos tmpE;que luego inicializaba el objeto en la línea 63 y este era el objecto que yo modificaba y agregaba a la lista. Pues he tenido que definirlo e inicializarlos a la vez en línea 63, para que al salir del if se destruya el objeto.

Gracias por la ayuda!

Hola, he empezado a mirar el código y lo primero que me llama la atención es la aparición de métodos y atributos static en clases como GestionDeProductos y EnvioDeProductos, lo que no parece normal ¿Por qué estás utilizando static en estas clases?

He mirado lo que indicas de un error pero no sé cómo localizarlo: haz una ejecución del programa y cópiame lo que vas introduciendo hasta que te salta el error, de esa manera podré tratar de hacer la misma ejecución y así ver lo mismo que te ocurre a tí.

Saludos

Los método static los utilizo para invocarlos desde la misma clase, por ejemplo si en la clase GestionProductos desde el método gestionarProductos quiero invocar a el método removeProducto, uso
removeProducto();


Si no pongo static me salta un error, por ese motivo uso los método static. ¿Alguna forma de hacerlo más correcta?

El error no es una excepcion que salte. Es viendolo desde el debugger.
Si entro en enviar y añado un producto, lo valido y regreso al menú, este pedido debería de haberse añadido a listaEnvio. Si lo miramos con el debugger en listaEnvio se encuentra este envio. Pero si hacemos otro envio, agregamos un producto, lo validamos y regresamos al menú inicial. En el depurador se ve que en listaEnvio seguimos teniendo sólo un producto, que es el último que hemos creado y que ha sobreescrito al anterior.

Se puede ver cuando el programa ejecuta esta parte de código, incluido en la clase GestionProductos

public static void presentacion () {
        Interaccion interactuar = new Interaccion ();
        EnvioDeProductos tmpE;
        String producto = null;
       
        while (producto == null) {
            producto = interactuar.responder("¿Con que clase de productos desea trabajar?\n"
                                        + "F - Productos frescos\n"
                                        + "R - Productos refrigerados\n"
                                        + "C - Productos congelados\n"
                                        + "E - Envios\n"
                                        + "EXIT - salir\n"
                                        + "Productos: ", 0);
            interactuar.mostrar("");
            if (producto.toUpperCase().substring(0,1).equals("F")) {
                producto = "productos frescos";
                gestionarProductos(producto, listaFrescos);
            } else if (producto.toUpperCase().substring(0,1).equals("R")) {
                producto = "productos refrigerados";
                gestionarProductos(producto, listaRefrigerados);
            } else if (producto.toUpperCase().substring(0,1).equals("C")) {
                producto = "productos congelados";
                gestionarProductos(producto, listaCongelados);
            } else if (producto.toUpperCase().equals("EXIT")) {
                break;
            } else if (producto.toUpperCase().substring(0,1).equals("E")) {
                String tmpMsg = "Introduce un nombre para el envío";
                tmpE = new EnvioDeProductos(interactuar.responder(tmpMsg), listaFrescos, listaCongelados, listaRefrigerados);
                tmpE.gestionEnvios();
                if (tmpE.getValidacion()) {
                    listaEnvios.add(tmpE.getListaEnvio());
                    interactuar.mostrar("Envio añadido");
                }
                producto = null;
            } else {
                interactuar.mostrar("No ha seleccionado un producto correcto, vuelva a seleccionar\n"); 
                producto = null;
            }
        }
    }

Espero haberme explicado bien.

Hola,

Estoy siguiendo el curso de programación en java pero este es el primer ejercicio que posteo por aquí. El ejercicio no es exactamente como se pide en el enunciado ya que no creo algunas subclases. El motivo de esto es que comencé a hacer el ejercicio y lo fui ampliando por mi cuenta y viendo lo que se iba pidiendo pues se me iban ocurriendo cosas.

Las clases que tengo son:
-Envio de productos
-GestionProductos
-Interaccion
-ListaDeProductos
-Productos
-ProductosFrescos
-ProductosCongelados
-ProductosRefrigerados
-TestProductos

Veréis que tengo un problema para continuar y es que en la clase GestionProductos se crea un ArrayList llamado listaEnvios y cuando me pongo a mirar con el depurador veo que cuando añado un envio a esa lista este borra el anterior que estaba en la posición

• , sobreescribiendo el envío creado anteriormente. Pero la intención es que se vayan creando envíos y añadiendo a la lista para poder a posteriori visualizar todos los envíos realizados.

A ver si alguno puede echarle un vistazo al codigo y ver donde está el error, a parte de alguna que otra sugerencia tanto en el estilo del código como en el programa en sí.

Muchas gracias.
Un saludo