Help with a ramdom word selection

Question

Hello everyone.
I'm trying to write a ramdom word program using a arraylist.
I'm NOT a student, I'm doing this as a fun project and to learn how to use ramdom method..
code is below.
class WordClass1 {
    private String wd_id;
    private String word_name;
    public WordClass1(String id) {
          wd_id = id;
    public WordClass1(String id, String Wd) {
         this.wd_id = id;
         this.word_name = Wd;
    // accessors
    public String getWd_id() {return wd_id;}
    public String getWord() {return word_name;}
public String toString() {
    return "(" + wd_id  + word_name + ")";
}//close class Person class
public class HangmanWords {
    static ArrayList<WordClass1> arlist;
    static Scanner kbd;
    public static WordClass1 makePerson() {
        WordClass1 temp = null;
        // prompt for data
        String id;
        String Wd;
System.out.print("Enter ID Number ==>");
id = kbd.next();
        System.out.print("Enter Last Name ==>");
        Wd = kbd.next();
        // make an object
        temp = new WordClass1(id, Wd);
        return temp;
    public static void main(String[] args) {
         // make array list object
          List < WordClass1 > arlist = new ArrayList < WordClass1 > ();
          arlist.add(new WordClass1("A1", "STRING"));
          arlist.add(new WordClass1("A2", "PERSON"));
          arlist.add(new WordClass1("B1", "CLASS"));
          arlist.add(new WordClass1("B2", "JAVA"));
         System.out.println(arlist);
        // make a scanner
        kbd = new Scanner(System.in);
        int choice;
          System.out.println("Make a Section: ");
          System.out.println("1. Enter Word ");
          System.out.println("2. Get the word ");
          System.out.println("3. Exit this Program ");
          System.out.print("\nPlease press Enter afer each response");
          System.out.println("\nEnter your choose please: ");
          choice = kbd.nextInt();
          kbd.nextLine();
          if (choice == 1) { // if 1 is select
        }// close while loop
        if (choice == 2) { // if 2 is select go to find
             int randomIndex = Math.abs(((Iterator<String>) arlist).next()getWord.length());
          if (choice == 3) {
               System.out.printf("Good bye");
          }// close the choice == 3
        // print out all elements of array list
        for (WordClass1 idx : arlist) {
             System.out.printf("Employee here are the list of all Employees Empoyeed");
             System.out.printf("Employee Id is %s%n", idx.getWd_id());
                System.out.printf("Name is %s  %s%n", idx.getWord());
                System.out.println("--------------------");
        }//close for loop
}//close main
}//close public classmy problem lies here in which I'm trying to get the word selected.
int randomIndex = Math.abs(((Iterator<String>) arlist).next()getWord.length());Any help would be great.
PS this code is for a hangman game.
nomad

Answer 1

A couple of suggestions:
1) You've got a nice WordClass1. Good to see some OOP going on. Why not do the same for HangmanWords class? Why not change the arraylist from a static variable to an instance variable? You can initialize it in the constructor. You then could create a third class: HangmanDriver that has your main. Think encapsulation.
2) arlist I hate to say it is a terrible name for a variable. It tells you what kind of data structure the data is in, something that could easily change in the future, but doesn't tell anything about what kind of data is contained in it. It would be better to call it something else, perhaps hangmanWords or wordList or somesuch thing.
3) try to encapsulate your user interface (I/O) routines into their own class. If you really get this program going, you're going to want to translate it from console to Swing GUI. Why not build it from the ground up in a way that would make this transition easy.
Here's one way the HangmanWords could look:
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import java.util.Random;
* encapsulates my hangman word list
* If I make this class implement the Iterable
* interface, I can loop through it more readily.
* @author petes1234
public class HangmanWords implements Iterable<WordClass1>
    private List<WordClass1> wordList;
    private Random rand;
    public HangmanWords()
        wordList = new ArrayList<WordClass1>();
        rand = new Random();
    public boolean add(WordClass1 word)
        return wordList.add(word);
    public boolean remove(WordClass1 word)
        return wordList.remove(word);
    public WordClass1 get(int i)
        return wordList.get(i);
    public WordClass1 getRandom()
        int randomIndex = rand.nextInt(wordList.size());
        return get(randomIndex);
    // this is all we need to implement the Iterable interface
    public Iterator<WordClass1> iterator()
        return wordList.iterator();
}