二つのサイコロを振って目の合計を出力

前提・実現したいこと

二つのサイコロを振って、目の合計を出力するプログラムを作りたいです。
条件としてgetRandomIntという自作関数を使用します。

発生している問題・エラーメッセージ

実行結果が以下のようになってしまいます。

該当のソースコード

java
1public class Ex08_01 {
2	static double getRandomDouble(double low, double high) {
3		double ans = 0.0;
4		double r = Math.random();
5		ans = low + r * high;
6		return ans;
7	}
8	static int getRandomInt(int low, int high) {//この関数を使います
9		int ans = 0;
10		ans = (int) getRandomDouble(low, high);
11		return ans;
12	}
13	public static void main(String[] args) {
14	    int i,total;
15	    int[] d = {0,0,0,0,0,0,0,0,0,0,0};
16	    i = 0;
17	    total = 0;
18	    while(i < 3600) {//3600回振ります
19	    	int x = getRandomInt(0, 6);//サイコロX
20	    	int y = getRandomInt(0, 6);//サイコロY
21	    	total = x + y;
22	    	d[total]++;
23	    	i++;
24	    }
25	    System.out.println("二つのサイコロを振ったとき、");//目の合計がそれぞれ何回あったのか出力します
26	    System.out.println("目の合計が2だったのは"+d[0]+"回");
27	    System.out.println("目の合計が3だったのは"+d[1]+"回");
28	    System.out.println("目の合計が4だったのは"+d[2]+"回");
29	    System.out.println("目の合計が5だったのは"+d[3]+"回");
30	    System.out.println("目の合計が6だったのは"+d[4]+"回");
31	    System.out.println("目の合計が7だったのは"+d[5]+"回");
32	    System.out.println("目の合計が8だったのは"+d[6]+"回");
33	    System.out.println("目の合計が9だったのは"+d[7]+"回");
34	    System.out.println("目の合計が10だったのは"+d[8]+"回");
35	    System.out.println("目の合計が11だったのは"+d[9]+"回");
36	    System.out.println("目の合計が12だったのは"+d[10]+"回");
37	    }
38}

試したこと

インターネットで調べましたが、自作関数を使用するという条件にあるため、解決できませんでした。

補足情報（FW/ツールのバージョンなど）

質問に不十分なところがあれば、修正希望お待ちしています。

また、プログラミング初心者のため根本的におかしいところがあるかもしれませんが、お手柔らかにお願いします。

LouiS0616

2019/11/24 07:09

『合計の値』というのは、出た目の総合計ということですか？

BeatStar

2019/11/24 07:11

何がわからないのでしょうか。質問の意図と内容が違う気がします。

deliveryboy

2019/11/24 08:22

二つのサイコロの出た目の合計ということです。例えば、サイコロX＝１、サイコロY＝１だったら、２になります。それを、３だった場合、４だった場合・・・12だった場合というように分類して出力したいです。

swordone

2019/11/24 09:22 編集

コードを貼って実行してみましたが、ちゃんと回数が表示されます。問題が再現しません。提示したコードが、提示した結果が出たときのコードと異なっていませんか？

deliveryboy

2019/11/24 10:04

swordoneさんのおっしゃる通りでした。気づかせてくださりありがとうございます。

行動規範の内容に同意します

回答4件

それはそれとして、一つ問題が。

java
1    static double getRandomDouble(double low, double high) {
2        double ans = 0.0;
3        double r = Math.random();
4        ans = low + r * high;
5        return ans;
6    }

lowからhighの間の乱数を返すメソッドなのでしょうが、
0 ≦ r < 1なので、0 ≦ r * high < highで、
low ≦ ans < low + highとなり、範囲がずれます。
これはたまたまlowが0なので問題にはなっていませんが。

乱数で得られる数は最大最小の差である必要があります。

java
1    static double getRandomDouble(double low, double high) {
2        double ans = 0.0;
3        double r = Math.random();
4        ans = low + r * (high - low);
5        return ans;
6    }

こうすると、0 ≦ r * (high - low) < high - lowなので、
low ≦ ans < highとなります。
ただし、この場合highそのものが範囲に入らないので、intにキャストした場合もhighの値そのものが返ってくることがないことに気を付けてください。
highそのものを含めたい場合は、rに掛ける数に1を加えてください。

投稿2019/11/24 12:26

編集2019/11/24 15:53

swordone

総合スコア20669