cahinostroza
commented
5 days ago La primera solución suena prometedora, pero hay que tener cuidado con las cosas estáticas, hemos tenido problemas al correr los tests con cosas estáticas. Con la segunda debes tener cuidado, porque puede que al agregar ese método, estés transformando RoundInfoDTO en un híbrido, todo depende de cómo sea esa clase.
Otra soluciones que he visto:
- Crear una clase objeto
Roundy esa clase se la pases a tus Efectos y Conditiones, en vez deRoundInfoDTO. - Agregar
RivalaUnit.
Recuerda que no hay una única solución, lo que tienes que tener en cuenta es:
- No tener códido duplicado.
- Tener en cuenta el SRP.
Hola! Tengo una duda con respecto a como manejar este problema. Mis clases SkillEffect y Condition ambas necesitan tener acceso al rival en cierto punto. Entonces, ambas tienen un método GetRival(), que a su vez obtiene el rival desde un RoundInfoDTO. Mis posibles soluciones hasta el momento serían crear una especie de Singleton o agregarle esta lógica a RoundInfoDTO. Sin embargo, no sé si son las adecuadas.