循环中使用Timer时计数器未正确递增问题排查

一、异常原因 #

这是闭包捕获循环变量的典型问题：

• 循环中的timerCtr是同一个变量的引用，Lambda表达式（Elapsed事件的回调）捕获的是这个变量的引用，而非循环迭代时的当前值。当循环执行完毕，timerCtr的值已经变为10，所有Timer触发Elapsed事件时，读取的都是这个最终值，所以弹窗全显示10。
• 另外代码还存在数组索引越界隐患：finalLabelArray长度为10，合法索引是0-9，但第一个循环中ctr从1到10，直接给finalLabelArray[ctr]赋值会导致ctr=10时触发索引越界异常。

二、更优实现方案 #

方案1：修复原Timer逻辑 #

在循环内部将timerCtr的值保存到局部变量，让闭包捕获局部变量（每个迭代的局部变量都是独立的），同时修正数组索引问题，还要注意UI线程切换（System.Timers.Timer的Elapsed事件在非UI线程触发，操作控件需要用Invoke）：

Control[] finalLabelArray = new Control[10];
// 修正数组索引赋值
for (int ctr = 1; ctr < 11; ctr++)
{
    finalLabelArray[ctr - 1] = this.Controls["key" + ctr.ToString()];
}

System.Timers.Timer[] timerArray = new System.Timers.Timer[finalLabelArray.Length];
for (int timerCtr = 0; timerCtr < timerArray.Length; timerCtr++)
{
    // 用局部变量保存当前循环索引
    int currentIndex = timerCtr;
    timerArray[currentIndex] = new System.Timers.Timer(currentIndex * 1000 + 1000);
    timerArray[currentIndex].Elapsed += (sender, e) =>
    {
        // 切换到UI线程操作控件
        this.Invoke(() =>
        {
            MessageBox.Show(currentIndex.ToString());
            Control targetLabel = finalLabelArray[currentIndex];
            // 简化标签编号解析
            int labelNumber = int.Parse(targetLabel.Name.Substring(3));
            // 这里添加显示标签的逻辑，比如 targetLabel.Visible = true;
        });
    };
    timerArray[currentIndex].AutoReset = false;
    timerArray[currentIndex].Start();
}

方案2：用async/await + Task.Delay替代Timer #

这是更简洁的实现方式，天然避免闭包陷阱，且await会自动回到UI线程，无需手动处理线程切换：

private async void Form_Load(object sender, EventArgs e)
{
    Control[] finalLabelArray = new Control[10];
    for (int ctr = 1; ctr < 11; ctr++)
    {
        finalLabelArray[ctr - 1] = this.Controls["key" + ctr.ToString()];
    }

    for (int index = 0; index < finalLabelArray.Length; index++)
    {
        // 等待指定间隔
        await Task.Delay(index * 1000 + 1000);
        MessageBox.Show(index.ToString());
        Control targetLabel = finalLabelArray[index];
        int labelNumber = int.Parse(targetLabel.Name.Substring(3));
        // 执行显示标签的逻辑
        targetLabel.Visible = true;
    }
}

这个方案的优势：代码更简洁，无需管理Timer数组，自动处理线程上下文，可读性和可维护性更高。

内容的提问来源于stack exchange，提问作者Kevin Melendres