Fallo en programa que muestra la frecuencia aparece caras de un dado

[jbgae]

Hola, tengo un problema con un ejercicio que estoy realizando en C++. Les cuento, tengo que escribir un programa que simule 10 000 000 de tiradas de un dado e imprima las frecuencias que se obtienen para cada una de las caras.

Mi propuesta para resolver este ejercicio es:

Código c++:

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
 
//Función que genera un número pseudoaleatorio entero en el intervalo discreto [a, b].Conseguir un entero entre [a,b] es igual a conseguirlo
//entre a + [0, b-a]. Para conseguir enteros entre 0 y b-a recurrimos al resto de la división entera, ya que el resto de dividir cualquier
//número entre b-a+1 nos da valores entre 0 y b-a, a los cuales les sumaremos el valor a obteniendo así enteros del intervalo [a, b].
int Numero_Pseudoaleatorio(int a, int b){
  return a+(rand()%(b-a+1));
}
 
int main()
{
  const int a=1, num_caras = 6;
  const float num_tiradas = 10000000;
  int v[num_caras], num_pseudo = 0;
  
  for(int i=1; i<=num_caras; i++)     // Inicializamos el vector a 0
    v[i]=0;
  
  srand(time(0));   // Usaremos como semilla un valor variable para poder observar distintos resultados en cada ejecución.
  
  for(int i=1; i<=num_tiradas; i++){    //realizamos las 10000000 tiradas del dado
    num_pseudo = Numero_Pseudoaleatorio(a,num_caras);   //almacenamos el resultado de la tirada (valor de la cara del dado)
    v[num_pseudo]++;    //incrementamos las veces que aparece cada valor.
   
  }
  cout << "****muestra el número de veces que aparece cada cara****" << endl;
  cout << "cara   1  -----> " << v[1] << endl;
  cout << "cara   2  -----> " << v[2] << endl;
  cout << "cara   3  -----> " << v[3] << endl;
  cout << "cara   4  -----> " << v[4] << endl;
  cout << "cara   5  -----> " << v[5] << endl;
  cout << "cara   6  -----> " << v[6] << endl;
  
  
  float suma = 0;
  
  for(int i=1; i <=num_caras; i++){  /* EN ESTE BUCLE ESTA EL PROBLEMA, ¿PORQUE V[6] CAMBIA SU VALOR? */
    cout<< "cara 6 -->" << v[6] << endl;  /*HAGO ESTO PARA COMPROBAR QUE V[6] CAMBIA SU VALOR AL ENTRAR EN EL BUCLE Y SE VA INCREMENTANDO*/
    
    cout << "La frecuencia relativa de " << i << " es: " << v[i]/num_tiradas << endl;
    
    suma = suma + v[i]/num_tiradas;
    cout << " frecuencia absoluta = " << suma << endl;
    
  }
  
}
 

pero cuando obtengo los resultados, veo que en el último bucle for cambia el valor de v[6] en cada iteración y realmente no entiendo porqué sucede esto.

Al final la frecuencia absoluta no me sale 1, por lo que se que el problema esta mal.

Espero que puedan ayudarme.
Muchas gracias y un saludo.

[CalgaryCorpus]

Los indices de los arreglos son validos desde 0 hasta el tamaño - 1, no desde 1 al tamaño.

[jbgae]

Muchas gracias CalgaryCorpus por tu contestación, ya lo he solucionado y funciona perfectamente, pero ¿me podrías indicar a que es debido esto? es decir ¿qué diferencia hay entre poner de 1 a tamaño ó poner de 0 a tamaño-1?
Saludos.

[carlos_belisario]

bueno si te fijas los indices de los arreglos van desde el 0 hasta el tamaño -1 entonces si kieres recorrer los valores de un vector debes de hacerlo igual ya que si lo haces comenzando desde 1 hasta el tamaño estarias obviando el primer indice y teniendo una entrada de mas en el ciclo ya que la ultima accion no haria nada al vector espero que me alla explicado bien saludos

[sam90]

Es normal empezar en cero por una cuestion aritmetica. Un array no es unas que un bloque de memoria. La memoria puede ser accedida mediante un numero.

Con los array se tiene el numero inicial del bloque y luego segun la posicion que se desea se suma un offset.
Por ejemplo supongamos que tenemos un array en la posicion 100. Entonces el primer elemento va a estar en 100+ 0 = 100
La segunda posicion va a ser 100+1 = 101...

Como te puedes imaginar para indice i la cuenta seria 100 + i... Por eso en c se empieza desde el cero ya que facilita el estas cuentas!!!

Pero que se entienda!

[Fw190]

En C, los índices para los arrays van siempre desde 0 a N-1, no como otros lenguajes (por ejemplo, PASCAL permite indicar el primero y el último, y el antiguo intérprete BASIC de MS-DOS permitía elegir entre 0 y 1). La razón no era más que simplificar el trabajo del compilador, piensa que en los primeros ordenadores la memoria era un bien muy escaso y muy caro y había que arañar cada byte, tanto de código como de datos.

[jbgae]

Muchas gracias a todos, ya lo he entendido perfectamente.
A continuación dejo la solución ya corregida.

Código C++:

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
 
//Función que genera un número pseudoaleatorio entero en el intervalo discreto [a, b].Conseguir un entero entre [a,b] es igual a conseguirlo
//entre a + [0, b-a]. Para conseguir enteros entre 0 y b-a recurrimos al resto de la división entera, ya que el resto de dividir cualquier
//número entre b-a+1 nos da valores entre 0 y b-a, a los cuales les sumaremos el valor a obteniendo así enteros del intervalo [a, b].
int Numero_Pseudoaleatorio(int a, int b){
  return a+(rand()&#37;(b-a+1));
}
 
int main()
{
  const int a=1, num_caras = 6;
  const float num_tiradas = 10000000;
  int v[num_caras], num_pseudo = 0;
  for(int i=0; i<=num_caras-1; i++)     // Inicializamos el vector a 0
    v[i]=0;
  srand(time(0));   // Usaremos como semilla un valor variable para poder observar distintos resultados en cada ejecución.
  for(int i=0; i<=num_tiradas-1; i++){  //realizamos las 10000000 tiradas del dado
    num_pseudo = Numero_Pseudoaleatorio(a,num_caras);   //almacenamos el resultado de la tirada (valor de la cara del dado)
    v[num_pseudo-1]++;  //incrementamos las veces que aparece cada valor.
  }
  for(int i=0; i<=num_caras-1; i++){ //mostramos las frecuencias relativas.
    cout << "La frecuencia relativa de " << i << " es: " << v[i]/num_tiradas << endl;
  }
}
 

Saludos.