Hola hace varios días que llevo atorado en este problema. He revisado el programa de principio a fin varias veces y no encuentro el error. Eclipse arroja un "index -1 out of bounds" y adedmás aparecen dos JLabels que se superponen (cuando no deberían) . La idea es simular un campeonato entero de fútbol. Necesito que el programa me muestre aleatoriamente qué equipo juega contra cuál, y a su vez, que cada véz que aprete el botón "jugar", cambien los adversarios para cada equipo, teniendo en cuenta que no puden volver a jugar contra los que ya jugaron antes. Para eso implementé dos vectores: uno que agrupa a todos los equipos, y otro vector interno de cada equipo en el que entran los clubes con los que ya se ha jugado. Les dejo el código:
NOTA1: El error surge de manera bastante aleatoria. El programa en un principio carga bien y luego empiza a tirar errores cuando presiono "jugar" una cantidad aleatoria de veces (Es decir, a veces tira el error la tercera vez que presiono, otras veces la quinta, etc) Y en ese momento no solo ocurre un problema en la consola sino que se superponen las etiquetas con los nombres de los equipos (ver foto)
NOTA2 2: Según Eclipse el error se da en el bucle WHILE que está dentro del método Fixture(). "Index -1 out of bounds". Yo ya sé que esto significa que el bucle está fallando e intentando mirar un objeto en la posición "-1" del vector (que obviamente no existe). Lo que no entiendo es por que sucede esto ya que en la lógica que pensé, el bucle debería cortarse mucho antes de llegar a -1. Tampoco entiendo la superposición de etiquetas.
dejo el link de la imagen con la superposicion de etiquetas (no me deja subirla al post no se por que) https://ibb.co/Jmh8vQg
package Pruebass;
import javax.swing.* ;
import java.awt.event.*;
public class PcFutbol extends JFrame implements ActionListener {
static int fecha = 1;
private JLabel proxpartido,posiciones,resultados,proxpartidoa;
Equipo fechalista[] = new Equipo [10];
int cargador = 0;
int medidab = 80;
JButton boton1;
Equipo[] equipos = { new Equipo("independiente"), new Equipo("SanLorenzo"), new Equipo("lanus"), new Equipo("boca"),
new Equipo("river"), new Equipo("racing"), new Equipo("banfield"), new Equipo("platense"),
new Equipo("Newells"), new Equipo("central") };
public PcFutbol() {
setLayout(null);
boton1=new JButton("Jugar");
boton1.setBounds(300,250,100,30);
add(boton1);
boton1.addActionListener(this);
fixture();
// mostrarResultado();
// ordenarTabla();
}
class Equipo {
String nombre;
Equipo[] partidosjugados = new Equipo[9];
boolean tienepartido = false; // ESTE BOOLEAN sirve para que el método que recorre el vector para encontrar rival detecte si un equipo ya fue seleccionado para jugar esa fecha o no.
int comojugó;
JLabel resultado = new JLabel("-");
JLabel puntosb = new JLabel("-");
JLabel nombres = new JLabel(nombre);
JLabel nombresb = new JLabel(nombre + " - ");
public Equipo(String a) {
puntos = 0;
calidad = (int) Math.floor(Math.random() * 30);
nombre = a;
nombres.setText(nombre);
}
public String toString(int a) {
if (a == 1) {
return nombre ;
}
else {
String puntox = String.valueOf(puntos);
return puntox;
}
}
}
public void actionPerformed(ActionEvent e) {
if (e.getSource()==boton1) {
// competir();
resetearDatos();
fecha++;
fixture();
// ordenarTabla();
}
}
public void fixture() {
int selecrival = 0;
for (int q = 0; q < 9; q++) {
if (equipos[q].tienepartido == true) {
continue;
}
selecrival = (int) Math.round(Math.random() * (9 - 1) + 1);
while (equipos[q] == equipos[selecrival]) {
selecrival = (int) Math.round(Math.random() * (9 - 1) + 1);
}
if (equipos[selecrival].tienepartido == true || yajugo(equipos[q], equipos[selecrival]) == true) {
int m = 9;
// EL BUCLE DE ABAJO ES EL QUE TIRA EL ERROR INDEX -1 OUT OF BOUNDS SEGÚN ECLIPSE
while (equipos[q] == equipos[m] || equipos[m].tienepartido == true || yajugo(equipos[q], equipos[m]) == true) {
m--;
}
fechaArmada(equipos[q], equipos[m]);
continue;
}
if (equipos[selecrival].tienepartido == false && yajugo(equipos[q], equipos[selecrival]) == false) {
fechaArmada(equipos[q], equipos[selecrival]);
}
}
}
public void fechaArmada(Equipo a , Equipo b) {
fechalista[cargador] = a;
fechalista[cargador+1] = b;
cargador = cargador+2;
a.partidosjugados[(fecha - 1)] = b;
b.partidosjugados[(fecha - 1)] = a;
a.tienepartido = true;
b.tienepartido = true;
a.nombresb.setText(a.nombre + " - ");
a.nombresb.setBounds(10, medidab, 300, 30);
b.nombresb.setText(b.nombre);
b.nombresb.setBounds(150,medidab,300,30);
add(a.nombresb);
add(b.nombresb);
medidab = medidab + 15;
}
public boolean yajugo(Equipo a, Equipo b) {
for (int i = 0; i < a.partidosjugados.length; i++) {
if (a.partidosjugados[i] == b) {
return true;
}
}
return false;
}
public void resetearDatos() {
for (int a = 0 ; a < equipos.length ; a++) {
equipos[a].tienepartido = false ;
}
cargador = 0;
medidab = 80;
}
public static void main(String[] args) {
// TODO Auto-generated method stub
PcFutbol uno = new PcFutbol();
uno.setBounds(10,20,400,300);
uno.setVisible(true);
uno.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
}
}
```
[1]: https://i.sstatic.net/HxX7F.png