Question

Here is what I’m suppose to accomplish:

Write a program that stimulates a bean machine Your program should prompt the user to enter the number of balls and the number of slots in the machine. Simulate the falling of each ball by printing its path.

EX.

Enter the number of balls: 5
Enter the number of slots: 7

LRLRLRL
RRLRLLL
LLRRLLR
LRLLRLR
RRRLRRL
_ _ 0
_ _ 0
0 0 0

Here is my code so far:

import javax.swing.JOptionPane;         public static void main(String[] args) {             int balls=0;             int slots=0;             char [] direction= new char [slots];             int slot=0;             int i=0;             int path=0;              balls= Integer.parseInt(JOptionPane.showInputDialog('Enter' +                     ' the number of balls to be dropped:'));             slots= Integer.parseInt (JOptionPane.showInputDialog('Enter ' +                     'the number of slots:'));              for (int j=1;j<=balls;j++){                 while(i<slots){                     path= (int)(Math.random()*100);                     if (path <50){                         direction [slots]='L';                     }                     else{                         direction [slots]='R';                     }                 i++;                 slot++;             }             System.out.println('The pathway is' +direction[0]+direction[1]+direction[2]+direction[3]+direction[4]);         }     } 

There are a few things that I’m having problems with:

1. In the last line of my code where I try to print the pathway I have to basically guess the number of slots the user selected. Is there a better way to print this?
2. How can I print the number ‘balls’ that the user entered in the pattern as shown above?
3. Are there any other problems with my code?

Answer 1

Well, for starters, I’m getting a consistent ArrayIndexOutOfBoundsException on the line direction[slots] = 'L'; (or 'R'). That’s because direction is always of length 0, since you initialized it to slots when slots was 0. Move the line

char [] direction= new char [slots]; 

to after slots is input.

Next, you always assign the ‘L’ or ‘R’ to the position immediately after the end of the array. That’s another reason for the ArrayIndexOutOfBoundsException I was getting. Change the assignment to

direction[i] = 'L'; // or 'R' 

Next, you don’t reset i after the while loop. So the path is calculated only for the first ball and then reused for all the others. I would make it a for loop instead, like this:

for (i = 0; i < slots; i++) {     // your code here (make sure you don't change i inside the loop) } 

Finally, as others have said, you should be using a loop to print out the path. You know how long the direction array is (it’s direction.length, if you didn’t know), so you can just loop through it and print out each letter.

Once you’ve made these changes, your program should work (edit: except that it doesn’t keep track of which slot each ball ends up in). It will still have some room for improvement, but finding those things is part of the fun–isn’t it?