trjhtr

Please also see the new version here

So I've been trying to make a blackjack game as my first attempt at an OOP project. The program is working, but not yet finished as it is lacking three major features that I couldn't figure out how to implement.

1. Not being able to deal the same card more than once


2. Automatically changing the value of Aces to 1 or 11 depending on the hand


3. The dealer's secret card

Also, I originally had the player and house hit methods in their respective classes but there were some issues in the event of the player going bust. Mainly I'm intrested code readabillity, commenting, naming, efficiency, code practicallity, class and method splitting. Also I'm afraid that I have made the cove needlessly complicated, if that is the case please mention that too in your review. Here is my old code (I posted a self answer with my changes and addition of the missing features).

Program Class:

using System;

namespace Blackjack_21

 class Program
 
 static void Main(string args)
 
 //Variable declaration
 Console.Write("Insert a name: ");
 string playerName = Console.ReadLine();
 bool play = true;
 while (play == true)
 
 Game game = new Game();
 game.Play(playerName);
 Console.WriteLine("Press ENTER to continue");
 Console.ReadLine();
 Console.Clear();
 //ask for replay
 Console.Write("Do you want to play again? yes/no: ");
 bool answer = true;
 string ans = Console.ReadLine();
 //Looping until a correct answer is given
 while (answer)
 
 Console.Clear();
 
 
 

Game Class:

using System;

namespace Blackjack_21

 enum Status
 
 gameOn,
 playerWin,
 houseWin,
 draw,
 blackjack
 

 class Game
 
 byte status = (byte)Status.gameOn;
 House house = new House();
 Player player1 = new Player("");
 Card deck = new Card();

 //player hits as many times as he wants when < 21
 public int PlayerHit()
 
 if (!player1.Bust)
 
 Console.Write("0 Hits", player1.Name);
 player1.Hand += deck.DealCard();
 
 return player1.Hand;
 

 public int HouseHit()
 
 while (house.Hand < 16)
 
 Console.Write("The House Hits");
 house.Hand += deck.DealCard();
 
 house.Stand();
 return house.Hand;
 

 public void Play(string pName)
 

 player1.Name = pName;
 bool playerIsBust = player1.CheckPlayerBust();

 //Initial cards
 PlayerHit();
 PlayerHit();
 Console.Write("The House hits");
 house.Hand += deck.DealCard();

 //Player wins automatically if he has a "blackjack"
 if (player1.HasBlackjack() == true)
 
 Console.WriteLine($"player1.Name has a Blackjack!");
 status = (byte)Status.blackjack;
 
 else
 
 //loops as long as the player hasn't gone bust
 while (!playerIsBust && !player1.isStanding)
 input == "D")
 
 house.ViewHand();
 
 else
 
 Console.Write("Unknown commandnPlease type a valid command ");
 input = Console.ReadLine();
 
 playerIsBust = player1.CheckPlayerBust();
 /*Check for Aces here:
 if(player has aces)
 
 make aces value 1 until his hand < 22
 
 */
 
 if (playerIsBust == true)
 
 Console.WriteLine("0's hand is 1 so he goes bust"
 , player1.Name, player1.Hand);
 
 HouseHit();
 CompareHands();
 
 AnounceWinner();
 

 public void AnounceWinner()
 
 switch (status)
 
 case 1:
 Console.WriteLine($"player1.Name wins!");
 break;
 case 2:
 Console.WriteLine("The House wins!");
 break;
 case 3:
 Console.WriteLine("Is is a draw");
 break;
 case 4:
 Console.WriteLine($"player1.Name wins!");
 break;
 
 

 void CompareHands()
 
 if (player1.Hand > 21)
 player1.Hand = 0;
 if (house.Hand > 21)
 house.Hand = 0;

 if (player1.Hand > house.Hand)
 
 status = (byte)Status.playerWin;
 
 else if (house.Hand > player1.Hand)
 
 status = (byte)Status.houseWin;
 
 else
 
 status = (byte)Status.draw;
 
 if (player1.Bust && house.bust)
 
 status = (byte)Status.draw;
 
 
 

House Class:

using System;

namespace Blackjack_21

 class House
 
 public int Hand = 0;
 public bool bust = false;

 public void Stand()
 
 CheckHouseBust();
 if (bust)
 
 Console.WriteLine("The Houses's hand was 0 so it went bust", Hand);
 Hand = 0;
 
 else
 
 Console.WriteLine("The House stands at 0", Hand);
 
 

 public void ViewHand()
 
 Console.WriteLine("The House's hand is 0", Hand);
 

 bool CheckHouseBust()
 
 return bust = (Hand > 21) ? true : false;
 
 

Player Class:

using System;

namespace Blackjack_21

 class Player
 
 public string Name get; set; 
 public int Hand = 0;
 public bool Bust = false;
 public bool isStanding = false;

 public Player(string name = "")
 
 Name = name;
 

 public bool HasBlackjack()
 
 return (Hand == 21) ? true : false;
 

 public void Stand()
 
 if(!Bust)
 
 Console.WriteLine("0 stands at 1", Name, Hand);
 
 isStanding = true;
 

 public void ViewHand()
 
 Console.WriteLine("0's hand is 1", Name, Hand);
 

 public bool CheckPlayerBust()
 
 return Bust = (Hand > 21) ? true : false;
 
 

Card Class:

using System;
using System.Collections.Generic;

namespace Blackjack_21

 class Card
 
 //TODO:
 //can't deal the same card
 //value of 1 or 11 for ace to be assigned based on hand

 private readonly Random randCard = new Random(DateTime.Now.Millisecond);

 public int DealCard()
 
 IDictionary<int, string> numbers = new Dictionary<int, string>
 
 2, "Two" ,
 3, "Three" ,
 4, "Four" ,
 5, "Five" ,
 6, "Six" ,
 7, "Seven" ,
 8, "Eight" ,
 9, "Nine" ,
 10, "Ten" ,
 11, "Jack" ,
 12, "Queen" ,
 13, "King" ,
 14, "Ace" 
 ;

 IDictionary<int, string> suits = new Dictionary<int, string>
 
 1, "Spades" ,
 2, "Diamonds" ,
 3, "Hearts" ,
 4, "Clubs" 
 ;

 int value = randCard.Next(2, 15);
 int suit = randCard.Next(1, 5);
 Console.Write(" and was dealt the 0 of 1n"
 , numbers[value], suits[suit]);

 //Making the faces have value of 10
 if(value > 10 && value < 14)
 
 value = 10;
 
 //Making the Aces have value of 11
 if (value == 14)
 
 value = 11;
 
 return value;
 
 

I don't know if including the namespace and "using System" is needed in every code snippet so I decided to include them.

You should have one Player class, the Dealer class should inherit the Player class so that it can make use of all the player methods inside of it.

If you need a Method for the dealer that you don't want the player to have, it is okay Those methods can be put into the Dealer class.

That is one of the benefits of OOP (Object Oriented Programming). By extending the Player class you DRY (Don't Repeat Yourself) your code.

So, I decided to revisit this project of mine that I had abandoned, thinking that I got what experience I could from it. I changed some parts of the program and was able to add the missing features. Firstly, I took into account Malachi's suggestion of having the dealer class inherit from the player class and made the change. I cleaned up the DealCard() method, splitting it into 2 methods: the CreateDeck() and DealCard() which was slightly changed to be able to give the dealer his secret card and prevent duplicate cards being dealt. Finally the Player class got 2 new methods: the GetTotal() method to add the total value of his cards that are now being put into a list first so the second method, ChangeAces() can work. Also I forgot to mention that the cards are in a list now instead of a dictionary.

Here is the updated parts of the code:

The Program class is the same as before

The Game class:

using System;

class Game

 byte status = (byte)Status.gameOn;
 House house = new House();
 Player player1 = new Player("");
 Card deck = new Card();

 //player hits as many times as he wants when < 21
 public void PlayerHit()
 
 player1.CheckIfBust();
 if (!player1.Bust)
 
 Console.Write("0 Hits", player1.Name);
 player1.hand.Add(deck.DealCard(false));
 player1.GetTotal();
 
 

 //Initial cards
 public void InitialHits()
 
 PlayerHit();
 PlayerHit();

 house.CheckIfBust();
 if (!house.Bust)
 
 Console.Write("The House Hits");
 house.hand.Add(deck.DealCard(false));
 Console.WriteLine("The House gets a secret card");
 house.hand.Add(deck.DealCard(true));
 house.CheckIfBust();
 house.ChangeAces();
 
 

 //House hits untill it reaches 16
 public int HouseHit()
 
 while (house.total < 16)
 
 Console.Write("The House Hits");
 house.hand.Add(deck.DealCard(false));
 house.CheckIfBust();
 house.ChangeAces();
 
 house.Stand();
 return house.total;
 

 public void Play(string pName)
 
 player1.Name = pName;
 player1.CheckIfBust();
 deck.CreateDeck();

 //Initial cards
 InitialHits();

 //Player wins automatically if he has a "blackjack"
 if (player1.HasBlackjack() == true)
 
 Console.WriteLine($"player1.Name has a Blackjack!");
 status = (byte)Status.blackjack;
 
 else
 
 //loops as long as the player hasn't gone bust
 while (!player1.Bust && !player1.isStanding)
 
 if (player1.Bust)
 
 Console.WriteLine("0's hand is 1 so he goes bust"
 , player1.Name, player1.total);
 
 HouseHit();
 CompareHands();
 
 AnounceWinner();
 

 public void AnounceWinner()
 
 switch (status)
 
 case 1:
 Console.WriteLine($"player1.Name wins!");
 break;
 case 2:
 Console.WriteLine("The House wins!");
 break;
 case 3:
 Console.WriteLine("Is is a draw");
 break;
 case 4:
 Console.WriteLine($"player1.Name wins!");
 break;
 
 

 void CompareHands()
 
 if (player1.total > 21)
 player1.total = 0;
 if (house.total > 21)
 house.total = 0;

 if (player1.total > house.total)
 
 status = (byte)Status.playerWin;
 
 else if (house.total > player1.total)
 
 status = (byte)Status.houseWin;
 
 else
 
 status = (byte)Status.draw;
 
 if (player1.Bust && house.Bust)
 
 status = (byte)Status.draw;
 
 

The Player class:

using System;
using System.Collections.Generic;

class Player

 public string Name get; set; 
 public List<int> hand = new List<int>();
 public bool Bust = false;
 public bool isStanding = false;
 public int total;

 public Player(string name = "")
 
 Name = name;
 total = 0;
 

 public int GetTotal()
 
 total = 0;
 foreach (int card in hand)
 
 total += card;
 
 return total;
 

 public bool HasBlackjack()
 
 return (total == 21) ? true : false;
 

 public virtual void Stand()
 
 CheckIfBust();
 if (!Bust)
 
 Console.WriteLine("0 stands at 1", Name, total);
 
 isStanding = true;
 

 public virtual void ViewHand()
 
 Console.WriteLine("0's hand is 1", Name, total);
 

 public bool CheckIfBust()
 
 GetTotal();
 return Bust = (total > 21) ? true : false;
 

 public void ChangeAces()
 
 while (Bust && hand.Contains(11))
 
 hand[hand.FindIndex(index => index.Equals(11))] = 1;
 CheckIfBust();
 
 

The Dealer class:

using System;

class House : Player

 public override void Stand()
 
 CheckIfBust();
 if (Bust)
 
 Console.WriteLine("The Houses's hand was 0 so it went bust", total);
 total = 0;
 
 else
 
 Console.WriteLine("The House stands at 0", total);
 
 

The Card class:

using System;
using System.Collections.Generic;

class Card

 private readonly Random randCard = new Random(DateTime.Now.Millisecond);
 public List<string> Deck = new List<string>();

 public void CreateDeck()
 
 List<string> values = new List<string>
 
 "2",
 "3",
 "4",
 "5",
 "6",
 "7",
 "8",
 "9",
 "10",
 "Jack",
 "Queen",
 "King",
 "Ace"
 ;

 List<string> suits = new List<string>
 
 "Spades",
 "Diamonds",
 "Hearts",
 "Clubs"
 ;

 foreach (string value in values)
 
 foreach (string suit in suits)
 
 Deck.Add($"value of suit");
 
 
 

 public int DealCard(bool secret)
 card[0] == '4'