Tengo un ejercicio en el que debo generar 10 cartas de la baraja española de forma aleatoria sin que se repita la misma carta. No puedo generar directamente las 48 cartas y luego sacar 10 por requisitos del problema.
Entonces, tengo que conseguir de forma DIRECTA generar las 10 cartas sin que se repita. La única manera en la que se me ocurre resolver esto es comparar la nueva carta generada con las anteriores que ya se han metido (Esto es el segundo for
) y si hay alguna igual quitarle a la i--
un valor y que pruebe otra vez, y si no está repetida pues que meta en el arrayList
la nueva carta generada.
Como veis, el código parece estar correctamente, pero cuando lo ejecutas varias veces siempre hay alguna carta que se repite, es decir, el mismo numero y palo y esto no tiene que ser así.
import java.util.*;
public class Ejercicio8Baraja {
public static void main(String[] args) {
//Objetos
Scanner teclado = new Scanner(System.in);
ArrayList<Baraja> cartas = new ArrayList<Baraja>(10);
//variables
int palo = 0, numero = 0, n = 0;
String palorandom = "";
numero = (int) Math.floor(Math.random() * (12 - 1 + 1) + 1); // Valor entre M y N, ambos incluidos.
//System.out.print(numero);
palo = (int) Math.floor(Math.random() * (4 - 1 + 1) + 1); // Valor entre M y N, ambos incluidos.
if (palo == 1) {
palorandom = "Oros";
} else if (palo == 2) {
palorandom = "Bastos";
} else if (palo == 3) {
palorandom = "Espadas";
} else {
palorandom = "Copas";
}
cartas.add(new Baraja(numero, palorandom));
for (int i = 0; i < 10; i++) {
System.out.println("Principio primer for");
numero = (int) Math.floor(Math.random() * (12 - 1 + 1) + 1);
palo = (int) Math.floor(Math.random() * (4 - 1 + 1) + 1);
if (palo == 1) {
palorandom = "Oros";
} else if (palo == 2) {
palorandom = "Bastos";
} else if (palo == 3) {
palorandom = "Espadas";
} else {
palorandom = "Copas";
}
for (int j = 0; j < cartas.size(); j++) {
n += 2;
//System.out.println("Principio segundo for"+n);
if (numero == cartas.get(j).getNumero() && palorandom.equals(cartas.get(j).getPalo())) {
i--;
break;
} else {
cartas.add(new Baraja(numero, palorandom));
break;
}
//System.out.println("Final segundo for");
}
} //fin primer for
//mostrar la baraja española
for (Baraja b: cartas) {
System.out.print(b.toString());
}
} //main
} //clase
i
) de unfor
dentro del bucle es muy feo.