FedericoFuentesWeber / TPs-PdeP-2018

0 stars 0 forks source link

Implementacion espejo #32

Closed MarianoSalerno closed 6 years ago

MarianoSalerno commented 6 years ago

El metodo soloMeContieneAMi esta violando el encapsulamiento, dado que sabe que su duenio (en este caso rolando) tiene una LISTA de artefactos, ese mensaje deberian delegarlo en el duenio. Mas alla de eso la implementacion esta mal porque nunca se fijan que el unico elemento que existe en la lista es el espejo, puede ser cualquier otro.

Lo mismo para el metodo maximoPoder, deben delegarlo al duenio tambien ya que el conoce sus artefactos entonces sabe cual es el que tiene mas poder.

Y esto es de rompebolas pero el if de poderDeLucha esta mal indentado