I have the following code which uses four if statements to check the value in four different combo boxes.
if (!String.IsNullOrEmpty(tx_cmb_Circuits1.Text.Trim()))
emp.CircuitEmail_Group_Rels.Add(
new Circuit_Email_Group_Rel
{
Circuits = (Circuits) tx_cmb_Circuits1.SelectedItem,
Group = emp
});
if (!String.IsNullOrEmpty(tx_cmb_Circuits2.Text.Trim()))
emp.CircuitEmail_Group_Rels.Add(
new Circuit_Email_Group_Rel
{
Circuits = (Circuits) tx_cmb_Circuits2.SelectedItem,
Group = emp
});
if (!String.IsNullOrEmpty(tx_cmb_Circuits3.Text.Trim()))
emp.CircuitEmail_Group_Rels.Add(
new Circuit_Email_Group_Rel
{
Circuits = (Circuits) tx_cmb_Circuits3.SelectedItem,
Group = emp
});
if (!String.IsNullOrEmpty(tx_cmb_Circuits4.Text.Trim()))
emp.CircuitEmail_Group_Rels.Add(
new Circuit_Email_Group_Rel
{
Circuits = (Circuits) tx_cmb_Circuits4.SelectedItem,
Group = emp
});
Is there are shorter way to do the same. Please review my code.
String.IsNullOrWhiteSpace
method instead ofIsNullOrEmpty
andTrim
. \$\endgroup\$