How to simplify code using a loop?

Question

How can I simplify the code below using a loop, I’m in doubt depending on the name of Labels which are sequential, as I put something variable in Fieldname1, NomeCampo2, ..., NomeCampoX.

lblNomeCampo1.Text = dt.Rows[1][0].ToString()+":";
lblCampo1.Text = dt.Rows[1][1].ToString();
lblNomeCampo2.Text = dt.Rows[2][0].ToString() + ":";
lblCampo2.Text = dt.Rows[2][1].ToString();
lblNomeCampo3.Text = dt.Rows[3][0].ToString() + ":";
lblCampo3.Text = dt.Rows[3][1].ToString();
lblNomeCampo4.Text = dt.Rows[4][0].ToString() + ":";
lblCampo4.Text = dt.Rows[4][1].ToString();
lblNomeCampo5.Text = dt.Rows[5][0].ToString() + ":";
lblCampo5.Text = dt.Rows[5][1].ToString();
lblNomeCampo6.Text = dt.Rows[6][0].ToString() + ":";
lblCampo6.Text = dt.Rows[6][1].ToString();
lblNomeCampo7.Text = dt.Rows[7][0].ToString() + ":";
lblCampo7.Text = dt.Rows[7][1].ToString();
lblNomeCampo8.Text = dt.Rows[8][0].ToString() + ":";
lblCampo8.Text = dt.Rows[8][1].ToString();
lblNomeCampo9.Text = dt.Rows[9][0].ToString() + ":";
lblCampo9.Text = dt.Rows[9][1].ToString();
lblNomeCampo10.Text = dt.Rows[10][0].ToString() + ":";
lblCampo10.Text = dt.Rows[10][1].ToString();

Answer 1

Essentially it doesn’t work this way. I’d give with reflection, but I don’t think it pays.

You can restructure the form that handles this by making the fields elements of array also, then everything involving them can be used with a loop. Then you could do so:

lblNomeCampo[i].Text = dt.Rows[i][0].ToString() + ":";
lblCampo[i].Text = dt.Rows[i][1].ToString();

I put in the Github for future reference.

Every time you have several variables with names that have a sequence of numbers as suffix you can replace with a array (or similar structure) in a simple and direct way, that is, just change the fixed number by the index of the array.

Obviously you have to create the array and create the screen objects on top of the elements of array.

Each element of array is a variable like any other, nothing changes in it except the fact that it is accessed by the set of the name plus the index. So treat them this way.

This solution is independent of language, although it would be better to know what it is (from what I understand is C#).

Answer 2

It’s not the best way but I could do something like this, I don’t know much about C#, but the way is more or less this:

for (var i=1; i <= 10; i++) {
   Eval("lblNomeCampo"+i+".Text = dt.Rows["+i+"][0].ToString() + \":\";");
   Eval("lblCampo"+i+".Text = dt.Rows["+i+"][1].ToString() + \":\";");
}

I suggest something more elaborate, like:

   foreach (int i in dt.Rows) {
       lblNomeCampo[i].Text = dt.Rows[i][0].ToString() + ":";
       lblCampo[i].Text = dt.Rows[i][1].ToString() + ":";
   }

Leave the fields in the form like this:

name="lblNomeCampo[]"

name="lblCampo[]"