ASP.NET动态生成DropdownList事件异常：均指向最后一个评论框

你的猜测完全正确！问题核心就出在Lambda表达式对循环变量的捕获机制上。在C#的foreach循环里，tc1和tc2属于循环外复用的变量引用——每次迭代时，这两个变量并没有重新创建实例，只是指向了新的TableCell对象。而Lambda事件处理器捕获的是变量的引用，不是当前迭代的实际TableCell实例。等循环跑完，所有处理器里的引用都指向了最后一次迭代的tc1和tc2，自然就出现了所有下拉框都只控制最后一个评论框的问题。

两种可行的解决方法 #

方法1：循环内创建局部变量捕获引用 #

在每次循环迭代中，把当前的tc1、tc2和ddl赋值给局部变量，让Lambda捕获这些局部变量的副本。这样每个迭代的事件处理器都会持有对应实例的独立引用：

foreach (Question q in questions) {
    if (q.Division.Equals("General") || q.Division.Equals(ddlDivisions.SelectedValue.ToString())) {
        tr = new TableRow();
        // ... 省略其他创建控件的代码 ...
        
        // 关键：在循环内部定义局部变量，保存当前迭代的控件实例
        TableCell currentTc1 = tc1;
        TableCell currentTc2 = tc2;
        DropDownList currentDdl = ddl;
        
        // 绑定事件时使用局部变量
        ddl.SelectedIndexChanged += (sender, e) => ScoreChanged(sender, e, currentTc1, currentTc2, currentDdl.SelectedIndex);
        
        tbl.Rows.Add(tr);
    }
}

方法2：通过控件ID关联（更推荐） #

放弃直接传递控件引用，改用控件ID的规则关联下拉框和评论框。比如下拉框ID是ddl_XXX，对应的评论框是txt_XXXcomment，在事件处理器中通过FindControl找到目标控件：

首先修改事件处理器：

protected void ScoreChanged(object sender, EventArgs e) {
    DropDownList ddl = sender as DropDownList;
    if (ddl == null) return;
    
    // 从下拉框ID中提取问题ID
    string questionId = ddl.ID.Replace("ddl_", "");
    // 从TableRow中找到对应的评论标题和输入框所在单元格
    TableRow parentRow = ddl.Parent.Parent as TableRow;
    TableCell tc1 = parentRow.FindControl(questionId) as TableCell;
    TextBox commentTxt = parentRow.FindControl("txt_" + questionId + "comment") as TextBox;
    TableCell tc2 = commentTxt?.Parent as TableCell;
    
    int score = ddl.SelectedIndex;
    if (score < 2) {
        tc1.Visible = false;
        tc2.Visible = false;
    } else {
        tc1.Visible = true;
        tc2.Visible = true;
    }
}

然后绑定事件时无需传递额外参数：

ddl.SelectedIndexChanged += ScoreChanged;

这种方式彻底避开了Lambda捕获变量的陷阱，代码逻辑也更清晰——控件之间的关联依赖ID规则，不需要依赖循环中的临时变量，后续维护也更方便。

额外小优化 #

你的代码中DropDownList ddl = new DropDownList();是在循环外声明的，虽然循环内重新实例化了，但建议把声明移到循环内部，避免潜在的引用混淆：

if (q.QuestionType.ToLower() == "singleselect") {
    // 把ddl的声明移到循环内部，每个迭代创建独立的变量
    DropDownList ddl = new DropDownList();
    ddl.ID = "ddl_" + q.ID;
    // ... 省略其他下拉框配置代码 ...
}

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