How to output cards in C#?

Question

1.00/5 (1 vote)

See more:

I made this code for a blackjack game, it works fine, but even when I created an array of cards to display the type of cards each player has at the end, it is not doing so, it is just displaying who won, who lost and the total number of card, never displays the type of cards of each player. Any suggestion, please?
Thanks

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace GUI_BLACKJACK
{
    public partial class Form1 : Form
    {

        class Card
        {

            public string Name { get; set; }
            public int Value { get; set; }
            public string Suit { get; set; }

            Card[] DeckOfCards = new Card[]
            {

                    new Card{Name="two",Value=2,Suit="Heart"},
                    new Card{Name="three",Value=3,Suit="Heart"},
                    new Card{Name="four",Value=4,Suit="Heart"},
                    new Card{Name = "five", Value = 5, Suit = "Heart" },
                    new Card{Name = "six", Value = 6, Suit = "Heart" },
                    new Card{Name = "seven", Value = 7, Suit = "Heart" },
                    new Card{Name = "eight", Value = 8, Suit = "Heart" },
                    new Card{Name = "Nine", Value = 9, Suit = "heart" },
                    new Card{Name = "ten", Value = 10, Suit = "Heart" },

                    new Card { Name = "two", Value = 2, Suit = "Spades" },
                    new Card { Name = "three", Value = 3, Suit = "Spades" },
                    new Card { Name = "four", Value = 4, Suit = "Spades" },
                    new Card { Name = "five", Value = 5, Suit = "Spades" },
                    new Card { Name = "six", Value = 6, Suit = "Spades" },
                    new Card { Name = "seven", Value = 7, Suit = "Spades" },
                    new Card { Name = "eight", Value = 8, Suit = "Spades" },
                    new Card { Name = "Nine", Value = 9, Suit = "Spades" },
                    new Card { Name = "ten", Value = 10, Suit = "Spades" },


                    new Card { Name = "two", Value = 2, Suit = "Diamonds" },
                    new Card { Name = "three", Value = 3, Suit = "Diamonds" },
                    new Card { Name = "four", Value = 4, Suit = "Diamonds" },
                    new Card { Name = "five", Value = 5, Suit = "Diamonds" },
                    new Card { Name = "six", Value = 6, Suit = "Diamonds" },
                    new Card { Name = "seven", Value = 7, Suit = "Diamonds" },
                    new Card { Name = "eight", Value = 8, Suit = "Diamonds" },
                    new Card { Name = "Nine", Value = 9, Suit = "Diamonds" },
                    new Card { Name = "ten", Value = 10, Suit = "Diamonds" },


                    new Card { Name = "two", Value = 2, Suit = "Clubs" },
                    new Card { Name = "three", Value = 3, Suit = "Clubs" },
                    new Card { Name = "four", Value = 4, Suit = "Clubs" },
                    new Card { Name = "five", Value = 5, Suit = "Clubs" },
                    new Card { Name = "six", Value = 6, Suit = "Clubs" },
                    new Card { Name = "seven", Value = 7, Suit = "Clubs" },
                    new Card { Name = "eight", Value = 8, Suit = "Clubs" },
                    new Card { Name = "Nine", Value = 9, Suit = "Clubs" },
                    new Card { Name = "ten", Value = 10, Suit = "Clubs" },
             };
        }
        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        private void label2_Click(object sender, EventArgs e)
        {

        }

        private void label1_Click(object sender, EventArgs e)
        {

        }
        
        private void button1_Click(object sender, EventArgs e)
        {
            Random random = new Random();
            int dealer = random.Next(1, 10);

            int player = random.Next(1, 10);

            label4.Text = ($"Let's play a modified version of the card game blackjack," +
                $" The dealer is showing a :{ dealer}, the other card is hidden");

            label1.Text = ("you were dealt 2 cards that total " + player);
        }

        private void label3_Click(object sender, EventArgs e)
        {

        }

        private void button2_Click(object sender, EventArgs e)
        {
            label4.Text = ("Do you want another card? Type [Y]es or [N]o");

        }

        private void label6_Click(object sender, EventArgs e)
        {

        }
      
        
        public string PlayGame(int dealer, int player)
        {


            string response = "";
           
            if (player <= 21 && dealer > 21)
            {
                label1.Text = ($"You WON with {player}");
                label2.Text = ($"The dealer ended with {dealer}");
            }

            else if (player < 21 && dealer < 21 && dealer > player)
            {
                label1.Text = ($"You lost with {player} ");
                label2.Text = ($"The dealer ended with {dealer}");
            }

            else if (player < 21 && dealer < 21 && player > dealer)
            {
                label1.Text = ($"You WON with {player}");
                label2.Text = ($"The dealer ended with {dealer}");
            }

            else if (player > 21 && dealer < player)
            {
                label1.Text = ($"You lost with {player}");
                label2.Text = ($"The dealer ended with  {dealer}");
            }
            else if (player == 21 && dealer < player)
            {
                label1.Text = ($"You WON with {player}");
                label2.Text = ($"The dealer ended with {dealer}");
            }
            else if (player > 21)
            {
                label1.Text = ($"You lost with {player}");
                label2.Text = ($"The dealer ended with  {dealer}");
            }
            
            {
                return response;
            }
        }
            
            public void button3_Click(object sender, EventArgs e)
            {
                Random random = new Random();
                int dealer = random.Next(1, 10);

                int player = random.Next(1, 10);

                int randomNumber = random.Next(1, 10);
                player += randomNumber;

                PlayGame(dealer, player);

            }

            private void button4_Click(object sender, EventArgs e)
            {

                Random random = new Random();
                int dealer = random.Next(1, 10);

                int player = random.Next(1, 10);

                int randomNumber = random.Next(1, 10);
                player += randomNumber;

                PlayGame(dealer, player);
            }

            private void label2_Click_1(object sender, EventArgs e)
            {

            }

        
}   }

What I have tried:

I have tried by adding {card} after {player} and after {dealer} at the label. text

label1.Text = ($"You WON with {player}{Card}");
label2.Text = ($"The dealer ended with {dealer} {Card}");
I have tried by adding {Suit} at the same positions above mentioned to see if at least the suit of cards are mentioned.

Posted 16-Sep-21 16:49pm

Diasalva5

Updated 16-Sep-21 19:53pm

Add a Solution

1 solution

Add a Solution

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Accepted Answer · 2021-09-16T19:53:00

Solution 1

There is no code in there to display cards: you would need to add it. I'd start by adding an override of ToString to your Card class - but frankly you need to use it first.

Your code to generate a card totally ignores your Card class, and looks like it was written in C and randomly translated to C# - it contains nothing to hold the two players "cards so far", just the total value, and can work with a deck containing more than 4 Aces for example. I'd stringy suggest you sit down and think about exactly what you have to do here, then design classes (note the plural, it's deliberate) to handle it. What you have is probably not a good starting place for this homework.

Posted 16-Sep-21 19:53pm

OriginalGriff

Comments

CPallini 17-Sep-21 2:10am

5.

Diasalva5 17-Sep-21 10:04am

Dear OriginalGiff
Thank you for your answer and ideas. One more question, please: In addition to study hard and practice, how could I become a good programmer? Any tips ?
Thanks

OriginalGriff 17-Sep-21 10:40am

Not really - there isn't a short cut!
THink of it like riding a bicycle: you can watch as much of the Tour de France on TV as you want, talk to as many bike riders as you can - but when you first get on a bike you are going to fall off. The reason is simple: you don't have the reflexes, the mindset to make the tiny adjustments that keep you and it upright without even having to think about them.

Development is the same: you can copy'n'paste code, look at other peoples work, watch as many Youtube videos as you can stand - none of it will make you a better coder because you don't have the mindset which lets you know instinctively what to do.

Look at the code you first showed us: did you actually write all that? Because I doubt it myself! :laugh:

Diasalva5 17-Sep-21 10:54am

Why are you doubting yourself that I wrote all my code by my own effort? Do you know me? You are guessing very wrong, I wrote all of that code my self.

OriginalGriff 17-Sep-21 11:04am

OK, Why?
Why go to all the effort of creating a Card class, and then ignore it in favour of what looks like Console based C code?

Diasalva5 17-Sep-21 11:13am

Because I am new in this world, I just started to learn how to code a few months ago, Mr. OriginalGriff, and I still don't have the amazing experience and "reflexes" and "the mindset" that you were born with, and to clarify: This code I did all my myself, in Console, it worked perfect, but then, I copied part of it and paste it to adapt it for making a GUI.

OriginalGriff 17-Sep-21 11:44am

Don't get snarky.
It's not a good look ... :D

"This code I did all my myself, in Console, it worked perfect, but then, I copied part of it and paste it to adapt it for making a GUI."
It shows. And that's why it looks like it was written by two different people.
Don't do that - or at least don't do that with anything even vaguely input / output related. The two environments are very different, and what works in one fails totally in the other, so it's difficult to move without producing really poor code.

Start with your Card class, and think about what you need for a game: a single deck of cards (shuffled), and two players (one user, and one Dealer - each has their own hand). Then you need inputs: New game, hit, stick (and optionally bet) buttons.

Then it's time to work out how to display all this. Me, I'd upgrade the Card class to a Control so it can display itself, add two Panels - one for each player - and as I deal a card I add it to the appropriate Panel. I'd probably use stock card images (to save myself the effort of graphics design): https://depositphotos.com/6084282/stock-photo-full-deck-of-playing-cards.html
for example.
But that's probably a bit more complex than you know how to do at this stage (although when you do, it's pretty easy) so start by thinking about what you do know how to do in a GUI, and how you can use that to display your game.
Have a think about it, and let me know what you come up with.

Diasalva5 17-Sep-21 12:07pm

Thank you for your guide, MR OriginalGriff, I will follow all of your advises, I am reading the Farrell C# (our text Book), too. You are right and I have realized that the Console is quite different from GUI. I started learn C# GUI first time in my life a couple of months ago. As soon as I come up with the final product, you will be first one to see it. I need to continue working on it, and I wont do that again, to copy and paste, not even my own worked code, is not a good way to learn coding. Thanks

OriginalGriff 17-Sep-21 12:35pm

You're welcome!
Good luck with it.

Diasalva5 23-Sep-21 11:21am

Mr. OrginalGriff, I keep my promised to show you my code when finished. Thanks

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
namespace GUI_BLACKJACK
{
public partial class Form1 : Form
{

public object DeckOfCards { get; private set; }

class Card
{

public string Name;
public int Value;
public string Suit;
public override string ToString() { return string.Format("{0} of {1}", Name, Suit); }

}

public Form1()
{
InitializeComponent();
}

private void Form1_Load(object sender, EventArgs e)
{

}

private void label2_Click(object sender, EventArgs e)
{

}

private void label1_Click(object sender, EventArgs e)
{

}

private void button1_Click(object sender, EventArgs e)
{

Random random = new Random();

int dealer = random.Next(1, 10);

int player = random.Next(1, 10);

label4.Text = ($"Let's play a modified version of the card game blackjack," +
$" The dealer is showing a {dealer}, the other card is hidden");

label1.Text = ("you were dealt 2 cards that total " + player);
}
private void label3_Click(object sender, EventArgs e)
{

}

private void button2_Click(object sender, EventArgs e)
{
label4.Text = ("Do you want another card? Type [Y]es or [N]o");

}

private void label6_Click(object sender, EventArgs e)
{

}

public string PlayGame(int dealer, int player)
{

do
{

string response = "";

if (player <= 21 && dealer > 21)
{
label1.Text = ($"You WON with {player}");
label2.Text = ($"The dealer ended with {dealer}");

}

else if (player < 21 && dealer < 21 && dealer > player)
{
label1.Text = ($"You lost with {player} ");
label2.Text = ($"The dealer ended with {dealer}");
}

else if (player < 21 && dealer < 21 && player > dealer)
{
label1.Text = ($"You WON with {player} ");
label2.Text = ($"The dealer ended with {dealer}");
}

else if (player > 21 && dealer < player)
{
label1.Text = ($"You lost with {player}");
label2.Text = ($"The dealer ended with {dealer}");
}
else if (player == 21 && dealer < player)
{
label1.Text = ($"You WON with {player}");
label2.Text = ($"The dealer ended with {dealer}");
}
else if (player > 21)
{
label1.Text = ($"You lost with {player}");
label2.Text = ($"The dealer ended with {dealer}");
}

{
return response;
}

}
while (dealer == player);

}
public void button3_Click(object sender, EventArgs e)
{

OriginalGriff 23-Sep-21 12:19pm

Well done!
Would you mind a few constructive criticisms?
First off, if a method is empty, remove it - including event handlers. No point in bloating the code with unused stuff!

Second, do yourself a favour and stop using Visual Studio default names for everything - you may remember that "TextBox8" is the mobile number today, but when you have to modify it in three weeks time, will you then? Use descriptive names - "tbMobileNo" for example - and your code becomes easier to read, more self documenting, easier to maintain - and surprisingly quicker to code because Intellisense can get to to "tbMobile" in three keystrokes, where "TextBox8" takes thinking about and 8 keystrokes...
Call it "button3" and I have no idea what it does without looking closely. As "butDealCardsToAllPlayers" it's very obvious. This applies to all controls, not just buttons!

Don't declare a Random as a local variable: declare one Random object as a private class level variable, and use that throughout. Each time you create an instance of Random, it is initialized from the system clock - so if two methods get executed close together and they both create their own Random, they can get the same sequence of numbers.

Why are button3 and button4 doing the same thing in different methods? the code is the same, so they do the same job ...

Diasalva5 1-Oct-21 14:25pm

Dear Mr.OriginalGriff

Thank you for your answer, I really appreciate it. Of course, that I accept your constructive criticism. You are welcome to give me any, YES, please!

1- You typed “stop using Visual Studio default names for everything” …"TextBox8 is the mobile number today, but when you have to modify it in three weeks’ time, will you then? Use descriptive names - "tbMobileNo" for example - and your code becomes easier to read, more self documenting, easier to maintain - and surprisingly quicker to code because Intellisense can get to to "tbMobile" in three keystrokes, where "TextBox8" takes thinking about and 8 keystrokes...
Call it "button3" and I have no idea what it does without looking closely. As "butDealCardsToAllPlayers" it's very obvious. This applies to all controls, not just buttons!”
Sorry, I kind of understand a little what you mean, but could you be more explanatory, please!! Explain it like I am a 6th grade boy, please. What do you mean with"TextBox8 is the mobile number today (?), but when you have to modify it in three weeks’ time, will you then?”, Please?
SORRY, please, I am not familiar with Software developer’s Slang. THANKS.
2-Additionally (Sorry, please) what do you mean when you typed “Don't declare a Random as a local variable: declare one Random object as a private class level variable, and use that throughout. Each time you create an instance of Random, it is initialized from the system clock - so if two methods get executed close together and they both create their own Random, they can get the same sequence of numbers.” Could you give me an example of this , please? thanks

OriginalGriff 1-Oct-21 14:48pm

1) Look at your code: it's full of "Form1", "Label2", "Button3" - the Visual Studio default names for controls when you drop them on a form, or create a new project.

Why? What does that tell me, the reader of your code about what the function is meant to be? What is supposed to be shown in "Label2" and how is that different from "Label1"? What do "Button1" and "Button2" do that "Button3" doesn't?
If when you drop a control you take ten seconds to give it a "sensible name" that describes what it does then a glance at the code is all it takes. And that has another big advantage: it's a lot more obvious when you type the wrong thing! If you see that you are displaying an error message in "Label6" that that's not obvious while displaying the same error in "TotalOwed" it is a lot more obvious that that's probably not what you wanted.
So your code becomes both more self documenting, and more reliable.

And think about "future you": in three months time when you come back to your app to fix bugs, or add features do you think you will remember what each control does, especially if you've knocked out half a dozen other apps all with "Label3", "Button16", ... in the mean time? Spending ten extra seconds to give it a proper name saves Future You loads of time later (as well as focussing your mind on what controls are for when you add them - if you can't think of a "good name" what is it doing there at all?)

2) Oh come on! What happens when you add the keyword private or public in to a variable declaration?
You know this stuff - you just aren't thinking about it yet. :D

Diasalva5 1-Oct-21 18:00pm

Thank you!

OriginalGriff 2-Oct-21 2:06am

You're welcome!

Diasalva5 9-Oct-21 11:08am

Dear Mr.OriginalGriff:
Good morning
I did the changes you advised me to do in my card war game, but my car game still have two issues: The 2 Players are just using the first 26 cards and they both continue using duplicate cards. Could you help me, please? Thanks

OriginalGriff 9-Oct-21 12:02pm

How do you expect me to be able to help you when I can't see your code?
:laugh:

What does the debugger show you is happening? Did you try debugging you code? How?

Diasalva5 9-Oct-21 13:41pm

Sorry, You right. I didn't send the code because I didn't want to bother you on taking much time analyzing it, You are probably busy or enjoying your weekend. Essentially, is working good, and at the end, eventually is giving a winner, but I realized that is just using the first 26 cards (Hearts and spades), and it repeats the cards. Anyway, I am sending it right away.Thanks.

OriginalGriff 9-Oct-21 13:48pm

You're right - I'm not wading through that lot, and I'm on a tablet right now so I can't compile it anyway.

So it's going to be up to you - what did the debugger show you was happening?

Diasalva5 9-Oct-21 13:56pm

Nothing, is working as expected (apparently), but is not using all the cards, just the first 26 cards of the deck and is repeating cards, I just need your advise for that and on Removing Duplicates from a List in C#. Thanks

OriginalGriff 9-Oct-21 14:39pm

That's what the debugger is *for*: finding out why your code doesn't do what you expected it to!
It lets you look at your code while it is running, view (and change) the content of variables, execute your code line by line, and even change the code while it's "paused" without having to recompile.

Getting your code to compile doesn't mean it's right: now you have to look at it and compare what your expected to happen with what actually did!

Start here:
https://docs.microsoft.com/en-us/visualstudio/get-started/csharp/tutorial-debugger?view=vs-2019
and start learning the most useful skill you will ever find - it'll teach you how to think logically about just about everything. (Don't try it on humans, though. It doesn't work on humans till you get to Level 99 ...) :D

Diasalva5 9-Oct-21 15:03pm

Thanks, I reading it right now! You are right, you talked to me about this before. Thanks.

OriginalGriff 9-Oct-21 15:05pm

You're welcome!

Diasalva5 9-Oct-21 13:44pm

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace Wargame2
{

public partial class Form1 : Form
{
class Card
{
public string Name;
public int Value;
public string Suit;
public override string ToString() { return string.Format("{0} of {1}", Name, Suit); }
public const int NCARDS = 52;

}
static List<card> DeckOfCard {get; set;}=new List<card>
{

new Card{Name=" two",Value=2,Suit=" ♣ Heart"},
new Card{Name=" three",Value=3,Suit=" ♣ Heart"},
new Card{Name=" four",Value=4,Suit=" ♣ Heart"},
new Card{Name="five", Value=5, Suit=" ♣ Heart"},
new Card{Name=" six",Value=6,Suit=" ♣ Heart"},
new Card{Name="seven",Value=7,Suit=" ♣ Heart"},
new Card{Name="eight",Value=8,Suit=" ♣ Heart"},
new Card { Name="Nine", Value=9, Suit=" ♣ heart"},
new Card{Name="ten",Value=10,Suit=" ♣ Heart"},
new Card{Name="jack",Value=11,Suit=" ♣ Heart"},
new Card{Name="queen",Value=12,Suit=" ♣ Heart"},
new Card { Name="king", Value=13, Suit=" ♣ heart"},
new Card { Name="Ace", Value=14, Suit=" ♣ Heart"},

new Card{Name="two",Value=2,Suit="Spades"},
new Card{Name="three",Value=3,Suit="Spades"},
new Card{Name="four",Value=4,Suit="Spades"},
new Card { Name="five", Value=5, Suit="Spades"},
new Card{Name="six",Value=6,Suit="Spades"},
new Card{Name="seven",Value=7,Suit="Spades"},
new Card{Name="eight",Value=8,Suit="Spades"},
new Card { Name="Nine", Value=9, Suit="Spades"},
new Card{Name="ten",Value=10,Suit="Spades"},
new Card{Name="jack",Value=11,Suit="Spades"},
new Card{Name="queen",Value=12,Suit="Spades"},
new Card { Name="king", Value=13, Suit="Spades"},
new Card { Name="Ace", Value=14, Suit="Spades"},

new Card{Name="two",Value=2,Suit="Diamonds"},
new Card{Name="three",Value=3,Suit="Diamonds"},
new Card{Name="four",Value=4,Suit="Diamonds"},
new Card { Name="five", Value=5, Suit="Diamonds"},
new Card{Name="six",Value=6,Suit="Diamonds"},
new Card{Name="seven",Value=7,Suit="Diamonds"},
new Card{Name="eight",Value=8,Suit="Diamonds"},
new Card { Name="Nine", Value=9, Suit="Diamonds"},
new Card{Name="ten",Value=10,Suit="Diamonds"},
new Card{Name="jack",Value=11,Suit="Diamonds"},
new Card{Name="queen",Value=12,Suit="Diamonds"},
new Card { Name="king", Value=13, Suit="Diamonds"},
new Card { Name="Ace", Value=14, Suit="Diamonds"},

new Card{Name="two",Value=2,Suit="Clubs"},
new Card{Name="three",Value=3,Suit="Clubs"},
new Card{Name="four",Value=4,Suit="Clubs"},
new Card { Name="five", Value=5, Suit="Clubs"},
new Card{Name="six",Value=6,Suit="Clubs"},
new Card{Name="seven",Value=7,Suit="Clubs"},
new Card{Name="eight",Value=8,Suit="Clubs"},
new Card { Name="Nine", Value=9, Suit="Clubs"},
new Card{Name="ten",Value=10,Suit="Clubs"},

Diasalva5 9-Oct-21 13:46pm

new Card{Name="jack",Value=11,Suit="Clubs"},
new Card{Name="queen",Value=12,Suit="Clubs"},
new Card { Name="king", Value=13, Suit="Clubs"},
new Card { Name=" ♣Ace", Value=14, Suit="Clubs"},

};

List<card> deckOfPlayer1 = new List<card>();
List<card> deckOfPlayer2 = new List<card>();

public Form1()
{

InitializeComponent();

int count = DeckOfCard.Count;

for (int i = 0; i < DeckOfCard.Count ; i ++)
{

label7.Text = ($" We have {DeckOfCard.Count} cards for this game, let's flip cards ");

}

Random random = new Random();
for (int i = 0; i < 26; i++)
{
//Card card = new Card();
//card = DeckOfCard[random.Next(26)];

deckOfPlayer1.Add(DeckOfCard[random.Next(26)]);
deckOfPlayer2.Add(DeckOfCard[random.Next(26)]);

label1.Text = ($"Player1 has {deckOfPlayer1.Count} ");
label2.Text = ($"Player2 has {deckOfPlayer2.Count}");
}

}

private void Form1_Load(object sender, EventArgs e)
{

}

private void label1_Click(object sender, EventArgs e)
{

}

private void label2_Click(object sender, EventArgs e)
{

}

private void button1_Click(object sender, EventArgs e)
{

label1.Text = "";
label2.Text = "";
label3.Text = "";
label4.Text = "";
label5.Text = "";
label7.Text = "";

// pick The TOP random card for player 1

Card c1 = deckOfPlayer1[0];

// pick The Top random card for player 2

Card c2 = deckOfPlayer2[0];

if (c1.Value > c2.Value)
{

label1.Text = ($"Player1 wins with {c1} as the top card");
label2.Text = ($"Player2 lost with {c2} as the top card");
deckOfPlayer1.Add(c2);
deckOfPlayer2.Remove(c2);
deckOfPlayer1.Add(c1);
deckOfPlayer1.Remove(c1);
label3.Text = ($"Player1 has {deckOfPlayer1.Count} ");
label4.Text = ($"Player2 has {deckOfPlayer2.Count}");
}
else if (c2.Value > c1.Value)
{
label2.Text = ($"Player2 wins with {c2} as the top card the top card");
label1.Text = ($"Player1 lost with {c1} as the top card the top card");
deckOfPlayer2.Add(c1);
deckOfPlayer1.Remove(c1);
deckOfPlayer2.Add(c2);
deckOfPlayer2.Remove(c2);
label3.Text = ($"Player1 has {deckOfPlayer1.Count} ");
label4.Text = ($"Player2 has {deckOfPlayer2.Count}");
}

else if (c2.Value == c1.Value)
{

label1.Text = ($"Player1 has {deckOfPlayer1[0].Name}{deckOfPlayer1[0].Suit} {deckOfPlayer1[0].Value}");
label2.Text = ($"Player2 has {deckOfPlayer2[0].Name}{deckOfPlayer2[0].Suit} {deckOfPlayer2[0].Value}");

label5.Text = ("Its a tie! Next three cards in deck are played face down, " +
"4th card face up, whoever has the highest card wins");

if (deckOfPlayer1.Count < 4 && deckOfPlayer2.Count < 4)
{
button1.Enabled = false;
}

/// pick The

Diasalva5 9-Oct-21 13:52pm

I have spent days checking codes websites and I have found some advises, but none of them seems to be useful for my code. Thanks
/// pick The fourth random card for player 1

Card c3 = deckOfPlayer1[3];

// pick The fourth random card for player 2

Card c4 = deckOfPlayer2[3];

if (c3.Value > c4.Value )
{

deckOfPlayer1.Add(c2);
deckOfPlayer2.Remove(c2);
deckOfPlayer1.Add(c1);
deckOfPlayer1.Remove(c1);

Card card1 = deckOfPlayer1[1];
deckOfPlayer1.Add(card1);
Card card2 = deckOfPlayer1[2];
deckOfPlayer1.Add(card2);
Card card3 = deckOfPlayer1[3];
deckOfPlayer1.Add(card3);

deckOfPlayer2.Remove(card1);
deckOfPlayer2.Remove(card2);
deckOfPlayer2.Remove(card3);

}
else
{

deckOfPlayer2.Add(c1);
deckOfPlayer1.Remove(c1);
deckOfPlayer1.Add(c2);
deckOfPlayer2.Remove(c2);

Card card1 = deckOfPlayer1[1];
deckOfPlayer1.Add(card1);
Card card2 = deckOfPlayer1[2];
deckOfPlayer1.Add(card2);
Card card3 = deckOfPlayer1[3];
deckOfPlayer1.Add(card3);

deckOfPlayer2.Remove(card1);
deckOfPlayer2.Remove(card2);
deckOfPlayer2.Remove(card3);
}

}
if (deckOfPlayer1.Count == 52)
{
label6.Text = ($"Player1 is the Final Winner with {deckOfPlayer1.Count} cards ");
button1.Enabled = false;
}
if (deckOfPlayer2.Count == 52)
{
label6.Text = ($"Player2 is the Final Winner with {deckOfPlayer2.Count} cards ");
button1.Enabled = false;
}

}

Diasalva5 23-Sep-21 11:24am

This is the continuation of the code. Space issues.

}
while (dealer == player);

}
public void button3_Click(object sender, EventArgs e)
{
Random random = new Random();
int dealer = random.Next(1, 10);

int player = random.Next(1, 10);

int randomNumber = random.Next(1, 10);
player += randomNumber;

PlayGame(dealer, player);

}

private void button4_Click(object sender, EventArgs e)
{

Random random = new Random();
int dealer = random.Next(1, 10);

int player = random.Next(1, 10);

int randomNumber = random.Next(1, 10);
player += randomNumber;

PlayGame(dealer, player);
}

private void label2_Click_1(object sender, EventArgs e)
{

}

private void button5_Click(object sender, EventArgs e)
{
label1.Text = "";
label2.Text = "";
label3.Text = "";
label4.Text = "";
}

private void button6_Click(object sender, EventArgs e)
{
List<card> DeckOfCards = new List<card>()
{

new Card{Name="two",Value=2,Suit="Heart"},
new Card{Name="three",Value=3,Suit="Heart"},
new Card{Name="four",Value=4,Suit="Heart"},
new Card{Name = "five", Value = 5, Suit = "Heart" },
new Card{Name = "six", Value = 6, Suit = "Heart" },
new Card{Name = "seven", Value = 7, Suit = "Heart" },
new Card{Name = "eight", Value = 8, Suit = "Heart" },
new Card{Name = "Nine", Value = 9, Suit = "heart" },
new Card{Name = "ten", Value = 10, Suit = "Heart" },

new Card { Name = "two", Value = 2, Suit = "Spades" },
new Card { Name = "three", Value = 3, Suit = "Spades" },
new Card { Name = "four", Value = 4, Suit = "Spades" },
new Card { Name = "five", Value = 5, Suit = "Spades" },
new Card { Name = "six", Value = 6, Suit = "Spades" },
new Card { Name = "seven", Value = 7, Suit = "Spades" },
new Card { Name = "eight", Value = 8, Suit = "Spades" },
new Card { Name = "Nine", Value = 9, Suit = "Spades" },
new Card { Name = "ten", Value = 10, Suit = "Spades" },

new Card { Name = "two", Value = 2, Suit = "Diamonds" },
new Card { Name = "three", Value = 3, Suit = "Diamonds" },
new Card { Name = "four", Value = 4, Suit = "Diamonds" },
new Card { Name = "five", Value = 5, Suit = "Diamonds" },
new Card { Name = "six", Value = 6, Suit = "Diamonds" },
new Card { Name = "seven", Value = 7, Suit = "Diamonds" },
new Card { Name = "eight", Value = 8, Suit = "Diamonds" },
new Card { Name = "Nine", Value = 9, Suit = "Diamonds" },
new Card { Name = "ten", Value = 10, Suit = "Diamonds" },

new Card { Name = "two", Value = 2, Suit = "Clubs" },
new Card { Name = "three", Value = 3, Suit = "Clubs" },
new Card { Name = "four", Value = 4, Suit = "Clubs" },
new Card { Name = "five", Value = 5, Suit = "Clubs" },
new Card { Name = "six", Value = 6, Suit = "Clubs" },
new Card { Name = "seven", Value = 7, Suit = "Clubs" },
new Car

How to output cards in C#?

1 solution

Solution 1

Add your solution here

Preview 0

How to output cards in C#?

1 solution

Solution 1

Add your solution here

Preview 0

Existing Members

...or Join us