Aspnetcore: for loop only render last value

[lakani]

i've modified a littel the counter page to try how i can render array of buttons, but i got a strange behavior
@page "/counter"

Counter

`

<p>Current count: @currentCount</p>

<button class="btn btn-primary" onclick="@IncrementCount">Click me</button>

<button class="btn btn-primary" onclick="@Reset">Reset</button>

@for (int i = 0; i < _Sarr.Length; i++) {
<button class="btn btn-primary" onclick="@(e => bReset( i ))">@_Sarr[i]</button>
}

@functions {
int currentCount = 0;

string[] _Sarr = new string[] { "one", "two", "three" };

void IncrementCount()
{
    currentCount++;
}

void bReset(int s)
{
    Console.WriteLine(s.ToString());
    currentCount += s;


}

void Reset()
{
    currentCount = 0;
}

}
`

but whenever i click on one of the buttons in the array , s variable passed by 3

am i doing someting wrong ?

[lakani]

Counter.zip

[Andrzej-W]

Change your loop to something similar to this:

@for (int i = 0; i < _Sarr.Length; i++) {
var local_i = i;
<button class="btn btn-primary" onclick="@(e => bReset( local_i ))">@_Sarr[local_i]</button>
}

This is standard C# behaviour where lambda expression @(e => bReset( local_i )) has access to variable and not to the value of the variable. You have to have variable which is local in for loop, otherwise your lambda expression always calls bReset(i) and i is equal 3 at the end of the loop.

[danroth27]

@Andrzej-W Thanks for answering!

[arivera12]

@Andrzej-W This worked for me as well thanks!