Help with creating a method within a class

Question

I need help creating another method within the same class.
Here is part of my progam, and could anyone help me convert the switch statement into a seperate
method within the same class?
import javax.swing.JOptionPane;
public class Yahtzee
     public static void main (String[] args)
          String numStr, playerStr, str, tobenamed;
          int num, times = 13, roll, x, y, maxRoll = 3, z = 0, t;
          int firstDie, secondDie, thirdDie, fourthDie, fifthDie, maxDie = 5, reroll;
          int rerolling = 0, categoryChoice, score, nextDie1, nextDie2, nextDie3, nextDie4;
          Die die1, die2, die3, die4, die5;
          do
          numStr = JOptionPane.showInputDialog ("Enter the number of player: ");
          num = Integer.parseInt(numStr);
          while (num < 1 || num > 6); //end of do loop
          String [] player = new String[num];
          // boolean //finsih later to make category choice only once.
          for (x = 0; x < num; x++)
               playerStr = JOptionPane.showInputDialog ("Name of Player" + (x + 1) + " :");
               player[x] = playerStr;  
          } //end of for loop
          die1 = new Die();
           die2 = new Die();
           die3 = new Die();
           die4 = new Die();
           die5 = new Die();
          int scoring [][] = new int [num][13];//scoring aray
          int[] numDie = new int[maxDie];
          String[][] usedCategory = new String [num][times];
          for (x=0; x < 13; x++)
               for (y = 0; y < num; y++)
                  usedCategory[y][x] = " ";
          for (int choices = 0; choices < times; choices++)
               //player turns for loop.
               for (x = 0; x < num; x++)
                    //rolls the die;
                      for (y = 0; y < maxDie; y++)
                           numDie[y] =  die1.roll();
                 numStr = JOptionPane.showInputDialog (player[x] + "\n~~~~~~~~~~~~~~~~~~~~~~~"
                                                                        + "\nDie1: " + numDie[0]
                                                                            + "\nDie2: " + numDie[1]
                                                                            + "\nDie3: " + numDie[2]
                                                                            + "\nDie4: " + numDie[3]
                                                                            + "\nDie5: " + numDie[4]
                                                                            + "\nWhich dice do you want to reroll");
                reroll = numStr.length();
                if (reroll == 0)
                    t = maxRoll;
               else{                                                      
                //reroll
                     for(t = 0; t < maxRoll; t++ )
                         for (y = 0; y < reroll; y++)
                                rerolling = numStr.charAt(y);
                             //create another mwthod later.
                                switch (rerolling)
                                     case '1':
                                          numDie[0] =  die1.roll();
                                          break;
                                     case '2':
                                          numDie[1] =  die1.roll();
                                          break;
                                     case '3':
                                          numDie[2] =  die1.roll();
                                          break;
                                     case '4':
                                          numDie[3] =  die1.roll();
                                          break;
                                     case '5':
                                          numDie[4] =  die1.roll();
                                          break;
                                     default:
                                     t = maxRoll;//to be changed
                                } //end of switch class
                          }//end of reroll for loop
                               JOptionPane.showMessegeDialog (player[x] + "\n~~~~~~~~~~~~~~~~~~~~~~~"
                                                                                  + "\nDie1: " + numDie[0]
                                                                                      + "\nDie2: " + numDie[1]
                                                                                      + "\nDie3: " + numDie[2]
                                                                                      + "\nDie4: " + numDie[3]
                                                                                      + "\nDie5: " + numDie[4]
                                                                                      + "\nWhich dice do you want to reroll"
                                           switch (rerolling)
                                     case '1':
                                          numDie[0] =  die1.roll();
                                          break;
                                     case '2':
                                          numDie[1] =  die1.roll();
                                          break;
                                     case '3':
                                          numDie[2] =  die1.roll();
                                          break;
                                     case '4':
                                          numDie[3] =  die1.roll();
                                          break;
                                     case '5':
                                          numDie[4] =  die1.roll();
                                          break;
                                     default:
                                     t = maxRoll; //not really anything yet, or is it?
                                } //end of rerolling switch class
                                   //categorychoice = Integer.parseInt(category);
                          }//end of t for loop            
               }//end of player for loop.

Answer 1

samuraiexe wrote:
well, i should have said it is a yahtzee program, and i ask the user what they want to reroll and whatever number they press, the switch will activate the case and the array that it will reroll.
HOw would i pass it as an argument?You put it in the argument list. Modifying your original code a bit:
public static void switchReroll (int[] dice, int position) {
     switch (position) {
     case '1':
             dice[0] =  die1.roll()
             break;
     case '2':
             dice[1] =  die1.roll();
             break;
     case '3':
             dice[2] =  die1.roll();
                 break;
     case '4':
             dice[3] =  die1.roll();
             break;
     case '5':
              dice[4] =  die1.roll();
             break;
     default:
             t = maxRoll;//to be changed
    } //end of switch
}which you could then call as this:
switchReroll(numDie, reroll);Note that your code still isn't done; the above won't compile. This is just an example of how you could pass the stuff as an argument (and note how the values have different names in the main method than in the switchReroll method -- this works, but also note that it's the same array of ints in both).
By the way -- you really don't need a switch statement for this. You can do the same thing with a couple lines of code, which would lessen the advantage for a separate method. But I'd suggest continuing down this path, and then you could change the implementation of the method later.