emanuelbas / practica-oo2-design-patterns Goto Github PK

1. Sobre el diagrama UML:

i. faltan los tipos de los atributos de las clases Autor, Album y TemaSpootify
ii. falta la cardinalidad en las asociaciones.
iii. la navegabilidad de las asociaciones es correcta pero no así su notación. Se debe utilizar la punta de flecha no rellena.
iv. faltan los nombres de los parámetros en las responsabilidades (están sus tipos, pero no sus nombres).

2. Sobre la implementación:

i. No se implementó la búsqueda case insensitive, tal como pedía el enunciado.

ii. Hay métodos, por ejemplo: #buscarPorTitulo: de la clase AutorSpootify, que deberían haber usado mayor nivel de abstracción como por ejemplo #flatCollect: en lugar de agregar a mano en una colección de colecciones.

3. Sobre los tests:
   i. Los escenarios de prueba son pobres. Debería haber un test para cada escenario a probar.

ii. No hay ningún test que controle que las 3 búsquedas sean case insensitive.

iii. No hay ningún test que controle que la búsqueda funcione para cuando el resultado es vacío.

iv. No hay tests para probar el agregado temas repetidos a la música favorita de un usuario.

v. No hay tests para probar la eliminación de una canción inexistente en la música favorita de un usuario.

1. El diagrama UML está OK salvo que está faltando la clase cliente del patrón Composite, es decir el File System que debe referenciar a una instancia de Directorio para representar el directorio raíz.

2. En el método de creación de Archivo, #llamado:creadoEl:kBytes: hacés "self new", sin embargo en el método de creación equivalente de la clase Directorio hacés "Directorio new". ¿Por qué esta diferencia?

En la implementación, tené en cuenta que si los decoradores no proveen los setters de los atributos tal como los files concretos los proveen, se desvirtúa el patrón, ya que los clientes externos no podrían utilizar un objeto concreto o un decorador indistintamente. En concreto, dado que agregaste setters para los atributos del file concreto, deberías haber agregado los correspondientes en los decoradores (similar a como resolviste los getters).