Как мне симулировать 3000 бросков костей и посчитать, сколько раз выпадают двойники? ДЖАВА

Я новичок в мире программирования, поэтому я буду очень признателен, если кто-нибудь из вас поможет мне стать лучше в этом. Моя цель в этой программе - симулировать 3000 бросков костей и подсчитать, сколько раз броски удваиваются для каждой из возможных пар двойников, используя цикл while. Результаты должны быть распечатаны в диалоговом окне.

   Random random;
   random = new Random();
   diceRolls = 1;
   snakeEyes = 1;
   doubleTwos = 1;
   doubleThrees = 1;
   doubleFours = 1;
   doubleFives = 1;
   doubleSixes = 1;


   while (diceRolls <= finalDiceRoll) {
       int diceRolls = 1;
       int die1 = random.nextInt(6) + 1;
       int die2 = random.nextInt(6) + 1;


       if (die1 == 1 && die2 == 1){
           //snakeEyes = snakeEyes + 1;    
           snakeEyes++;

       }
       else if (die1 == 2 && die2 == 2 ) {
           doubleTwos++;
       }
       else if (die1 == 3 && die2 == 3) {
           doubleThrees++;
       }
       else if (die1 == 4 && die2 == 4) {
           doubleFours++;
       }
       else if (die1 == 5 && die2 == 5) {
           doubleFives++;
       }
       else if (die1 == 6 && die2 == 6) {
           doubleSixes++;
       }
       JOptionPane.showMessageDialog (null, "You rolled snake eyes " + snakeEyes + " times\nYou rolled double twos " + doubleTwos + " times\nYou"
                                     + " rolled double threes " + doubleThrees + " times\nYou rolled double fours " + doubleFours + " times\nYou"
                                     + " rolled double fives " + doubleFives + " times\nYou rolled double sixes " + doubleSixes + " times");

}

Проблема, с которой я столкнулся, заключается в том, что результаты, которые я получаю от программы, не кажутся "правдоподобными" Например, из 3000 бросков игральных костей я получаю 1 пару каждого дубля. Что я делаю неправильно?

Источник

4 ответа

Переместить showMessageDialog вне цикла while и увеличивайте diceRolls переменная. Инициализируйте все остальные целочисленные переменные с 0.
Лучшим (более чистым и коротким) подходом было бы использование двумерного массива или карты.

Random random = new Random();
int snakeEyes = 0;
int doubleTwos = 0;
int doubleThrees = 0;
int doubleFours = 0;
int doubleFives = 0;
int doubleSixes = 0;
int diceRolls = 1;
while (diceRolls <= 3000) {
    int die1 = random.nextInt(6) + 1;
    int die2 = random.nextInt(6) + 1;

    if (die1 == 1 && die2 == 1) {
        snakeEyes++;
    } else if (die1 == 2 && die2 == 2) {
        doubleTwos++;
    } else if (die1 == 3 && die2 == 3) {
        doubleThrees++;
    } else if (die1 == 4 && die2 == 4) {
        doubleFours++;
    } else if (die1 == 5 && die2 == 5) {
        doubleFives++;
    } else if (die1 == 6 && die2 == 6) {
        doubleSixes++;
    }
    diceRolls++;
}
JOptionPane.showMessageDialog(null, "You rolled snake eyes " + snakeEyes + " times\nYou rolled double twos " + doubleTwos + " times\nYou"
        + " rolled double threes " + doubleThrees + " times\nYou rolled double fours " + doubleFours + " times\nYou"
        + " rolled double fives " + doubleFives + " times\nYou rolled double sixes " + doubleSixes + " times");

Вышеупомянутая более короткая версия:

Random random = new Random();
int[] doubled = new int[6];

for (int diceRolls = 0; diceRolls < 3000; diceRolls++) {
    int die1 = random.nextInt(6);
    int die2 = random.nextInt(6);

    if (die1 == die2)
        doubled[die1]++;
}
JOptionPane.showMessageDialog(null, "You rolled snake eyes " + doubled[0] + " times\nYou rolled double twos " + doubled[1] + " times\nYou"
        + " rolled double threes " + doubled[2] + " times\nYou rolled double fours " + doubled[3] + " times\nYou"
        + " rolled double fives " + doubled[4] + " times\nYou rolled double sixes " + doubled[5] + " times");
Источник

Несколько изменений, которые вы должны внести в свой код,

1. Инициализируйте все ваши переменные 0,

2. изменить while-loop условие для

 while (diceRolls <= finalDiceRoll)  // hoping that finalDiceRoll = 3000

3 удалить int diceRolls = 1; от while-loop и добавить diceRolls++; в конце while-loop,

4. Поставь свой JOptionPane вне while-loop, если нет, вы должны закрыть 3000 раз JOptionPane dailog.

После внесения этих изменений в ваш код вы узнаете, что вы сделали неправильно, и ваш код будет работать нормально.

Источник

Логика счета должна идти следующим образом:

doubleTwos = 0;
doubleThrees = 0;
doubleFours = 0;
doubleFives = 0;
doubleSixes = 0;

finalDiceRoll = 3000;
int diceRolls = 0;
while (diceRolls < finalDiceRoll) {
    ++diceRolls;

С любым сообщением после цикла.

С массивом вы можете сохранить набрав:

int[] doubleCounts = new int[6]; // By 0 based dice values
for (int i = 0; i < finalDiceRoll; ++i) {
    int die1 = random.nextInt(6); // With dice values 0-5
    int die2 = random.nextInt(6);

    if (die1 == die2) {
       ++doubleCounts[die1];
    }
}
int snakeEyes = doubleCounts[0];
int doubleTwos = doubleCounts[1];
...
Источник

Упростить. Устранить дубликаты. Не используйте несколько переменных, когда вы можете использовать массив.

int[] doubles = new int[7]; // make the index the number rolled (ignore index 0)
int diceRolls = 0;
while (diceRolls++ < 3000) {
  int die1 = random.nextInt(6) + 1;
  int die2 = random.nextInt(6) + 1;
  if (die1 == die2)
       doubles[die1]++;
}
String output = "";
for (int i = 1; i <= 6; i++)
    output += "You rolled double " + i + "'s " + doubles[i] + " times\n";

JOptionPane.showMessageDialog (null, output);

Это все, что вам нужно.

Источник
Другие вопросы по тегам