Anonymous C# delegate within a loop [duplicate]

Question

Hi all I am trying to write and anonymous delegate. as the integer variable is shared among the delegate I need it to be the local instance of every delegate such that rs[0] always gets nics[0], rs[1] always gets nics[1] and so on... how will I achieve this.

for (int i = 0; i < nics.Count; i++)
{
    rs[i] = new RollingSeries(monitor, new RollingSeries.NextValueDelegate(delegate()
    {
        return GetNetworkUtilization(nics[i]);
    }));
}

Be extremely careful about what you ask for. Do you want your first closure to always get the value in nics[0] at the time the delegate is invoked, or do you want the closure to get the value that happened to be in nics[0] at the time the delegate was created? Those are two different things; nics[0] is a variable, so its value can change over time. Do you want the closure to see that change, or ignore that change? — Eric Lippert
– Eric Lippert, Commented Nov 18, 2009 at 16:34

bruno conde · Accepted Answer · 2009-11-18 16:14:11Z

23

Make a local copy of i:

   for (int i = 0; i < nics.Count; i++)
   {
         int j = i;
         rs[i] = new RollingSeries(monitor, new RollingSeries.NextValueDelegate(delegate()
         {
            return GetNetworkUtilization(nics[j]);
          }));
    }

The Beauty of Closures

answered Nov 18, 2009 at 16:14

bruno conde

48.3k15 gold badges103 silver badges118 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Dykam Over a year ago

Or an local copy of the item in the array.

JaredPar · Accepted Answer · 2009-11-18 16:14:24Z

6

Use a local to get a different value per iteration

for (int i = 0; i < nics.Count; i++)
   {
         int localI = i;
         rs[i] = new RollingSeries(monitor, new RollingSeries.NextValueDelegate(delegate()
         {
            return GetNetworkUtilization(nics[localI]);
          }));
    }

answered Nov 18, 2009 at 16:14

JaredPar

759k152 gold badges1.3k silver badges1.5k bronze badges

Comments

Heinzi · Accepted Answer · 2009-11-18 16:24:39Z

4

Put int j = i inside your loop and refer to j within the lambda expression.

If you are curious about why this happens, here is an MSDN blog entry containing a detailed technical explanation: Closing over the loop variable considered harmful

edited Nov 18, 2009 at 16:24

answered Nov 18, 2009 at 16:14

Heinzi

173k61 gold badges386 silver badges554 bronze badges

Collectives™ on Stack Overflow

Anonymous C# delegate within a loop [duplicate]

3 Answers 3

1 Comment

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Linked

Related