Does not Exist Issue

Question

0.00/5 (No votes)

See more:

So int count = (int)button.tag is saying button does not exist because button is being set to a variable outside of its scope. I have tried numerous things to fix this because I can not place it inside the button_click since that would mess up this count.
Any ideas what to do?

C#

public MapForm(String rows, String cols)
{
    InitializeComponent();
     _col = int.Parse(cols);
    _row = int.Parse(rows);
    Begin();
}

public void Begin()
{
    int width = groupBox1.Width;
    int height = groupBox1.Height;
    int bW = width / _col;
    int bH = height / _row;

    Button[,] buttonArray = new Button[_row, _col];

    for (int i = 0; i < _row; i++)
    {
        for (int j = 0; j < _col; j++)
        {
            Button button = new Button();
            buttonArray[i, j] = button;
            button.Width = bW;
            button.Height = bH;
            button.Left = j * bW;
            button.Top = i * bH;
            button.Tag = 0;

            button.Click += new EventHandler(button_Click);

            groupBox1.Controls.Add(button);
        }
     }
}

int count = (int)button.Tag;

void button_Click(object sender, EventArgs e)
{
    Button button = (Button)sender;

    if (count < 3)
    {
        count++;

        if (count == 1)
        {
            button.Text = "a";
        }

        if (count == 2)
        {
            button.Text = "b";
        }
        if (count == 3)
        {
            button.Text = "c";
        }
    }
}

Posted 22-Mar-14 10:42am

angelandbuffy9

Updated 22-Mar-14 10:45am

André Kraak

v2

Add a Solution

Comments

Kornfeld Eliyahu Peter 22-Mar-14 16:54pm

That part of 'int count' does not make any sense...
Can you explain why it's there?

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)

Marcin Kozub · Answer 1 · 2014-03-22T10:53:00

Move your int count = (int)button.Tag;... line lower:

C#

void button_Click(object sender, EventArgs e)
{
Button button = (Button)sender;
int count = (int)button.Tag;
if (count < 3)

[UPDATED]

C#

void button_Click(object sender, EventArgs e)
{
    Button button = (Button)sender;
    int count = (int)button.Tag;

    if (count < 3)
    {
        count++;

        if (count == 1)
        {
            button.Text = "a";
        }

        if (count == 2)
        {
            button.Text = "b";
        }
        if (count == 3)
        {
            button.Text = "c";
        }
        button.Tag = count; // You need to store value of COUNT to your Button's TAG property
    }
}

I hope it helps you.

angelandbuffy9 · Answer 2 · 2014-03-22T12:34:00

Solution 2

But if I do that the button text is always a

Posted 22-Mar-14 12:34pm

angelandbuffy9

Comments

Marcin Kozub 22-Mar-14 19:08pm

It's very simple. When you're creating your buttons dynamically, you set theirs Tag property to 0:
...
button.Top = i * bH;
button.Tag = 0;
...

Next in your ButtonClick event handler you've got code:
if (count < 3)
{
count++;
if (count == 1)
{
button.Text = "a";
}

Your first condition will be true (if count < 3) because count = button.Tag which is initialized with 0.
Next you're incrementing value of count by 1 (count++). So, your count value will be 1. That's why you are getting button.Text = "a"...

I think that you want to store back your count value to button's Tag property. I've updated my answer.

angelandbuffy9 22-Mar-14 19:28pm

Thank you!! :)

Marcin Kozub 22-Mar-14 20:44pm

If I helped you, it would be nice, if you accept my solution.