C#循环内绑定Click事件时变量i值异常问题求助

你遇到的是闭包捕获变量的常见陷阱：在for循环中绑定的Click事件lambda表达式，捕获的是循环变量i的引用，而非循环每一轮i的具体值。当循环执行完毕，i已经变成了循环结束时的最终值（也就是你看到的3），因此不管点击哪个LinkLabel，都会使用这个最终的i值。

解决方法一：用局部变量保存当前索引 #

在循环内部创建一个独立的局部变量，存储当前i的值，让lambda捕获这个局部变量：

for (int i = 0; i < idlist.Length - 1; i++)
{
    // 新增局部变量保存当前循环的索引值
    int currentIndex = i;
    LinkLabel ln = new LinkLabel();

    // ... 其他原有代码不变 ...

    ln.Click += (object senderw, EventArgs ew) => {
            string strUrl = linkbuilder[currentIndex];  // 使用局部变量代替i
            Process proc = new Process();
            ProcessStartInfo startInfo = new ProcessStartInfo(strUrl);
            proc.StartInfo = startInfo;
            proc.Start();
    };
    this.listView1.Controls.Add(ln);
}

每一轮循环都会生成一个独立的currentIndex，lambda捕获的是该轮次变量的引用，点击时就能获取到对应循环的索引值。

解决方法二：将URL绑定到控件Tag属性 #

直接把当前轮次的URL存入LinkLabel的Tag属性，点击时从触发事件的控件本身取数据，完全避免对循环变量的依赖：

for (int i = 0; i < idlist.Length - 1; i++)
{
    LinkLabel ln = new LinkLabel();

    // ... 其他原有代码不变 ...

    // 将当前URL存入控件的Tag属性
    ln.Tag = linkbuilder[i];
    ln.Click += (object senderw, EventArgs ew) => {
            LinkLabel clickedLink = senderw as LinkLabel;
            if (clickedLink != null && clickedLink.Tag != null)
            {
                string strUrl = clickedLink.Tag.ToString();
                Process proc = new Process();
                ProcessStartInfo startInfo = new ProcessStartInfo(strUrl);
                proc.StartInfo = startInfo;
                proc.Start();
            }
    };
    this.listView1.Controls.Add(ln);
}

这种方式逻辑更直观，也不容易出现闭包相关的陷阱。

内容的提问来源于stack exchange，提问作者Supun Madhushanka