René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Some code corrections to avoid violations on SOLID principles","articleBody":"Hola, \r\n\r\nHe visto la mayoría de tus proyectos y los encuentro muy buenos para enseñar. Sin embargo, he notado algunas violaciones de los principios de diseño SOLID. Aquí dejo algunas correcciones de cómo los diferentes proyectos podrían verse, resolviendo estas violaciones, para que puedan enseñarse o ponerse en práctica de una mejor manera.\r\n\r\n1. ISP y LSP en las clases bird y pengin, tambien en la interface Fly\r\n\r\n`public interface Flyer { \r\n    void fly(); \r\n}  \r\n\r\npublic interface Swimmer { \r\n    void swim(); \r\n} \r\n\r\npublic interface Runner { \r\n    double speed(); \r\n} \r\n\r\n// Calse abstracta Bird \r\n\r\nprivate abstract class Bird { \r\n\r\n    private String name; \r\n    private boolean hasFeather; \r\n\r\n    protected Bird(String name) { \r\n        this.name = name; \r\n        this.hasFeather = true; \r\n    } \r\n\r\n    protected Bird(String name, boolean hasFeather) { \r\n        this.name = name; \r\n        this.hasFeather = hasFeather; \r\n    } \r\n\r\n    public boolean isHasFeather() { \r\n        return hasFeather; \r\n    } \r\n\r\n    public String getName() { \r\n        return name; \r\n    } \r\n    public abstract void spawnEgg(); \r\n\r\n    @Override \r\n    public String toString() { \r\n        return \"Bird{\" + \r\n                \"name='\" + name + '\\'' + \r\n                \", hasFeather=\" + hasFeather + \r\n                '}'; \r\n    } \r\n} \r\n\r\n// Clase Penguin que implementa las interfaces necesarias \r\n\r\nclass Penguin extends Bird implements Swimmer, Runner { \r\n    public Penguin(String name) { \r\n        super(name); \r\n    } \r\n\r\n    public Penguin(String name, boolean hasFeather) { \r\n        super(name, hasFeather); \r\n    } \r\n\r\n    @Override \r\n    public double speed() { \r\n        return 10.0; \r\n    } \r\n\r\n    @Override \r\n    public void swim() { \r\n        System.out.println(\"yo puedo nadar.\"); \r\n    }  \r\n\r\n    // Método específico de Penguin \r\n\r\n    public void spawnEgg() { \r\n        System.out.println(\"Pinguino poniendo un huevo.\"); \r\n    } \r\n} `\r\n\r\n![image](https://github.com/flyworker/java-workshop/assets/150173288/808e6278-dd32-4399-bed2-6bf7f0b8962e)\r\n\r\n2. DIP en las classes ImplementacionNube \r\n\r\n`// Interfaz para servicios de nube \r\n\r\npublic interface ServicioNube { \r\n    boolean iniciarSesion(String usuario, String contrasena); \r\n    void subirArchivo(File archivo); \r\n    String descargarArchivo(String nombreArchivo); \r\n} \r\n\r\n// Implementación de la interfaz ServicioNube \r\n\r\npublic class ImplementacionNube implements ServicioNube { \r\n    // Constructor y métodos \r\n\r\n} \r\n\r\n// Cliente de nube que depende de la abstracción (interfaz) \r\n\r\npublic class ClienteNube { \r\n    private ServicioNube servicioNube; \r\n    \r\n    public ClienteNube(ServicioNube servicioNube) { \r\n        this.servicioNube = servicioNube; \r\n    } \r\n\r\n    public boolean iniciarSesion(String usuario, String contrasena) { \r\n        return servicioNube.iniciarSesion(usuario, contrasena); \r\n    } \r\n\r\n    public void subirArchivo(File archivo) { \r\n        servicioNube.subirArchivo(archivo); \r\n    } \r\n\r\n    public String descargarArchivo(String nombreArchivo) { \r\n        return servicioNube.descargarArchivo(nombreArchivo); \r\n    } \r\n\r\n} `\r\n\r\n![image](https://github.com/flyworker/java-workshop/assets/150173288/ff8a4038-a161-42ed-9cdd-8e0ba83846a6)\r\n\r\n3. SRP en la clase conexión nube\r\n\r\n`// Clase para manejar la conexión a la nube \r\n\r\npublic class ConexionNube { \r\n    private String usuario; \r\n    private String contrasena; \r\n\r\n    public ConexionNube(String usuario, String contrasena) { \r\n        this.usuario = usuario; \r\n        this.contrasena = contrasena; \r\n    } \r\n    // Métodos para la conexión \r\n} \r\n\r\n // Clase para operaciones específicas de la nube \r\n\r\npublic class OperacionesNube { \r\n    private ConexionNube conexionNube; \r\n\r\n    public OperacionesNube(ConexionNube conexionNube) { \r\n        this.conexionNube = conexionNube; \r\n    } \r\n\r\n    public void subirArchivo(String archivo) { \r\n        // Lógica para subir el archivo \r\n    } \r\n\r\n    public String descargarArchivo(String archivo) { \r\n        // Lógica para descargar el archivo \r\n    } \r\n\r\n    public String descargaRapida(String archivo) { \r\n        // Lógica del método \r\n    } \r\n\r\n} `\r\n\r\n![image](https://github.com/flyworker/java-workshop/assets/150173288/ef027408-bf79-4356-8312-ff321cf03653)\r\n\r\n4. OCP en las clases EstrategiaNubeAWS, ServicioNube y EstrategiaNubeAzure\r\n\r\n`// Interfaz para la estrategia de nube \r\n\r\ninterface EstrategiaNube { \r\n    void subir(String archivo); \r\n    String descargar(String archivo); \r\n\r\n} \r\n\r\n// Implementaciones de nube \r\n\r\nclass EstrategiaNubeAWS implements EstrategiaNube { \r\n    public void subir(String archivo) { \r\n        // Implementación específica de AWS  \r\n    } \r\n\r\n    public String descargar(String archivo) { \r\n        // Implementación de AWS para decargar \r\n    }\r\n\r\n} \r\n\r\nclass EstrategiaNubeAzure implements EstrategiaNube { \r\n    public void subir(String archivo) { \r\n        // Implementación para subir \r\n    } \r\n\r\n    public String descargar(String archivo) { \r\n        // Implementación para descargar \r\n    } \r\n\r\n} \r\n\r\n// Clase de servicio de Nube \r\nclass ServicioNube { \r\n    private EstrategiaNube estrategiaNube; \r\n\r\n    public ServicioNube(EstrategiaNube estrategiaNube) { \r\n        this.estrategiaNube = estrategiaNube; \r\n    } \r\n\r\n    public void subirArchivo(String archivo) { \r\n        estrategiaNube.subir(archivo); \r\n    } \r\n\r\n    public String descargarArchivo(String archivo) { \r\n        return estrategiaNube.descargar(archivo); \r\n    } \r\n\r\n} `\r\n\r\n![image](https://github.com/flyworker/java-workshop/assets/150173288/b278bfe6-0dfc-4fe0-89f7-b96649bdfcde)\r\n","author":{"url":"https://github.com/LuisVergaraA","@type":"Person","name":"LuisVergaraA"},"datePublished":"2024-06-21T02:31:30.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/2/java-workshop/issues/2"}