C# WinForm：foreach循环添加Panel仅显示第一条数据的问题排查求助

嘿，我一眼就瞅出问题啦——你动态创建的所有Panel都重叠在同一个位置了，这就是为啥只能看到第一条数据的原因！

核心错误点 #

你把int x = 56;写在了foreach循环的内部，这意味着每次循环开始时，x都会被重新赋值为56。计算Panel的Y坐标时，++x +66每次都会得到57+66=123，所有Panel都堆在了同一个位置，自然只能看到最上面的那一个。

修复方案 #

1. 修正Panel的位置计算逻辑

把x的初始化移到循环外面，让它在每次循环时累计递增，这样每个Panel的Y坐标会依次往下偏移，避免重叠：

private void loopPanel(List<Bug> list) { 
    // 将x的初始化移到循环外部，作为起始偏移量
    int x = 56; 
    foreach (var bug in list) { 
        Panel panel = new Panel(); 
        this.panelBugs.Controls.Add(panel); 
        Label lblProject = new Label(); 
        Label lblClass = new Label(); 
        Label lblMethod = new Label(); 
        PictureBox pictureBox = new PictureBox(); 

        // ... 保留你原有的Panel和控件属性设置代码 ...

        // 修改位置计算：用当前x值作为Y坐标基准
        panel.Location = new System.Drawing.Point(10, x + 66); 
        // 累加x的值，确保下一个Panel在当前Panel的下方（这里加了10作为间距，可按需调整）
        x += panel.Height + 10; 

        // ... 保留你原有的其他控件初始化代码 ...
    } 
}

2. 优化控件名称（可选但推荐）

你现在设置控件Name的方式会生成类似System.Windows.Forms.Panel123的奇怪字符串，建议改成更直观的命名：

panel.Name = $"panel_Bug_{bug.BugId}";
lblProject.Name = $"lblProject_Bug_{bug.BugId}";
lblClass.Name = $"lblClass_Bug_{bug.BugId}";
lblMethod.Name = $"lblMethod_Bug_{bug.BugId}";
pictureBox.Name = $"pic_Bug_{bug.BugId}";

3. 避免重复添加控件（可选）

如果你的loopPanel方法可能被多次调用，建议先清空容器里的旧控件，防止重复叠加：

private void loopPanel(List<Bug> list) { 
    int x = 56; 
    // 清空panelBugs里的旧控件
    panelBugs.Controls.Clear();
    foreach (var bug in list) { 
        // ... 后续代码 ...
    } 
}

测试效果 #

修改之后，每个Bug对应的Panel会垂直排列在panelBugs容器中，不会再重叠，你就能看到所有数据啦！

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