Which is best?

[esd]

Hello folks, the two code blocks below do the same thing, which is set a few form field values. Clearly the first method is the most elegant but my question is which is best in terms of lightest server loading and efficiency? Is looping through collections more time and memory consuming then simply dropping through a switch-block?

METHOD 1:

foreach (Control c in this.pnlRows.Controls)
{
if (c is RadioButtonList && ((RadioButtonList)c).ID.ToString().Contains(RowsCt rlID))
{
((RadioButtonList)c).Items.FindByValue(RowsRequire d).Selected = true;
}
if (c is TextBox && ((TextBox)c).ID.Contains(RowsCtrlID))
{
((TextBox)c).Text = RowsItemText;
}
}

================================================== ========================

METHOD 2:


switch (RowsCtrlID)
{
case "fRowText1":
fRowText1.Text = RowsItemText;
fRowText1Required.Items.FindByValue(RowsRequired). Selected = true;
break;

case "fRowText2":
fRowText2.Text = RowsItemText;
fRowText2Required.Items.FindByValue(RowsRequired). Selected = true;
break;

case "fRowText3":
fRowText3.Text = RowsItemText;
fRowText3Required.Items.FindByValue(RowsRequired). Selected = true;
break;

case "fRowText4":
fRowText4.Text = RowsItemText;
fRowText4Required.Items.FindByValue(RowsRequired). Selected = true;
break;

case "fRowText5":
fRowText5.Text = RowsItemText;
fRowText5Required.Items.FindByValue(RowsRequired). Selected = true;
break;

case "fRowText6":
fRowText6.Text = RowsItemText;
fRowText6Required.Items.FindByValue(RowsRequired). Selected = true;
break;

case "fRowText7":
fRowText7.Text = RowsItemText;
fRowText7Required.Items.FindByValue(RowsRequired). Selected = true;
break;

case "fRowText8":
fRowText8.Text = RowsItemText;
fRowText8Required.Items.FindByValue(RowsRequired). Selected = true;
break;

case "fRowText9":
fRowText9.Text = RowsItemText;
fRowText9Required.Items.FindByValue(RowsRequired). Selected = true;
break;

case "fRowText_10":
fRowText_10.Text = RowsItemText;
fRowText_10Required.Items.FindByValue(RowsRequired ).Selected = true;
break;

case "fRowText_11":
fRowText_11.Text = RowsItemText;
fRowText_11Required.Items.FindByValue(RowsRequired ).Selected = true;
break;

case "fRowText_12":
fRowText_12.Text = RowsItemText;
fRowText_12Required.Items.FindByValue(RowsRequired ).Selected = true;
break;

case "fRowText_13":
fRowText_13.Text = RowsItemText;
fRowText_13Required.Items.FindByValue(RowsRequired ).Selected = true;
break;

case "fRowText_14":
fRowText_14.Text = RowsItemText;
fRowText_14Required.Items.FindByValue(RowsRequired ).Selected = true;
break;

case "fRowText_15":
fRowText_15.Text = RowsItemText;
fRowText_15Required.Items.FindByValue(RowsRequired ).Selected = true;
break;

case "fRowText_16":
fRowText_16.Text = RowsItemText;
fRowText_16Required.Items.FindByValue(RowsRequired ).Selected = true;
break;

default:
break;
}

[Plater]

I would say the first one is better overall. The complier can probably optimize its own functions much better.
PLUS: You don't really use the value of RowsCtrlID in the switch statement very effectively. A lot of extra code is compiled for it.