Yo entendí tu clase Equipos
como un equipo de competidores en un combate o evento, algo así como un equipo de fútbol, un equipo de lucha, un equipo de tenis, etc. Si no es ese el contexto, y Equipo
se refiere a algo así como una consola de juego, es evidente que el enfoque debería cambiar (teniendo en cuenta el comentario de @MrDave1999).
Si la clase es un grupo de competidores, la aproximación acorde con los principios de la POO sería esta.
Antes de presentar las clases, quiero hacer notar que la convención de nombre es importante. Generalmente las clases se llaman en singular, porque representan entidades únicas. Es un concepto que parece banal, pero que ayuda mucho a entender la POO. Es decir, una clase es como un molde para crear objetos de ese tipo. Cuando creas una instancia de Equipo
o de Luchador
no creas varios equipos o varios luchadores, creas un solo equipo o un solo luchador.
El modelo que te propongo es este. Es un modelo mínimo, faltan métodos que luego puedes incorporar, he puesto lo esencial sólo para mostrar cómo funciona.
Clase Equipo
class Equipo
{
private String nombreDios;
private String universo;
private List<Luchador> lstLuchadores;
public Equipo(String n,String u)
{
this.nombreDios=n;
this.universo=u;
}
public String getUniverso()
{
return this.universo;
}
public void setLuchadores(List<Luchador> lstLuchadores){
this.lstLuchadores=lstLuchadores;
}
}
Clase Luchador
Nótese que esta clase no extiende de Equipo
. En la lógica del main
veremos cómo desde la lista que contiene a los luchadores podemos obtener los datos de la clase Equipo
que necesitemos.
class Luchador
{
protected int ptosVida;
protected int ptosResistencia;
protected int ptsAtaqueFisico;
protected String nombre;
public Luchador(String nombre, int ptosVida, int ptosResistencia, int ptsAtaqueFisico)
{
this.nombre=nombre;
this.ptosVida=ptosVida;
this.ptosResistencia=ptosResistencia;
this.ptsAtaqueFisico=ptsAtaqueFisico;
}
public String getNombre ()
{
return this.nombre;
}
public int getPtosVida ()
{
return this.ptosVida;
}
public int getPtosResistencia ()
{
return this.ptosResistencia;
}
public int getPtosAtaqueFisico ()
{
return this.ptsAtaqueFisico;
}
/*
OJO a este método
En tu código lanzabas la suma de puntos en
el escenario de ataque si necesidad
*/
public int getTotalPuntos ()
{
return (this.ptosVida)+(this.ptosResistencia)+(this.ptsAtaqueFisico);
}
/*
Pongo sólo un setter de prueba
*/
public void cambiarPtosVida (int ptosValue)
{
/*
Imaginemos aquí una lógica para verificar
si se le da Game Over por ejemplo
...Solamente restamos, por brevedad
*/
this.ptosVida=this.ptosVida+(ptosValue);
}
public String toString ()
{
return String.format("Nombre: %s %n Puntos: %d <v: %d r: %d a: %d>",
nombre, getTotalPuntos(), ptosVida, ptosResistencia, ptsAtaqueFisico);
}
}
Clase Saiyajin
class Saiyajin extends Luchador
{
/*
Este campo sería una PROPIEDAD que tiene un Saiyajin
y que no tiene la clase padre Luchador
*/
private String propioDeSaiyajin;
private int ptosAtaqueKi;
public Saiyajin(String nombre, int ptosVida, int ptosResistencia, int ptsAtaqueFisico, int ptosAtaqueKi)
{
/*
Llamamos a super y le pasamos los campos
que tiene en común con el hijo
*/
super(nombre, ptosVida, ptosResistencia, ptsAtaqueFisico);
this.ptosAtaqueKi=ptosAtaqueKi;
}
public int getPtosAtaqueKi()
{
return this.ptosAtaqueKi;
}
@Override
public int getTotalPuntos ()
{
return (super.getTotalPuntos())+(this.ptosAtaqueKi);
}
@Override
public void cambiarPtosVida (int ptosValue)
{
/*
Imaginemos aquí una lógica para verificar
si se le da Game Over por ejemplo
...Solamente restamos, por brevedad
*/
ptosVida=ptosVida+(ptosValue);
}
@Override
public String toString ()
{
return String.format("Nombre: %s %n Puntos: %d <v: %d r: %d a: %d k: %d>",nombre,getTotalPuntos(),
ptosVida, ptosResistencia, ptsAtaqueFisico,ptosAtaqueKi);
}
}
Prueba en cualquier contexto
Teniendo el modelo de datos montado de ese modo, vamos a lanzar una prueba. Aquí la clase se llama Rextester
porque el fiddle exige que se llame así. Esta podría ser tu clase principal u otra.
Lo que se hace aquí es:
- Manejar dos objetos de la clase
Equipo
- Manejar dos listas de objetos de la clase
Luchador
. Esas listas se llenarán con objetos de la clase Luchador
o de la clase Saiyajin
según sea necesario
- Se simularán dos combates tomando jugadores aleatorios entre ambas listas. Es interesante pensar aquí en una clase
Combate
que defina todas las reglas sobre los combates, etc. Si esta clase existiera, toda la lógica implementada en los métodos combatOne()
y combatTwo()
iría allí y no en la clase principal. Pero eso ya te corresponde a ti, refinar el modelo, una vez hayas entendido el quid del asunto.
Nada más, todo está más o menos explicado en el código. Obsérvalo y analízalo, para que veas cómo se obtiene la información de cada objeto de forma transparente y sin complicaciones ni código spaguetti.
class Rextester
{
private static List <Luchador> luchadoresEqOne;
private static List <Luchador> luchadoresEqTwo;
private static Equipo eqOne;
private static Equipo eqTwo;
public static void main(String args[])
{
/*Equipo 1*/
eqOne=new Equipo("Equipo1","Universo1");
/*Lista de luchadores*/
luchadoresEqOne=new ArrayList<Luchador>();
/*Asignamos cada luchador*/
luchadoresEqOne.add(new Luchador ("LowMan-E1",1,2,3));
luchadoresEqOne.add(new Luchador ("HitMan-E1",10,20,30));
luchadoresEqOne.add(new Saiyajin ("Chino-E1",10,20,30,40));
/*Mediante el setter de Equipo asignamos la lista de luchadores*/
eqOne.setLuchadores(luchadoresEqOne);
/*Eqipo 2: hacemos lo mismo que con el Equipo 1*/
eqTwo=new Equipo("Equipo2","Universo2");
luchadoresEqTwo=new ArrayList<Luchador>();
luchadoresEqTwo.add(new Luchador ("Batman-E2",20,30,40));
luchadoresEqTwo.add(new Saiyajin ("Nokko-E2",30,20,20,60));
luchadoresEqTwo.add(new Saiyajin ("Gokuuu-E2",1,50,2,90));
eqTwo.setLuchadores(luchadoresEqTwo);
/*
Lanzamos dos combates seguidos de prueba
convendría tener una clase Combate que
para lanzar combartes y establecer las reglas
*/
combatOne();
combatTwo();
}
private static void combatOne()
{
Random randomizer = new Random();
Luchador randomOne = luchadoresEqOne.get(randomizer.nextInt(luchadoresEqOne.size()));
Luchador randomTwo = luchadoresEqTwo.get(randomizer.nextInt(luchadoresEqTwo.size()));
randomizer=null;
/*
En todo este bloque podrás comprobar cómo se obtiene datos
sea de la clase Equipo, sea de la clase Luchador
sea de la clase Sayayyin, según sea el objeto
*/
System.out.format("------------------------------------------------------------%n%n");
System.out.format("COMBATE 1: %s (%s) vs %s (%s) %nAntes del enfrentamiento:%n%n",
randomOne.getNombre(),eqOne.getUniverso(),randomTwo.getNombre(),eqTwo.getUniverso());
/*
Obsérvese la utilidad del método toString
*/
System.out.println(randomOne.toString());
System.out.println(randomTwo.toString());
/*
Un cálculo simple aquí sólo para demostración de los métodos
*/
System.out.format("%nDespués del enfrentamiento:");
if(randomOne.getPtosResistencia() > randomTwo.getPtosAtaqueFisico())
{
System.out.format("%n%s perderá 4 puntos y %s ganará 1 punto%n%n",randomOne.getNombre(),randomTwo.getNombre());
randomOne.cambiarPtosVida(-4);
randomTwo.cambiarPtosVida(1);
}else{
System.out.format("%n%s ganará 3 puntos y %s perderá 5 puntos%n%n",randomOne.getNombre(),randomTwo.getNombre());
randomOne.cambiarPtosVida(3);
randomTwo.cambiarPtosVida(-5);
}
System.out.println(randomOne.toString());
System.out.println(randomTwo.toString());
System.out.format("------------------------------------------------------------%n%n");
}
private static void combatTwo()
{
Random randomizer = new Random();
Luchador randomOne = luchadoresEqOne.get(randomizer.nextInt(luchadoresEqOne.size()));
Luchador randomTwo = luchadoresEqTwo.get(randomizer.nextInt(luchadoresEqTwo.size()));
randomizer=null;
System.out.format("COMBATE 2: %s (%s) vs %s (%s) %nAntes del enfrentamiento:%n%n",
randomOne.getNombre(),eqOne.getUniverso(),randomTwo.getNombre(),eqTwo.getUniverso());
/*
Obsérvese la utilidad del método toString
*/
System.out.println(randomOne.toString());
System.out.println(randomTwo.toString());
/*
Un cálculo simple aquí sólo para demostración de los métodos
*/
System.out.format("%nDespués del enfrentamiento:");
if(randomOne.getPtosResistencia() > randomTwo.getPtosAtaqueFisico())
{
System.out.format("%n%s perderá 2 puntos y %s ganará 2 punto%n%n",randomOne.getNombre(),randomTwo.getNombre());
randomOne.cambiarPtosVida(-2);
randomTwo.cambiarPtosVida(2);
}else{
System.out.format("%n%s ganará 0 puntos y %s perderá 2 puntos%n%n",randomOne.getNombre(),randomTwo.getNombre());
//randomOne.cambiarPtosVida(3); No se modifica porque no ganará ni perderá puntos
randomTwo.cambiarPtosVida(-2);
}
System.out.println(randomOne.toString());
System.out.println(randomTwo.toString());
System.out.format("------------------------------------------------------------%n%n");
}
}
Salida:
La salida de este código sería algo como esto. Dado que es aleatorio, podría variar en cada prueba:
------------------------------------------------------------
COMBATE 1: HitMan-E1 (Universo1) vs Gokuuu-E2 (Universo2)
Antes del enfrentamiento:
Nombre: HitMan-E1
Puntos: 60 <v: 10 r: 20 a: 30>
Nombre: Gokuuu-E2
Puntos: 143 <v: 1 r: 50 a: 2 k: 90>
Después del enfrentamiento:
HitMan-E1 perderá 4 puntos y Gokuuu-E2 ganará 1 punto
Nombre: HitMan-E1
Puntos: 56 <v: 6 r: 20 a: 30>
Nombre: Gokuuu-E2
Puntos: 144 <v: 2 r: 50 a: 2 k: 90>
------------------------------------------------------------
COMBATE 2: LowMan-E1 (Universo1) vs Batman-E2 (Universo2)
Antes del enfrentamiento:
Nombre: LowMan-E1
Puntos: 6 <v: 1 r: 2 a: 3>
Nombre: Batman-E2
Puntos: 90 <v: 20 r: 30 a: 40>
Después del enfrentamiento:
LowMan-E1 ganará 0 puntos y Batman-E2 perderá 2 puntos
Nombre: LowMan-E1
Puntos: 6 <v: 1 r: 2 a: 3>
Nombre: Batman-E2
Puntos: 88 <v: 18 r: 30 a: 40>
------------------------------------------------------------
Demo
VER DEMOSTRACIÓN ONLINE
PD
Este modelo es sólo para que tengas una idea, no es que tenga que ser exactamente así, algunas cosas pueden variar según el contexto. En las clases faltan métodos, he puesto sólo lo básico por motivos de brevedad.
Luchadores
no debería extender deEquipos
, son dos entidades totalmente distintas. Además poner los nombres de las clases en plural no te ayuda a entender lo que pretende la POO. Por poner un ejemplo,Luchador
extendería dePersona
, porque unLuchador
puede ser unaPersona
, pero ¿puede unLuchador
ser unEquipo
? No. Puede ser parte de unEquipo
, pero no es unEquipo
. Entender eso es fundamental, sino estarás perdido totalmente en tu diseño. Creo que ya te comenté algo sobre eso antes.Red
yBlue
. En este caso, tomamosRed
yBlue
como "equipos", no como "colores". Por lo tanto,Red
si puede ser un equipo, siempre y cuando, nos estemos refiriendo a dicho contexto (es decir, al gamemode).