C# Флажки Оптимизация

Question

У меня есть панель с 5 флажками, по одному на каждый будний день. Пользователь в этом сценарии бронирует встречи на неделю. У меня есть Еженедельная книга . Этот класс имеет пять классов: понедельник, вторник, среда, четверг, пятница. По умолчанию все bools установлены в false. Когда флажок установлен, соответствующий bool становится истинным.

        private void CheckCBs(WeeklyBooking week)
    {
        if (MondayCB.Checked)
        {
            week.monday = true;
        }
        if (TuesdayCB.Checked)
        {
            week.tuesday = true;
        }
        if (WednesdayCB.Checked)
        {
            week.wednesday = true;
        }
        if (ThursdayCB.Checked)
        {
            week.thursday = true;
        }
        if (FridayCB.Checked)
        {
            week.friday = true;
        }
        return;
    }

Это метод, который я вызываю в другом методе, пока создаю объект для помещения в таблицу SQL. Есть ли у кого-нибудь совет, как оптимизировать этот метод? - Спасибо.

Answer 1

В операторах if() нет необходимости. Свойство Checked ComboBox уже является bool и будет ложным, если флажок снят. Просто присвойте значение свойства Checked вашим WeeklyBooking свойствам объекта:

private void CheckCBs(WeeklyBooking week)
{
    week.monday = MondayCb.Checked;
    week.tuesday = TuesdayCb.Checked;
    week.wednesday = WednesdayCb.Checked;
    week.thursday = ThursdayCb.Checked;
    week.friday= FridayCb.Checked;
}