Здравствуйте.
С c#, и особенно winforms, знаком на достаточно поверхностном уровне, поэтому хочу спросить вашего совета и разъяснений.
Есть форма, на которой отображаются различные данные об объекте. При этом используются разные типы компонентов: TextBox
, DateTimePicker
и RadioButton
. Рядом с каждым компонентом есть Label, в котором отображается название поля. Просто как пример: Имя
, Фамилия
, Дата рождения
, Пол
и др. При открытии формы данные берутся с какого-то источника и цель в том, чтобы поля, которые не заполнены в базе, подсвечивались другим цветом (не сами поля, а метки рядом с ними), а при изменении данных, выделение цветом убиралось.
Сейчас я это реализовал через поле Tag
в компонентах - в нём я указываю название компонента Label
, в котором нужно изменять цвет. Далее, в событиях изменения данных (нажатие мышью на RadioButton
, нажатие клавиши в TextBox
и т.д.), вызываются функции:
private void RecordChanged(object sender, KeyEventArgs e)
{
ChangeColor(ref sender);
}
private void DataChanged(object sender, MouseEventArgs e)
{
ChangeColor(ref sender);
}
private void DataChanged(object sender, EventArgs e)
{
ChangeColor(ref sender);
}
Здесь первый вопрос: можно ли избавиться от такого зоопарка функций, используя только одну?
Дальше по поводу самой функции ChangeColor
. В неё передаётся объект sender, из которого я получаю название типа компонента и вытаскиваю из компонента значение Tag:
var t = sender.GetType();
string tag = "";
dynamic q;
switch (t.Name)
{
case "RadioButton":
q = (RadioButton) sender;
tag = q.Tag.ToString();
break;
case "TextBox":
q = (TextBox) sender;
tag = q.Tag.ToString();
break;
case "DateTimePicker":
q = (DateTimePicker) sender;
tag = q.Tag.ToString();
break;
}
Здесь второй вопрос: можно ли реализовать это более красиво? Можно ли вытянуть данные с объекта без его распаковки?
Следом идёт перебор всех компонентов на форме:
// Перебор всех компонентов на форме
for (var i = 0; i < Controls.Count; ++i)
{
if (Controls[i].GetType() == typeof (Label))
{
var n = Controls[i].Name;
if (n == tag)
Controls[i].ForeColor = changed;
}
// перебор всех компонентов в GroupBox'ах
else if (Controls[i].GetType() == typeof (GroupBox))
{
for (var j = 0; j < (Controls[i]).Controls.Count; ++j)
{
if ((Controls[i]).Controls[j].GetType() == typeof (Label))
{
if ((Controls[i]).Controls[j].Name == tag)
(Controls[i]).Controls[j].ForeColor = changed;
}
}
}
}
Когда пытался сделать перебор через foreach (var comp in Controls)
, в нём comp является объектом. Опять же, возвращаясь к уже заданному вопросу - как распаковать этот объект (или как вытягивать из него данные, не распаковывая)?
И ещё один вопрос: можно ли как-то более красиво находить нужные компоненты на форме? Может быть, LINQ или ещё как-нибудь?
А может быть, есть какие-то совсем другие способы решения данной задачи, более простые?
Спасибо за внимание.
.NET Framework 4, VS2010