2016-11-27 2 views
0

Я пытаюсь сделать этот код, который катит две кости и отслеживает сумму в векторе. Я придумал следующий код, но я продолжаю получать ошибку диапазона, если я введу число выше 12. Может кто-нибудь, пожалуйста, скажите мне, что я делаю неправильно?Simuate Rolling Of Two Dice и Save Sum In Vector

#include "std_lib_facilities.h" 

void simulate(vector<int>&, size_t); 
void size_int(const vector<int>, size_t); 
void print_vector(const vector<int>&, size_t); 

int main() 
{ 
    size_t n; 
    cout << "Enter the amount of rolls: "; 
    cin >> n; 
    vector<int> v(12); 
    for(int i=0;i<v.size();i++) 
    { 
     v[i] = 0; 
    } 
    simulate(v, n); 
    print_vector(v, n); 
} 

void simulate(vector<int>& sum, size_t n) 
{ 
    int dice_1, dice_2; 
    cout << "Rolling the dice " << n << " times produces:\n"; 
    srand(time(NULL)); 
    size_t i; 

for (i = 0; i < n; ++i){ 
    dice_1 = 1+(rand() % 6); 
    dice_2 = 1+(rand() % 6); 

    sum[dice_1+dice_2]++; 
    } 
} 

void print_vector(const vector<int>& v, size_t n) 
{ 
    size_t i; 
    double x = 1.0*n; 
    cout << "Roll\tFrequency\tProbablity\n"; 
    for(i = 1;i<v.size();++i) 
    { 
     cout << i+1 << "\t" << v[i] << "\t\t" << v[i]/x << endl; 
    } 
} 
+0

Вы зашиты вектор размера 12: 'вектор V (12),' –

+0

изменить размер вектора с номером введенному в ней ограничено 12. –

+0

Вопросы по Stack. Для переполнения требуется [Минимальный, полный и проверенный пример] (http://stackoverflow.com/help/mcve). Код здесь не минимален. – MrEricSir

ответ

0
sum[dice_1+dice_2]++; 

dice1+dice2 будет иметь значение от 2 до 12 лет включительно. sum имеет только элементы от 0 до 11 включительно. Вам нужно сделать размер sum до .

0

Как ваш вектор начинается с 0 следует хранить значения

sum[dice_1+dice_2-1]++; 

И во время печати.

начало от

for(i=0....)