search

X-Force3 / Zork

Trabajo Práctico del Taller: Zork
0 stars 0 forks source link

Constructores grandes #4

Open delucas opened 4 years ago

delucas commented 4 years ago

https://github.com/X-Force3/Zork/blob/449a86471b9724ba5696fef8a83a2c1fccca1828/src/entidades/Npc.java#L15-L25

Cuando un constructor tiene demasiados parámetros suele ser un indicador de que estamos creando entidades con demasiados atributos. Considerar si puede extraerse una parte a ser configurada luego de construirse, o si puede dividirse en más entidades. NPC es el ejemplo, pero les pasa con varias entidades.

Corcione-Luciano commented 4 years ago

Seria una solución que el constructor reciba solo nombre, genero y numero. Y que después los demás atributos sean cargados en otros métodos?

delucas commented 4 years ago

Evaluen cómo lo podrían reducir.

jialonso1993 commented 4 years ago

Se modificaron los constructores de las clases Lugar, Item, Npc y Ubicacion. Ahora en ningún caso los constructores reciben como parámetro más de cuatro atributos del objeto. Los atributos que son listas de String o de objetos de otra clase, se asignan a través de setters sólo cuando es necesario utilizarlos. Un saludo, profe @delucas ...