Generating poker hands for N number of players (five cards each)

Question

Problem: Write a program Deal.java that takes an command-line argument N and prints N poker hands (five cards each) from a shuffled deck, separated by blank lines.

What I have:

public static void main(String[] args)
{
    int N = Integer.parseInt(args[0]);


    String[] suit = { "Clubs", "Diamonds", "Hearts", "Spades" };
    String[] rank = 
    { 
        "2", "3", "4", "5", "6", "7", "8", "9", "10", 
        "Jack", "Queen", "King", "Ace" 
    };

    // initialize cards in a deck
    String[] deck = new String[suit.length * rank.length];
    for (int i = 0; i < rank.length; i++)
        for (int j = 0; j < suit.length; j++)
            deck[rank.length*i + j] = rank[i] + " of " + suit[j];

    // shuffle deck
    int d = deck.length;
    for (int i = 0; i < d; i++)
    {
        int r = i + (int) (Math.random() * (d-i));
        String temp = deck[r];
        deck[r] = deck[i];
        deck[i] = temp;
    }

    // repeat for N number of people
    for (int t = 0; t < N; t++)
    {
        // print 5 random cards
        for (int i = 0; i < 5; i++)
            System.out.print(deck[i] + " ");
        System.out.println();
    }   


}

Can anybody tell me what I'm doing wrong here? I'm getting an error ArrayIndexOutOfBoundsExceptions, not sure why. This is a problem out of an exercise set from my book, not homework.

What you are doing wrong: Not giving any details on what isn't actually working. Not summarizing what you have done to try to correct this problem. Not giving any error messages. — RussS, Feb 19 '12 at 17:56
+1 for @RussS - you need to do some debugging. If you cannot debug, you cannot develop software. Also, when I deal, I don't put any cards back in the pack. — Martin James, Feb 19 '12 at 18:04
i foolishly assumed that my book had no typos whatsoever. i should've caught the part that ruakh corrected myself — user1164937, Feb 19 '12 at 18:11

score 2 · Accepted Answer · answered Feb 19 '12 at 18:00

2

I assume you're getting an array-index-out-of-bounds exception on this line:

deck[rank.length*i + j] = rank[i] + " of " + suit[j];

where you must have meant this:

deck[suit.length*i + j] = rank[i] + " of " + suit[j];

answered Feb 19 '12 at 18:00

ruakh

175,680
26
273
307

ah thx! there was a typo in the book so i would have never guessed (i just copied that part down) sadly, i can't upvote due to 2 ppl downvoting me :( – user1164937 Feb 19 '12 at 18:07
@MarioA.CorcheroJiménez Why wouldn't it run properly? Looks correct to me. – Daniel Fischer Feb 19 '12 at 18:13
it wouldn't it run properly because I read wrongly, I'm sorry – Mario Corchero Feb 19 '12 at 18:23
I mean, it's completely ok, my fault – Mario Corchero Feb 19 '12 at 18:23

score 2 · Answer 2 · answered Feb 19 '12 at 18:00

2

In addition to the index error spotted by ruakh, you are printing out the same hand N times:

// repeat for N number of people
for (int t = 0; t < N; t++)
{
    // print 5 random cards
    for (int i = 0; i < 5; i++)
        System.out.print(deck[i] + " ");
    System.out.println();
}

To print out different hands, use another index variable,

// repeat for N number of people
int j = 0
for (int t = 0; t < N; t++)
{
    // print 5 random cards
    for (int i = 0; i < 5; i++, j++)
        System.out.print(deck[j] + " ");
    System.out.println();
}

answered Feb 19 '12 at 18:00

Daniel Fischer

181,706
17
308
431

while it may not have been the answer i was looking for, you may have already answered what i was going to ask next! thanks – user1164937 Feb 19 '12 at 18:08
That's precognition :) Earnestly, you hadn't yet mentioned the exception, so I took what leaped into eyesight. – Daniel Fischer Feb 19 '12 at 18:11

score 0 · Answer 3 · answered Feb 19 '12 at 18:16

I can't see what's the problem seems to work ok, try to give some further details.

Something you should improve is:

Check the parameter is set at the beginning properly:

public static void main(String[] args)
{
    int N = Integer.parseInt(args[0]);

String[] suit = { "Clubs", "Diamonds", "Hearts", "Spades" };
String[] rank = 
{ 
    "2", "3", "4", "5", "6", "7", "8", "9", "10", 
    "Jack", "Queen", "King", "Ace" 
};

    if(N < 0 || N > suit.length * rank.length)
      throw new IllegalArgumentException("A number between 0 and " + suit.length * rank.length + "should be provided as argument");

as @ruanhk said: deck[suit.length*i + j] – Mario Corchero Feb 19 '12 at 18:24 — Mario Corchero, Feb 19 '12 at 18:24

Generating poker hands for N number of players (five cards each)

3 Answers3