C# WPF动态获取按钮并随机设置颜色时程序崩溃问题

我帮你排查一下这个问题——直接调用没问题，但批量循环就崩溃，大概率是空引用异常在搞鬼，再加上随机数生成的小坑，咱们一步步来解决：

先看问题代码的核心隐患 #

你的代码片段：

private void But0_0_Click(object sender, RoutedEventArgs e) {
    for (int i = 1; i <= 20; i++) {
        var ele = MainGrid.FindName("But0_" + i);
        Button button = ele as Button;
        Change_color(button);
    }
    Change_color(But0_0);
}
private void Change_color(Button name) {
    Random rand = new Random();
    Color[] colors = new Color[] { /* 你的颜色数组 */ };
    // 这里设置背景色的代码会访问name的属性
}

崩溃的主要原因 #

1. 空引用异常：循环里的But0_1到But0_20中，可能有某个按钮在XAML里不存在（比如命名拼写错了、漏建了），导致FindName返回null，button也是null，调用Change_color时访问name.Background就会直接崩溃。
2. 随机数重复问题：每次调用Change_color都新建Random对象，短时间内多次调用会因为系统时钟精度问题生成相同的随机数，导致所有按钮颜色一样（这个不崩溃，但体验不好）。

修复方案 #

1. 先处理空引用，避免崩溃

在循环里先判断按钮是否存在，再执行修改：

private void But0_0_Click(object sender, RoutedEventArgs e) {
    for (int i = 1; i <= 20; i++) {
        var ele = MainGrid.FindName("But0_" + i);
        Button button = ele as Button;
        
        // 先检查按钮是否不为null，再调用方法
        if (button != null) {
            Change_color(button);
        }
        // 可选：打印日志排查哪个按钮找不到
        else {
            System.Diagnostics.Debug.WriteLine($"警告：找不到名为But0_{i}的按钮");
        }
    }
    Change_color(But0_0);
}

2. 优化随机数生成，避免重复颜色

把Random对象改成类的成员变量，只初始化一次，这样多次调用能生成不同的随机数：

// 类的成员变量，全局只初始化一次
private Random _colorRandom = new Random();

private void Change_color(Button button) {
    // 双重保险：如果按钮是null直接返回，避免崩溃
    if (button == null) return;
    
    Color[] colors = new Color[] { 
        Colors.Red, Colors.Green, Colors.Blue,
        Colors.Cyan, Colors.Magenta, Colors.Yellow
        // 这里替换成你自己的颜色数组
    };
    
    int randomColorIndex = _colorRandom.Next(colors.Length);
    button.Background = new SolidColorBrush(colors[randomColorIndex]);
}

3. 最后检查按钮命名

去XAML里确认所有But0_1到But0_20的按钮都存在，命名完全符合格式（比如有没有写成But0_10还是But0_01？有没有拼写错误？），这是最容易忽略的点。

内容的提问来源于stack exchange，提问作者Michał Łahchim