Posible solución para las clases PosicionAtaque y PosicionDefensa

[romeromaxi]

Pensé en ésta solución para las clases de posición en ataque y en defensa.

• Lo malo: esos if anidados en ambas, no se me ocurrió otra forma de comparar si no es con los if, y también la cantidad de parámetros que utiliza.
• Lo bueno: que la pregunta de quién ganó en el enfrentamiento se hace solo una vez. Y Posicion se encargaría de mandar al cementerio a las cartas monstruo correspondientes, y de disminuir o no la vida de los jugadores.

No se que opinan ustedes? Por las dudas no hice commit para no perder lo anterior

[ghost]

Con respecto a los if fijate que lo que preguntas en posición defensa es lo mismo que haces en posición ataque ( posición ataque solo le agrega el caso de una diferencia negativa, habría que ver de alguna manera de tener solo una vez esos if). Veo que posición recibe demasiados parámetros, hay que ver como podríamos reducirlos. Podemos hacer que en vez de enviar un mensaje al monstruo de ir al cementerio, podemos enviar al jugador un mensaje de que el monstruo con un "nombreMonstruos" se murió y debe ir al cementerio. En ese método mandamos a la carta correspondiente al cementerio y eliminamos de la lista. De esta manera matamos al monstruo y mantenemos la colección de cartas monstruos actualizadas.

[manudelca]

Para sacar esos ifs podríamos crear una clase que represente a la diferencia del enfrentamiento. Este objeto guardaria el resultado de las batallas entre los monstruos y, en base a eso, haría cambios a los estados de todos los objetos afectados (los jugadores por ejemplo). Así, tal vez, queden más subdividos esos ifs en métodos de esta clase o quizás termine complicando más el código, pero es lo único que me viene a la cabeza ahora.