C# how to check if chess pieces are contact?

Question

0.00/5 (No votes)

See more:

for example i have only two pieces`pawn and bishop,i need to check that two pieces located on the same line,it's mean that bishop can hit pawn, then when i click on button Check, i need All the diagonal drawers available for the bishop to be painted in red.

What I have tried:

private void Form1_Paint(object sender, PaintEventArgs e)
        {
            for (int i = 0; i < 8; i++)
            {
                for (int j = 0; j < 8; j++)
                {
                    if((i+j) % 2 != 0)
                    gf.FillRectangle(Brushes.Black, (j + 1) * 50, (i+1)*50, 50, 50);
                }
                Label lb = new Label();
                lb.Text = Convert.ToChar('A' + i).ToString();
                lb.Font = new Font("Tahoma", 18);
                lb.Location = new Point(50 * (i + 1), 450);
                lb.Width = 50;
                lb.Height = 26;
                this.Controls.Add(lb);
            }
            for (int i = 8; i > 0; i--)
            {
                Label lab = new Label();
                lab.Text = i.ToString();
                lab.Font = new Font("Tahoma", 18);
                lab.Location = new Point(0, 50 *(9- i));
                lab.Width = 50;
                lab.Height = 26;
                this.Controls.Add(lab);
            }
        }


 private void Create_Click_1(object sender, EventArgs e)
        {
           
            if (comboBox1.SelectedItem == "bishop")
            {
                string position = textBox1.Text;
                string first_letter = position[0].ToString().ToUpper();
                int x = (int)(first_letter[0] - 64);
                int y = int.Parse(position[1].ToString());
                gf.DrawImage(Properties.Resources.bishop, x * 50, (8 - y + 1) * 50, 50, 50);
            }
            else if(comboBox1.SelectedItem == "pawn")
            {
                string position = textBox1.Text;
                string first_letter = position[0].ToString().ToUpper();
                int x = (int)(first_letter[0] - 64);
                int y = int.Parse(position[1].ToString());
                gf.DrawImage(Properties.Resources.pawn, x * 50, (8 - y + 1) * 50, 40, 50);
            }
        }


 private void button2_Click(object sender, EventArgs e)
        {
           //This is my Check button,How can i check it? what should i write here?
        }

Posted 6-Mar-18 21:09pm

Suren97

Updated 6-Mar-18 21:49pm

Add a Solution

Comments

Richard MacCutchan 7-Mar-18 5:01am

You are still doing it wrong. Learn how to create proper classes and use inheritance and common methods. Until you learn how to write proper C# code you are not going to make any progress with this.

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

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

OriginalGriff · Answer 1 · 2018-03-06T21:44:00

We can't just solve every problem for you!
You haven't appeared to implement a board, which you need to do in order to do this task, as well as the last one: C# simple chess error[^]

I would strongly suggest that you approach this by creating an abstract Piece class which has a location, and a "side" (Black or White) and a Board class that encapsulates a 2D array of Piece instances.
Then declare your Pawn class as a Piece, your Rook as a Piece, and so on.
You can then start delegating tasks to the classes.
By adding an abstract ShowMoves method to the Piece class and passing it a Board you can get the actual piece to generate the legal moves from its current position - and use that to generate a "map" of where to draw red squares.
Similarly, you delegate the Paint task to the Board - passing it the graphics context - and it delegates actual piece painting to the Piece class through another abstract method.

What you appear to be doing at the moment is jumping into code without actually planning what you need - and that doesn't generate a good, solid application at the end, it generates a hard-to-maintain mess that is riddled with bugs.

Maciej Los · Answer 2 · 2018-03-06T21:50:00

All wrong! You've been talking about this with Richard MacCutchan[^], especially here: C# simple chess error[^].

The conclusion from above discussion is: you should work on data, not on its string representation!

I'd strongly recommend to take a look at "working example": Chess Program in C#[^]. That's the way you have to do this.

More examples you'll find on: CodeProject's KnowledgeBase[^]

C# how to check if chess pieces are contact?

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0