Clicking a Button in a button array

Question

I have the code below.

public void WepaonEquip(Object sender, System.EventArgs e)
{
if (button[0].BackColor == Color.Beige)
{
    button[0].BackColor = Color.OrangeRed;
}

else if (button[1].BackColor == Color.Beige)
{
    button[1].BackColor = Color.OrangeRed;
}

else if (button[2].BackColor == Color.Beige)
{
    button[2].BackColor = Color.OrangeRed;
}
}

The code in the class containing this chunk of code generates a button array. What I want is that the user will click a button and the colour of the button clicked will change.

However, when the user clicks, lets say, the 3rd button, the first button in the array changes colour, not the one clicked. Any idea as to why this isn't working? I believe the logic of the code works, perhaps I'm missing something.

tinstaafl · Accepted Answer · 2014-01-21 03:09:18Z

3

Set each button in the panel to use the same Click Event handler. In the handler cast sender as a button and change the color

Assuming that WeaponEquip is the click event handler for the buttons it would look something like this:

public void WepaonEquip(Object sender, System.EventArgs e)
{
    Button clickedbutton = (Button)sender
    clickedbutton.BackColor = Color.OrangeRed;

}

edited Jan 21, 2014 at 3:09

answered Jan 21, 2014 at 2:44

tinstaafl

6,9782 gold badges17 silver badges22 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

user3170251 Over a year ago

sorry, panel1 is irrelevent, ill take that off

tinstaafl Over a year ago

That's fine, the concept will still work with whichever buttons you want to handle.

user3170251 Over a year ago

alright, i added them into one click event, but nothing changed, see above for edited code

Mark Hall Over a year ago

@user3170251 You do not need to use conditional logic to test for every button, the sender is the button that was clicked, just do what tinstaafl said and cast it to a button then test your color.

Collectives™ on Stack Overflow

Clicking a Button in a button array

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related