Question

我一直试图在这段代码中找到错误的位置已经有一段时间了，我无法弄明白。该程序将6面模具滚动300次，然后输出每个数字滚动的时间。但出于某种原因，它不是滚动300次，而是滚动320次。我没有看到for循环有什么问题，所以我真的不知所措。

public static void dieRoll(){
    int[] roll = new int [300];
    int[] count = new int[] {1,2,3,4,5,6};

    for(int i = 1; i<300; i++){
            roll[i] = (int) Math.ceil( (int) (Math.random()*6)+1 );

//          roll[i] = (int) Math.ceil(roll[i]);
//          System.out.println(roll[i]);

            if(roll[i]==1){
                count[0]++;
            }
            else if(roll[i]==2){
                count[1]++;
            }
            else if(roll[i]==3){
                count[2]++;
            }
            else if(roll[i]==4){
                count[3]++;
            }
            else if(roll[i]==5){
                count[4]++;
            }
            else if(roll[i]==6){
                count[5]++;
            }

        //  System.out.println(roll[i]);

    }//i loop   

    System.out.println("The die landed on 1 " + count[0] + " times.");
    System.out.println("The die landed on 2 " + count[1] + " times.");
    System.out.println("The die landed on 3 " + count[2] + " times.");
    System.out.println("The die landed on 4 " + count[3] + " times.");
    System.out.println("The die landed on 5 " + count[4] + " times.");
    System.out.println("The die landed on 6 " + count[5] + " times.");
    System.out.println("The die was rolled this many times: " + (count[0]+count[1]+count[2]+count[3]+count[4]+count[5]));

}//dieRoll()

如果有人可以请指出错误可能表现在哪里，那将是非常棒的。谢谢。

Answer 1

你初步计算了你的计数：

int[] count = new int[] {1,2,3,4,5,6};

现在，1 + 2 + 3 + 4 + 5 + 6等于21.你的循环从1到299，这是299次迭代。当然，299 + 21是320。

您应该将数组初始化为全零。

最后，您的代码可以简化：

for( int i = 0; i < 300; i++ )
{
    roll[i] = (int) Math.ceil( (int) (Math.random()*6)+1 );
    count[roll[i] - 1]++;
}

用Java创建单个掷骰程序，但不是滚动300次，而是滚动320次

1 个答案: