No uses rand
.
Estás programando en C++, sin embargo sigues paradigmas de C y usas utilidades de dicho lenguaje; no uses rand
pues no forma parte de la especificación de C++ y en consecuencia puede no ser portable y puede ofrecer resultados y rendimiento cuestionables. Por ello se está estudiando deprecarlo.
A partir del estándar C++11 el lenguaje C++ ofrece una completa librería de generación de números pseudoaleatorios que permite escoger la distribución de probabilidad (uniforme, Bernoulli, Poisson, normal, discreta, constante, lineal...), el tipo subyacente del valor generado e incluso el algoritmo a usar (minstd, mt19937, ranlux, knuth...).
Estás falseando la distribución.
La distribución numérica de std::rand
es homogénea entre 0
y RAND_MAX
, esto significa que cualquier número dentro de dicho rango tiene las mismas probabilidades de ser esogido (1 entre RAND_MAX
).
Al hacer módulo (%
) sobre el resultado de std::rand
rompes la homogeneidad si el divisor no es múltiplo de RAND_MAX
. Suponiendo un RAND_MAX
de 32767
con un módulo sobre 6
obtenemos que los números del 1
al 5
tienen una probabilidad de aparición menor que el 0
(un 0,003% menor).
Propuesta.
Teniendo en cuenta lo anterior, podrías crear un objeto dado que incluyera una distribución homogénea de valores entre 1 y 6:
template <int MIN, int MAX>
struct Dado
{
int lanzar()
{
/* Generamos un número pseudo-aleatorio con el algoritmo
mt19937 distribuido uniformemente entre MIN y MAX */
return distribucion(generador);
}
private:
// Tenemos control sobre el algoritmo y distribución a usar.
std::random_device device;
std::mt19937 generador{device()};
std::uniform_int_distribution<> distribucion{MIN, MAX};
};
Con ese objeto Dado
puedes crear una función que siga tus premisas:
- Lanzar dos dados de 6 caras.
- Si el valor es igual lanzarlos de nuevo.
// Alias de dado de 6 caras.
using D6 = Dado<1, 6>;
// Nuestros dados.
D6 dado1, dado2;
int tirada()
{
int tirada1, tirada2, avances{};
do
{
avances += (tirada1 = dado1.lanzar()) + (tirada2 = dado2.lanzar());
std::cout << "Primer dado: " << tirada1
<< "\nSegundo dado: " << tirada2
<< "\n\tAvances: " << avances << '\n';
} while (tirada1 == tirada2);
return avances;
}
Puedes ver el código funcionando en Wandbox 三へ( へ՞ਊ ՞)へ ハッハッ.