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

Hola, ahora el diseño sí se ve mucho mejor armado y los nombres de clase mucho más coherentes. Creo que el ejercicio lo puedes dar por resuelto, el programa funciona perfectamente. De todas formas te voy a comentar todo lo que he visto.

La clase Producto la veo correcta, también ProductoFresco, ProductoRefrigeradoCongelado, ProductoRefrigerado, ProductoCongelado, ProductoCongeladoPorAgua, ProductoCongeladoPorNitrogeno, GestionadorEntradasTeclado, VisualizadorMenues, PreparadorDeProductos.

En la clase ListaDeProducto usas

        for (Producto tmp:miLista){
            System.out.println("Producto Nº: "+posicion);
            System.out.println(tmp.mostrarProducto());
            posicion++;
        }

Normalmente si necesitamos un índice (como posicion) usamos un for normal, el for extendido es más habitual cuando no necesitamos índice o cuando no sabemos el número de elementos en una colección. De todas formas se puede hacer como lo has hecho.

En ProductoCongeladoPorAire has utilizado como atributo un array  private int[] composicionDelAire; considerando que los distintos campos se corresponden con un componente:

nitrogeno = composicionDelAire[0];
oxigeno = composicionDelAire[1];
co2 = composicionDelAire[2];

Si ves otros ejercicios resueltos otros compañeros suelen definir tres int en lugar de un array, pero como tú lo has hecho también está bien y sirve para ver distintas formas de trabajar, es bueno saber aplicar distintos recursos.

En la clase EnviosADomicilio se ve algo extraño. Los atributos son:


    ListaDeProducto listaProductosPedidos;
    ArrayList<ListaDeProducto> miListaDeEnvios;


El nombre de la clase parece dar a entender que la clase representa un envío a domicilio, aunque en la descripción pone

 * clase EnviosADomicilio
 * Esta clase sirve para crear y manipular los pedidos de envío a domicilio realizados por los clientes.

Creo que el diseño mejoraría si por un lado tienes una clase Envio que represente un envío, y por otro lado una clase ListaDeEnvios si consideras que te es necesaria, y por otro lado una clase GestorDeEnvios si consideras que es necesaria. Parece que en esta clase has mezclado varias cosas, y cada clase debe representar algo concreto y tener unas tareas concretas. Es algo interesante tener en cuenta para futuros programas.

Saludos!

Hola, ahora sí he visto que muestra los productos frescos. Los nombres de clases ya podemos decir también que son ahora correctos.

creo una nueva instancia y le quito el static a esta clase, pero podrías explicarme porque es mejor crear nueva instancia que usar sus métodos en static como una clase abstracta?

Lo primero es aclarar que el concepto de clase abstracta en Java se refiere a una cosa muy concreta que se explica en la entrega CU00695B del curso y no tiene que ver con static, por tanto no debemos emplear el término abstracto excepto para referirnos a lo que en Java se entiende por abstracto (esto es un pequeño lío, pero es la terminología de java).

El motivo por el que es mejor crear una nueva instancia es para tratar de trabajar como programadores bajo el paradigma o filosofía de la orientación a objetos. Según esta filosofía los programas se conforman mediante la interacción entre objetos. Si defines una clase donde todos sus elementos son estáticos no necesitas crear un objeto para acceder a ella, te estás saliendo de la filosofía de la orientación a objetos. Todo esto tiene una razón de ser, aunque al principio quizás no es fácil entender todos los conceptos y toda la filosofía que encierra este tipo de programación. Un motivo que se suele citar es el de introducir seguridad en el código: dar acceso libre desde cualquier lugar (declarando todo static) del código es más inseguro que forzar a que se cree un objeto, lo cual introduce un mayor grado de control. Es cierto que crear objetos no nos hace buenos programadores, pero entra dentro de lo que se consideran prácticas adecuadas en programación orientada a objetos.

Todo tiene sus excepciones, hay casos en los que trabajaremos con objetos únicos o con clases static, pero no debemos verlo como una práctica habitual, sino algo a usar cuando justificadamente se considere.

Visto el código corregido, lo único que veo faltaría es eliminar la clase PCAireAguaHidro y unificarla dentro de la clase PCongelado.

Saludos!

Hola arcanFAC, con las correcciones introducidas creo que has redondeado el ejercicio, gracias por subirlo pues puede servir de modelo para otras personas que estén realizando el curso. Saludos!

Hola arcanFAC, en primer lugar felicitarte porque este es un ejercicio un poco extenso y se ve que lo has trabajado, y bien trabajado. Tanto el diseño (estructura) como el código en general podemos decir que están bien planteados. En algunos aspectos no has seguido exactamente lo que pedía el enunciado. Voy a comentarlo por partes.

Dentro de la clase Producto has incluido como atributos private String fechaEnvasado; y private String paisOrigen; Aunque el enunciado no lo dice explícitamente, si se lee con detenimiento son atributos de todos los productos, por tanto has hecho bien llevando esos atributos a un nivel superior, evitando la repetición en las diferentes subclases.

También has hecho bien creando la clase ProductoRefCong como abstracción que engloba todo lo común de ProductoRefrigerado y Productocongelado. Esto se corresponde con el apartado del enunciado donde dice

Crear superclases intermedias (aunque no se correspondan con la descripción dada de la empresa) para agrupar atributos y métodos cuando sea posible. Esto corresponde a “realizar abstracciones” en el ámbito de la programación, que pueden o no corresponderse con el mundo real.

En la clase CongeladoPorAire has definido estos atributos:

public class CongeladoPorAire extends ProductoCongelado{
   private String nitrogeno;
   private String oxigeno;
   private String dioxidoCarbono;
   private String vaporAgua;

Dado que los datos de composición son un %, es decir, un valor numérico, hubiera sido más adecuado declarar estos atributos como valores numéricos (por ejemplo double):


public class CongeladoPorAire extends ProductoCongelado{
   private double nitrogeno;
   private double oxigeno;
   private double dioxidoCarbono;
   private double vaporAgua;

Tener en cuenta que los String son para almacenar información textual. Si se trata de valores numéricos debemos usar un tipo numérico.

Lo mismo ocurre en las clases CongeladoPorAgua y CongeladoPorNitrogeno: deberías haber declarado los atributos como de tipo numérico en lugar de String.

Por lo que comentas esto ya lo conocías, pero si lo conocías lo lógico hubiera sido resolverlo usando los tipos numéricos

Has definido una clase de nombre Intermedio. No me parece mal. Lo único que te recomendaría es que pongas un nombre a la clase que describa lo que hace. Intermedio no describe lo que hace la clase. El nombre debería ser GestorProductos ó GestorListasProductos u otro descriptivo. Dentro de esta clase el método public void agregarProducto(String caracteristica) parece una muy buena idea para evitar la repetición de código, creo que es un buen ejemplo para que pueda ser consultado por personas que revisen ejercicios en los foros.

Te has saltado esta parte del enunciado:

Crear una clase testHerencia3 con el método main donde se creen: dos productos frescos, tres productos refrigerados y cinco productos congelados (2 de ellos congelados por agua, otros 2 por agua y 1 por nitrógeno). Mostrar la información de cada producto por pantalla.

Pero a cambio has incluido un menú y opciones que demuestran que dominas perfectamente lo que se pretendía con el ejercicio.

Aunque con algunos detalles que se podrían corregir, me ha parecido muy buen ejercicio. Mi enhorabuena.

Saludos.

Hola Dan, todo correcto.

La declaración LinkedList <Boolean> listaBoleana = new LinkedList<>(); también puede hacerse así: List <Boolean> listaBoleana = new LinkedList<>(); pero esto es una cuestión que se estudia más adelante en el curso y no hay que adelantar acontecimientos. Adelante.

Saludos

Hola Dan todo bien aunque prestando atención comprobamos que nos dice que el número de intentos es uno menos de los realmente realizados. La corrección es sencilla:

       else if(num == aleatorio)
            {
                intentos++;
                System.out.println("Has acertado!!! el numero es: " + num + " (intentos " + intentos +")");
            }
Saludos

Hola en primer lugar felicitarte por haber resuelto el ejercicio, que es un tanto complejo. Si tienes interés en mejorar las cosas mejorables son:

- Hay repetición de código en tu solución. La repetición de código debe evitarse. Esto es una cuestión de diseño del código, no es fácil hacer buenos diseños. Normalmente las personas que saben hacer buenos diseños es porque han practicado mucho y consultado diseños que hacen otras personas y con el tiempo han ido aprendiendo a realizar buenos diseños. Te recomiendo que consultes esta solución propuesta https://www.aprenderaprogramar.com/foros/index.php?topic=2089 y la compares con la tuya. Seguidamente deberías "reformar" tu código por tí mismo tratando de hacerlo más compacto y evitar la repetición.

- Sería aconsejable que para practicar te acostumbres a ir trabajando con programas donde intervenga más de una clase. Por ejemplo aquí podrías haber usado una clase como EntradaDeTeclado.

Saludos

Hola Dan la respuesta al ejercicio es correcta. Para mejorar te propongo que compares tu solución con esta: https://www.aprenderaprogramar.com/foros/index.php?topic=2108.0

Si te fijas en tu código hay cierta "repetición". En la medida en que puedas hacer el código más compacto y con menos repetición mejor programador serás, aunque no hay que obsesionarse por optimizar y compactar es bueno por lo menos ser consciente de ello. En la programación profesional se encuentran con frecuencia códigos muy poco optimizados...

Saludos.

Hola Lorenzo, el ejercicio está bien resuelto, de hecho has hecho más de lo que se pedía, ya que tan sólo se pedía documentar una clase y has documentado tres y todo bien, así que perfecto. Saludos.

Hola Lorenzo, esta solución responde perfectamente a lo que pedía el ejercicio:

import java.util.Arrays;
import java.util.LinkedList;

public class BooleanosArray55{   
    public static void main (String[] args){
        Boolean [] arrayBooleanos = {true, true, false, false, true, false};
        LinkedList <Boolean> listaUnida = new LinkedList <Boolean>(Arrays.asList(arrayBooleanos));
        System.out.println(listaUnida);
    }
}

La transformación a String es algo que no pedía el ejercicio, ni es algo que se suela hacer habitualmente (no sé bien si lo has querido hacer para probar o si interpretaste que debías hacerlo).

Hay algo interesante: Java nos permite transformar un array de objetos (por ejemplo Boolean o String) en una lista usando asList. Pero no nos permite transformar un array de tipos primitivos en una lista usando asList.


Puede ocurrir que necesitemos transformar un array de tipo primitivo boolean a algún tipo de lista como LinkedList ó ArrayList de Boolean (es decir, pasar de un array de tipos primitivos a una colección de objetos).

Aunque no lo pedía el ejercicio, para quien tenga interés esto podría hacerse así:

import java.util.Arrays;
import java.util.LinkedList;

public class BooleanosArray9{   
    public static void main (String[] args){
        boolean [] arrayBooleanos = {true, true, false, false, true, false};
        LinkedList<Boolean> boolList = new LinkedList<Boolean>();
    for (int index = 0; index < arrayBooleanos.length; index++)   {             
                                       boolList.add(arrayBooleanos[index]);    }   
        System.out.println("boolList: "+boolList);
    }
}

Saludos