NahuelMGithub / Estructura-de-Datos

TP's Estructura de Datos Nahuel López
0 stars 0 forks source link

Pr2: la función `esSenior` tiene demasiadas responsabilidades #6

Closed fidel-ml closed 2 years ago

fidel-ml commented 2 years ago

Al hacer doble PM la estás recargando. Si estás mirando el Rol, no mirás sus componentes, sino que delegás en otras funciones auxiliares que corresponda...

NahuelMGithub commented 2 years ago

Listo.

fidel-ml commented 2 years ago

Mejora. Pero si en seniorYParticipa vas a usar una subtarea, ¿por qué hacer análisis de casos? LAS SUBTAREAS se encargan de los subcasos; solamente tenés que hacer eso si usás diferentes subtareas en los diferentes casos, pero no es lo que pasa acá...

NahuelMGithub commented 2 years ago

Tal vez no estoy entendiendo, peor el analisis de casos se refire a que seniorYParticipa pregunta por lista [] y luego por (p:ps) Eso es lo que está mal? o que en seniorYParticipa r (p:ps) = ..... use dos subtareas ?

fidel-ml commented 2 years ago

Claro. ¿Para qué querés mirar si la lista es vacía o (:), si después usás subtareas? Fijate además que no estás mirando nunca el primer proyecto, porque lo descartaste al hacer un PM innecesario.

NahuelMGithub commented 2 years ago

Listo., era simplemente sacar el primer caso (el de lista vacía, porque mi subtarea perteneceProyecto ya mira eso)