Hello, I wanted some help fixing my code. I want my player class to draw 5 cards and avoid duplicates. I don't plan on using a set, so I want to shuffle the cards and then draw, however, It seems my shuffle isn't working. When i ran the program, i got duplicates as shown in the screenshot. I will paste my code here too if you'd like to see what I did and where i might be wrong. I'd love some help understanding how to fix it with explanations if possible. Or if there's a better solution with the shuffle, I'd be open to that as well. please and thank you.

this is my CARD CLASS:

public class Card
{

   public static final int ACE = 1;
   public static final int TWO = 2;
   public static final int THREE = 3;
   public static final int FOUR = 4;
   public static final int FIVE = 5;
   public static final int SIX = 6;
   public static final int SEVEN = 7;
   public static final int EIGHT = 8;
   public static final int NINE = 9;
   public static final int TEN = 10;
   public static final int JACK= 11;
   public static final int QUEEN = 12;
   public static final int KING = 13;

   public static final int CLUBS = 1;
   public static final int DIAMONDS = 2;
   public static final int HEARTS = 3;
   public static final int SPADES = 4;

   public int face, suit;
   public String faceName, suitName;

   Random rand = new Random();


   public Card()
   {
       int randomFace = rand.nextInt(13) + 1;
       int randomSuit = rand.nextInt(4) + 1;

       face = randomFace;
       suit= randomSuit;
   }
   public Card(int face, int suit)
   {
       this.face = face;
       this.suit = suit;
   }

   @Override
   public String toString()
   {
       switch(face)
       {
       case ACE:
           faceName = "Ace";
           break;
       case TWO:
           faceName = "Two";
           break;
       case THREE:
           faceName = "Three";
           break;
       case FOUR:
           faceName = "Four";
           break;
       case FIVE:
           faceName = "Five";
           break;
       case SIX:
           faceName = "Six";
           break;
       case SEVEN:
           faceName = "Seven";
           break;
       case EIGHT:
           faceName = "Eight";
           break;
       case NINE:
           faceName = "Nine";
           break;
       case TEN:
           faceName = "Ten";
           break;
       case JACK:
           faceName = "Eleven";
           break;
       case QUEEN:
           faceName = "Twelve";
           break;
       case KING:
           faceName = "Thirteen";
           break;
       }

       switch(suit)
       {
       case CLUBS:
           suitName = "Clubs";
           break;
       case DIAMONDS:
           suitName = "Diamonds";
       case HEARTS:
           suitName = "Hearts";
       case SPADES:
           suitName = "Spades";
       }
       return faceName + " of " + suitName;
   }
}

this is my PLAYER class:

public class Player
{

   private Card[]cards;
   private int numCards;
   Random rand = new Random();
   public Player()
   {
       cards = new Card[5];
   }


   public void deck()
   {
       Card[] deckOfCards = new Card[52];

       int cardIndex = 0; // index of top card (==next card to deal)

       for (int face = Card.ACE; face <= Card.KING; face++)
           for (int suit = Card.CLUBS; suit <= Card.SPADES; suit++)
               deckOfCards[cardIndex++] = new Card(face, suit);

       this.numCards = 52;
   }


   public void shuffle(Player deck)
   {
       // add R. Sedgewick's video code (shown in Class 7)

       int N = deck.numCards;
       for (int i = 0; i < N; i++)
       {
           int r = rand.nextInt(i + 1);
           exch(deck, i, r);
       }
   }

   private void exch(Player deck, int i, int r) {
       Card swap = cards[r];
       cards[r] = cards[i];
       cards[i] = swap;
   }

   public void drawCards()
   {

       for (int i=0; i<5; i++)
       {
           cards[i] = new Card();
       }
   }

   @Override
   public String toString()
   {
       String cardsToString = "";

       for (int i = 0; i < cards.length; i++)
       {
           cardsToString=cardsToString+cards[i].toString()+"\n";
       }

       return cardsToString;
   }
}

this is my DRIVER CLASS:

public class Driver
{

   public static void main(String[]args)
   {

       Player b = new Player();
       b.drawCards();
       System.out.println(b);
   }
}

Question

Hello, I wanted some help fixing my code. I want my player class to draw 5 cards and avoid duplicates. I don't plan on using a set, so I want to shuffle the cards and then draw, however, It seems my shuffle isn't working. When i ran the program, i got duplicates as shown in the screenshot. I will paste my code here too if you'd like to see what I did and where i might be wrong. I'd love some help understanding how to fix it with explanations if possible. Or if there's a better solution with the shuffle, I'd be open to that as well. please and thank you.this is my CARD CLASS:public class Card{   public static final int ACE = 1;   public static final int TWO = 2;   public static final int THREE = 3;   public static final int FOUR = 4;   public static final int FIVE = 5;   public static final int SIX = 6;   public static final int SEVEN = 7;   public static final int EIGHT = 8;   public static final int NINE = 9;   public static final int TEN = 10;   public static final int JACK= 11;   public static final int QUEEN = 12;   public static final int KING = 13;     public static final int CLUBS = 1;   public static final int DIAMONDS = 2;   public static final int HEARTS = 3;   public static final int SPADES = 4;         public int face, suit;   public String faceName, suitName;     Random rand = new Random();       public Card()   {       int randomFace = rand.nextInt(13) + 1;       int randomSuit = rand.nextInt(4) + 1;             face = randomFace;       suit= randomSuit;   }   public Card(int face, int suit)   {         this.face = face;       this.suit = suit;   }     @Override   public String toString()   {       switch(face)       {       case ACE:           faceName = &#34;Ace&#34;;           break;       case TWO:           faceName = &#34;Two&#34;;           break;       case THREE:           faceName = &#34;Three&#34;;           break;       case FOUR:           faceName = &#34;Four&#34;;           break;       case FIVE:           faceName = &#34;Five&#34;;           break;       case SIX:           faceName = &#34;Six&#34;;           break;       case SEVEN:           faceName = &#34;Seven&#34;;           break;       case EIGHT:           faceName = &#34;Eight&#34;;           break;       case NINE:           faceName = &#34;Nine&#34;;           break;       case TEN:           faceName = &#34;Ten&#34;;           break;       case JACK:           faceName = &#34;Eleven&#34;;           break;       case QUEEN:           faceName = &#34;Twelve&#34;;           break;       case KING:           faceName = &#34;Thirteen&#34;;           break;       }             switch(suit)       {       case CLUBS:           suitName = &#34;Clubs&#34;;           break;       case DIAMONDS:           suitName = &#34;Diamonds&#34;;       case HEARTS:           suitName = &#34;Hearts&#34;;       case SPADES:           suitName = &#34;Spades&#34;;       }       return faceName + &#34; of &#34; + suitName;     }}  this is my PLAYER class: public class Player{   private Card[]cards;   private int numCards;   Random rand = new Random();   public Player()   {       cards = new Card[5];   }       public void deck()   {       Card[] deckOfCards = new Card[52];       int cardIndex = 0; // index of top card (==next card to deal)             for (int face = Card.ACE; face <= Card.KING; face++)           for (int suit = Card.CLUBS; suit <= Card.SPADES; suit++)               deckOfCards[cardIndex++] = new Card(face, suit);             this.numCards = 52;   }       public void shuffle(Player deck)   {       // add R. Sedgewick's video code (shown in Class 7)             int N = deck.numCards;       for (int i = 0; i < N; i++)       {           int r = rand.nextInt(i + 1);           exch(deck, i, r);       }   }   private void exch(Player deck, int i, int r) {       Card swap = cards[r];       cards[r] = cards[i];       cards[i] = swap;   }     public void drawCards()   {             for (int i=0; i<5; i++)       {           cards[i] = new Card();       }   }     @Override   public String toString()   {       String cardsToString = &#34;&#34;;             for (int i = 0; i < cards.length; i++)       {           cardsToString=cardsToString+cards[i].toString()+&#34;
&#34;;       }             return cardsToString;   }} this is my DRIVER CLASS: public class Driver{   public static void main(String[]args)   {             Player b = new Player();       b.drawCards();       System.out.println(b);   }} 5607810111213 }14public static void main(String[] args){}Player b = new Player ();b.drawCards ();System.out.println(b);Problems @JavadocDeclarationConsole X Driver (1) [Java Application] C:\Program Files\Java\jdk-17.0.4\bin\javaw.exe (Feb 18, 2023, 8:02:32 PM - 8:02:32 PM)Eight of SpadesSix of SpadesEight of SpadesEleven of ClubsTwelve of Spades

Accepted Answer

The issue in your code is that you are not calling the shuffle() method anywhere in your code, so…