search

Christian-VH-99 / our-zork

Proyecto TCP de Programación Avanzada - UNLAM
0 stars 5 forks source link

Cohesión en los métodos #13

Closed delucas closed 4 years ago

delucas commented 4 years ago

https://github.com/Christian-VH-99/our-zork/blob/39c5c11a62f4e02cae1aacc2e45895eee29af003/Zork/src/Ubicacion/Place.java#L91-L98

Cuando un método tiene una conjunción en el nombre, suele ser indicador de que está haciendo dos cosas a la vez. ¿Podría desagregarse dicha responsabilidad en dos métodos? ¿Podría renombrarse dicho método? Revisen el codebase por nuevas ocurrencias de este tipo de problemas menores, que terminan enchastrando el código :)

facuegon481 commented 4 years ago

Se renombró el método a un nombre mas significativo.

delucas commented 4 years ago

:+1: