RodrigoToroIcarte / IIC2113-2023-1

6 stars 0 forks source link

Duda acortar clase Player #205

Open SantinoDaloy opened 8 months ago

SantinoDaloy commented 8 months ago

Hola, tengo una duda que me surge de la issue #190 , estoy considerando cambiar mi clase Player a una EDD, y que los métodos pasen a ser clases "Handlers" (ReversalEffectHandler, TurnHandler, CardMovingHandler, etc) ya que mi clase player tiene muchísimos métodos. Me surgen dudas para ver si hago este cambio o no.

Primero, que pasa con los Train wreks, si quiero asegurarme de no tener descuento por player.Hand.TakeCard() (Considerando que no rompe ley de Demeter) debería siempre separar player.Hand en una variable dentro del método o de mi Handler que lo usa?

Y segundo, si es recomendable cambiar a una EDD a estas alturas, ví esta issue del semetre pasado #83 , entonces me hace dudar si debería hacer este cambio o no.

Muchas gracias.

RodrigoToroIcarte commented 8 months ago

¡Hola!

Sobre la primera pregunta, yo me iría a la segura y rompería los train wrecks agregando variables intermedias. De esa forma es imposible que te descuenten puntaje.

Sobre la segunda pregunta, el semestre pasado no tuvimos una campaña tan fuerte sobre los beneficios que tiene hacer que el Player sea una estructura de datos. Por eso en la E3 creo que casi no descontamos por eso. Pero sí lo hicimos en la E4. En este semestre, siento que hay que ser muy porfiado para no hacer que el Player sea una EDD :P