Бросаем 2 шестигранных кубика 1000 раз, неожиданные результаты - PullRequest
Новая
       14

Бросаем 2 шестигранных кубика 1000 раз, неожиданные результаты

0 голосов
/ 13 января 2020

При имитации броска 2 кубиков 1000 раз и отображении результата он продолжает возвращать неожиданные значения. Моя программа: 

import java.util.*;

public class Prob1 {
    private int sides; //Amount of sides on die
    private int sumOfRolls; 
    private int[] arr; //Array that holds the count of each sum of the rolls
    private int count; //Temporarily holds value of sum of rolls before passing to array
    public Prob1() { //Default constructor, sets sides to 6
        sides = 6;
        sumOfRolls = 0;
        count =0;
        arr = new int[(sides*2)+1];
    }
    public void roll() { //Method that rolls the 2 dice
        for(int i=0;i<1000;i++) { //Repeats 1000 times
            int roll1 = (int) (Math.random()*sides)+1; //Rolls 1 die
            int roll2 = (int) (Math.random()*sides)+1; //Rolls other die
            sumOfRolls = roll1+roll2; //Sums both rolls
            for(int j=2;j<(sides*2)+1;j++) { //Goes through all the possible values of sums of roll
                if(sumOfRolls==j) { //Checks if sum of rolls is equal to the value presented by the for loop
                    count++; //Adds 1 to count of value of sum of roll
                    arr[j]=count; //Adds that value to array
                }
            }
        }
        System.out.println("Roll: " + sides + "-sided dice"); //Displays what kind of dice
    }
    public void frequency() { //Frequency table
        System.out.println("\n\n\nRoll Total  Frequency");
        System.out.println("---------------------");
        for(int i=2; i<(sides*2)+1; i++) { //Displays each value of array
            System.out.print(i + "           " + arr[i] + "\n");

        }

    }


}

Когда вызываются методы крена и частоты, это результат: 

Roll: 6-sided dice



Roll Total  Frequency
---------------------
2           952
3           995
4           976
5           989
6           991
7           999
8           1000
9           998
10           992
11           994
12           997

Это ошибочно, так как сумма каждого значения превышает 1000, количество бросков двух костей. Есть ли какое-то решение для этого?

1 Ответ

2 голосов
/ 13 января 2020

Проблема в том, что вы не сбрасываете переменную счета после каждого броска

Удалите count++ и просто выполните arr[j]++.

Редактировать

Как уже упоминалось в комментариях , l oop не нужен, потому что вы можете просто сделать arr[sumOfRolls]++.

...