C# WinForm循环仅打印最后一个元素问题求助

你的代码里每次循环都直接给Label的Text属性赋值，这会覆盖掉之前的内容，所以最终只能看到最后一次循环的结果。要显示所有元素，需要把每个元素的内容拼接起来，而不是每次替换。另外，多次创建Random实例可能导致生成重复随机数，建议把它声明为类的成员变量。

修改后的完整代码 #

const int numberOfItems = 20;
// 把Random声明为类成员，避免重复创建导致随机数重复
private Random _random = new Random();

private void Form1_Load(object sender, EventArgs e)
{
    CultureInfo ci = new CultureInfo("en-US");
    Thread.CurrentThread.CurrentUICulture = ci;
    Thread.CurrentThread.CurrentCulture = ci;

    int[] numbers = new int[numberOfItems];
    double total = 0;
    // 用StringBuilder拼接内容，效率更高
    StringBuilder sb = new StringBuilder();
          
    for (int i = 0; i < numbers.Length; i++)
    {
        numbers[i] = _random.Next(501);  
        sb.AppendLine($"Element {i} = {numbers[i]}");
        total += numbers[i];
    }
    // 最后一次性把拼接好的内容赋值给Label
    beforeLoop.Text = sb.ToString();
}

private void compare_Click(object sender, EventArgs e)
{
    if (!int.TryParse(numberBox.Text, out int number))
    {
        printLabel.Text = "请输入有效的整数";
        return;
    }

    int[] numbers = new int[numberOfItems];
    double total = 0;
    StringBuilder sb = new StringBuilder();
    
    for (int i = 0; i < numbers.Length; i++)
    {
        numbers[i] = _random.Next(501);
              
        int numberBigger = numbers[i] + 10;
        int numberSmaller = numbers[i] - 5;
        total += numbers[i];

        if (number > numbers[i])
        {
            sb.AppendLine($"Element {i} = {numberBigger}");
        }
        else
        {
            sb.AppendLine($"Element {i} = {numberSmaller}");
        }
    }
    printLabel.Text = sb.ToString();
}

关键修改点 #

• 使用StringBuilder拼接所有元素的文本，避免每次赋值Label时的性能损耗和内容覆盖
• 循环结束后一次性将拼接好的文本赋值给Label的Text属性
• 将Random实例改为类成员，避免短时间内多次创建导致生成重复的随机数
• 增加int.TryParse判断，防止输入非整数时抛出异常

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