Diagrama de clases - Githubissues

Tulmot / Sklearn-Multilabel

Ensembles de clasificadores Multi-Label en Scikit-Learn

2 stars 0 forks source link

Diagrama de clases #100

Closed Tulmot closed 6 years ago

Tulmot commented 6 years ago

Empezar a completar la parte diseño del anexo.

Tulmot commented 6 years ago

Hecho este diagrama de flujo: imagen

Es uno que he hecho básico falta poner el tipo y alguna cosa más pero para saber si así es la idea

alvarag commented 6 years ago

Supongo que querías decir diagrama de clases. Repasa los apuntes de Carlos de UML, la herencia no está expresada y las agregaciones se encuentran en el lado contrario al que deberían. En cuanto a los tipos, faltarían de definir entiendo porque en todos pones type y no tienen parámetros de entrada

Tulmot commented 6 years ago

Entiendo que así estaría bien el diagrama de clases.

untitled diagram

alvarag commented 6 years ago

Hay cosas que siguen sin gustarme:

El constructor debería recibir argumentos, sino quítalos.
HomogeneusEnsemble es abstracta la clase.
Lo que hay en HomogeneusEnsemble yo no lo repetiría en las subclases
Además añadiría las dependencias que tienes de clases de Sklearn: tus clases heredan de ellas.

Si ves que se complica el diagrama al poner las clases de Sklearn de las que heredan, puedes hacer este diagrama por un lado y luego uno general (sin métodos) con el diagrama general.

Tulmot commented 6 years ago

El constructor lo he quitado porque tenías muchos argumentos y sino las clases me quedaban muy anchas. Por lo que he visto de las clases abstractas se heredan los métodos, por lo que de las clases he quitado los atributos y he heredado los métodos de la clase abstracta.

classdiagram

Las dependencias de las clases de Sklearn como has dicho haré un diagrama a parte general, ya que veo que se entenderá mejor con dos diagramas que uno con todo.