Address Form checkbox methods programatically?

Hello.

That's the name of the program I probably should have started C# with. Instead, I've built a small application that identifies musical keys from individual notes through the application of falsification offsets from identified notes.

The program works, but I've got a dozen tickbox methods that all do exactly the same thing save for

1. A single numerical value which informs the offset falsifier method
2. The name of the checkbox control, which is used to link the boolean value of the truthtable to the checkbox value, and provide forward and backwards functionality (i.e., it sets true on check, and false on uncheck).

There has to be a better way to do this. Ideally, I'd be able to parameterize the control name, and the passed num value. Even more ideally, I would be able to handle all of the tickboxes with a single method, with the linked values bound to the control state rather than repeating myself a dozen times.

Or, just as likely, there is some solution that simply escapes my imagination.

The values in question are highlighted below.

Your input is appreciated.

//Is there a way to handle all of the checkboxes with a single entry?
//Perhaps a way to bind a numerical value to each tickbox,
//which would be passed to a handler method?
private void CTick_CheckedChanged(object sender, EventArgs e)
{
byte num = 0;
Roots[num] = CTick.Checked; //C is Checked, Roots[0] = True, else False
KillMaj(); //Eliminate impossible keys
DisplayKeys(); //Update display
}
//Same thing 11 more times
private void CSharpTick_CheckedChanged(object sender, EventArgs e)
{
byte num = 1;
Roots[num] = CSharpTick.Checked;
KillMaj();
DisplayKeys();
}
private void DTick_CheckedChanged(object sender, EventArgs e)
{
byte num = 2;
Roots[num] = DTick.Checked;
KillMaj();
DisplayKeys();
}
[... you get the idea]

Someone at another site came up with using the Checkbox.Tag property to bind the data to the control, and another bit of fancy footwork to replace 12 event handlers with a single handler.

namespace Key_Finder
{
public partial class Form1 : Form
{
[...]
public Form1()
{
InitializeComponent();
Shown += TickWatch;
}
private void TickWatch(object sender, EventArgs e)
{
Controls.OfType()
.ToList()
.ForEach(cb =>
{
cb.CheckedChanged += CheckedChangedEvent;
});
}
private void CheckedChangedEvent(object sender, EventArgs e)
{
var checkBox = (CheckBox)sender;
var currentIndexer = Convert.ToByte((checkBox).Tag.ToString());
DoTickEvent(currentIndexer);
}
private void DoTickEvent(byte NoteNum)
{
Roots[NoteNum] ^= true;
KillMaj();
DisplayKeys();
}
}
}

1 Reply

Know the answer? Post it — somebody with the same question will find it here.

Alan Avery7y ago
Someone at another site came up with using the Checkbox.Tag property to bind the data to the control, and another bit of fancy footwork to replace 12 event handlers with a single handler.
namespace Key_Finder
{
    public partial class Form1 : Form
    {
        [...]
        public Form1()
        {
            InitializeComponent();
            Shown += TickWatch;
        }

        private void TickWatch(object sender, EventArgs e)
        {
            Controls.OfType()
                .ToList()
                .ForEach(cb =>
                {
                    cb.CheckedChanged += CheckedChangedEvent;
                });
        }

        private void CheckedChangedEvent(object sender, EventArgs e)
        {
            var checkBox = (CheckBox)sender;
            var currentIndexer = Convert.ToByte((checkBox).Tag.ToString());
            DoTickEvent(currentIndexer);
        }

        private void DoTickEvent(byte NoteNum)
        {
            Roots[NoteNum] ^= true;
            KillMaj();
            DisplayKeys();
        }
    }
}
0