Creating an array of arrays in Java

Question

Say I have a Player class such as:

public class Player {

String name;
int chips;
int betVal;

}

Is the following code correct for creating the array of the players?

public static void main(String[] args) {
    int playerCount;
    int startingChip;
    out.print("How many players? ");
    playerCount = myScanner.nextInt();
    Player[] aPlayer = new Player[playerCount + 1];

    for (int i = 0; i < playerCount + 1; i++){
        aPlayer[i] = new Player();
    }

    out.print("Enter starting chip amount: ");
    startingChip = myScanner.nextInt();

}

If so, how would I assign the name, chip amount and the betVal to each player? How would I access and alter them later on in the code?

EDIT: Will it be easier leaving the Player as an object or an array (name,chips,betVal) for accessing it later on?

yes. though you could instead write `for (int i = 0; i < p.length; i++)` — nullpotent, Oct 30 '12 at 15:38
What does this have to do with an array of arrays? You have just a simple array of `Player` objects, not an array of arrays. — Jesper, Oct 30 '12 at 15:38
Yep as Assylias said you are creating 1 player too many. But other then that it is fine. And because it is an array of Player objects, you can just assign values to it. — Lyrion, Oct 30 '12 at 15:40
I was wondering whether it will be easier to have an array of arrays (i.e. the name, chips, betVal) or just an array of the 'Player' objects to access the values. Sorry that was not clear in the question — Rhiokai, Oct 30 '12 at 15:40

score 2 · Answer 1 · answered Oct 30 '12 at 15:44

First off, your variables need to be private (there's a section of programmers who prefer public variables but most prefer private).

You could set the values of the Player object either through an overloaded constructor or through the setters.

public class Player {


    private String name;
    private int chips;
    private int betVal;


    public Player(){
    //default constructor to initialize without any parameters
    }

    public Player(String name, int chips, int betVal){
        this.name=name;
        this.chips=chips;
        this.betVal=betVal;

    }


    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public int getChips() {
        return chips;
    }

    public void setChips(int chips) {
        this.chips = chips;
    }

    public int getBetVal() {
        return betVal;
    }

    public void setBetVal(int betVal) {
        this.betVal = betVal;
    }
}

In which case, your initialization could be either

aPlayer[i] = new Player("Jason", 5,1000);

or using the setters as in

for (int i = 0; i < playerCount + 1; i++){
        aPlayer[i] = new Player();
 }


 aPlayer[i].setName("Jason");
 aPlayer[i].setChips(5);
 aPlayer[i].setBetVal(1000);

Considering your sample program, I guess option 2 plays well.

score 1 · Answer 2 · edited Oct 30 '12 at 15:59

1

You can access each player by telling the array which one you want to access.

`aPlayer[0].name = "JAG";`

would work for example.

edited Oct 30 '12 at 15:59

Chris Gerken

16,221
6
44
59

answered Oct 30 '12 at 15:40

JAG90

11
3

Malexandre · Answer 3 · 2012-10-30T16:06:38.107

Why not ask all the user's inputs before creating anything ? For exemple :

public static void main(String[] args) {
    int playerCount;
    int startingChip;

    out.print("How many players? ");
    playerCount = myScanner.nextInt();
    out.print("Enter starting chip amount: ");
    startingChip = myScanner.nextInt();

    Player[] aPlayer = new Player[playerCount];

    for (int i = 0; i < playerCount; i++){
        aPlayer[i] = new Player();
        aPlayer[i].setChips(startingChip);
    }
}

You should also use a List. And finally, if you want to ask for the name of each player, do it directly in the loop :

for (int i = 0; i < playerCount; i++){
    aPlayer[i] = new Player();
    aPlayer[i].setChips(startingChip);

    out.print("What's the player " + i + " name? ");
    aPlayer[i].setName(myScanner.next());
}

I edited my code. I forgot to change the "nextInt" to a "next" when I copy/paste. — Malexandre, Oct 30 '12 at 15:51

score 0 · Answer 4 · answered Oct 31 '12 at 10:55

try something like this... ask for individual players info in the loop itself.

    System.out.println("How many players? ");
    Scanner myScanner = new Scanner(System.in);;
    playerCount = myScanner.nextInt();
    Player[] aPlayer = new Player[playerCount];

    for (int i = 0; i < playerCount; i++){
        aPlayer[i] = new Player();

        System.out.println("Enter Name for Player " + i+1);
        String name = myScanner.next();

        System.out.println("Enter chips for Player " + i+1);
        int chips = myScanner.nextInt();

        System.out.println("Enter betVal for Player " + i+1);
        int betVal = myScanner.nextInt();

        aPlayer[i].name = name;
        aPlayer[i].chips = chips;
        aPlayer[i].betVal = betVal;
    }

Also make the instance variables of Player class private, and access them using getters and setters.

Creating an array of arrays in Java

4 Answers4