为何WPF动态生成控件时触发System.InvalidOperationException？

嘿，踩过这个坑的人来给你指条明路😉！你遇到的异常，核心原因其实很简单：你创建Label的时候从来没给它设置Name属性，但在鼠标点击事件里却试图通过k.Name=="label"去查找——这自然找不到匹配的元素，First()方法找不到结果就会抛出这个异常！

另外还有个隐藏的小坑：循环里的grid变量会被lambda表达式的闭包捕获，导致所有鼠标点击事件都会指向最后一个创建的Grid，这绝对不是你想要的效果！

两步解决你的问题： #

1. 简化Label查找逻辑（或补全Name属性）
   因为你每个Grid里只添加了一个Label，完全不用通过Name查找，直接取第一个Label类型的元素就行，省得还要维护Name的唯一性。如果非要用Name，记得给Label加上label.Name = "label";。
2. 修复闭包捕获的问题
   在循环里把当前的grid赋值给一个局部变量，确保每个事件处理程序都绑定到正确的Grid实例。

修改后的完整代码： #

for (int i = 0; i < numberOfBooks; i++) 
{ 
    Grid grid = new Grid(); 
    RowDefinition row = new RowDefinition(); 
    ColumnDefinition column = new ColumnDefinition(); 
    ColumnDefinition column2 = new ColumnDefinition(); 
    ColumnDefinition column3 = new ColumnDefinition(); 
    Label label = new Label(); 
    label.Content = Books[i].Titel; 
    // 若坚持用Name查找，需添加这行
    // label.Name = "label"; 

    upperGrid.RowDefinitions.Add(row); 
    grid.ColumnDefinitions.Add(column); 
    grid.ColumnDefinitions.Add(column2); 
    grid.ColumnDefinitions.Add(column3); 
    Grid.SetRow(label, 0); // 注意：Label是当前Grid的第0行，不是upperGrid的行号i
    Grid.SetColumn(label, 0); 
    Grid.SetRow(grid, i); 
    upperGrid.Children.Add(grid); 
    grid.Children.Add(label); 

    // 修复闭包问题：将当前grid赋值给局部变量
    var currentGrid = grid;
    grid.MouseLeftButtonDown += (sen, evg) => 
    { 
        // 每个Grid仅一个Label，直接取第一个即可
        Label lbl = currentGrid.Children.OfType<Label>().First(); 
        string result = lbl.Content.ToString(); // 你要的是内容，不是Name吧？
        Console.WriteLine(result); 
    }; 
}

额外优化建议： #

• 你之前写Grid.SetRow(label, i)是错误的：Label是放在当前创建的grid内部，它的行号应该是0，而循环变量i是当前grid在upperGrid里的行号。
• 如果目标是获取书本数据，不用绕弯子找Label——可以把Books[i]绑定到Label的Tag属性：label.Tag = Books[i];，之后在事件里直接取：

  var book = (Book)currentGrid.Children.OfType<Label>().First().Tag;
  Console.WriteLine(book.Titel);
  

• 长远来看，WPF动态生成列表项更推荐用ItemsControl或ListView这类控件，配合数据绑定（MVVM模式），比手动创建Grid和Label简洁得多，也更易维护。

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