使用线程实现Car对象移动时遇到的三类问题求助

让我们逐个拆解你遇到的三个问题，然后给出针对性的修复方案：

问题1：旧车辆停止移动的原因 #

你代码里的br是全局变量，每次点击都会自增。而每个Draw线程里都在引用这个全局的br——当你第二次点击时，br的值变了，旧线程里的循环就开始操作新的Car对象，旧的Car自然就没人管了，所以会停止移动。

问题2：点击越多移动越快的原因 #

每次点击你都启动一个新的Draw线程，每个线程都在循环调用pictureBox1.Refresh()。多个线程同时触发界面刷新，相当于把刷新频率叠加了，视觉上就会觉得车辆移动越来越快，而且这种多线程直接操作界面的方式本身就很容易导致卡顿和资源浪费。

问题3：关闭窗体时的异常 #

窗体关闭后，pictureBox1的窗口句柄已经被销毁了，但你的Draw线程还在后台运行，当线程执行到pictureBox1.Invoke(...)时，因为控件句柄不存在，就会抛出InvalidOperationException。

完整修复方案 #

我们需要重构代码，用更合理的方式管理车辆和动画逻辑：

1. 去掉全局线程和br变量，改用定时器统一绘制

WinForms里推荐用System.Windows.Forms.Timer来处理界面动画，它本身就在UI线程执行，不需要额外的Invoke，而且能统一控制所有车辆的绘制和移动：

// 全局变量只保留车辆列表
private List<Car> _cars = new List<Car>();
private Timer _animationTimer;

// 窗体加载时初始化定时器
private void Form1_Load(object sender, EventArgs e)
{
    _animationTimer = new Timer();
    _animationTimer.Interval = 100; // 和你之前的Sleep时间一致
    _animationTimer.Tick += AnimationTimer_Tick;
}

// 定时器Tick事件：统一更新所有车辆并触发重绘
private void AnimationTimer_Tick(object sender, EventArgs e)
{
    // 先让所有车辆更新位置
    foreach (var car in _cars)
    {
        car.Move();
    }
    // 触发PictureBox重绘（绘制逻辑放在Paint事件里）
    pictureBox1.Refresh();
}

// PictureBox的Paint事件：在这里统一绘制所有车辆
private void pictureBox1_Paint(object sender, PaintEventArgs e)
{
    foreach (var car in _cars)
    {
        car.DrawCar(e.Graphics, Color.Red);
    }
}

2. 修改点击事件，添加车辆并启动定时器（避免重复启动）

private void pictureBox1_MouseClick(object sender, MouseEventArgs e)
{
    // 添加新车辆到列表
    _cars.Add(new Car(e.X, e.Y, 40, 40));
    
    // 如果定时器未运行，就启动它
    if (!_animationTimer.Enabled)
    {
        _animationTimer.Start();
    }
}

3. 处理窗体关闭，停止定时器释放资源

private void Form1_FormClosing(object sender, FormClosingEventArgs e)
{
    // 停止定时器并释放资源，避免后台逻辑继续执行
    _animationTimer.Stop();
    _animationTimer.Dispose();
}

4. 调整Car类的DrawCar方法（使用Paint事件提供的Graphics）

原来自己创建Graphics的方式容易导致资源泄漏，现在直接用Paint事件传入的e.Graphics更高效：

// 你的Car类DrawCar方法示例
public void DrawCar(Graphics g, Color color)
{
    // 这里写你的绘制逻辑，比如绘制矩形车辆
    g.FillRectangle(new SolidBrush(color), X, Y, Width, Height);
}

为什么这样修复？ #

• 用定时器统一管理动画，避免了多线程操作界面的冲突，不会出现点击越多速度越快的情况；
• 所有车辆都保存在列表中，每次定时器触发都会更新所有车辆的位置并绘制，旧车辆不会停止移动；
• 窗体关闭时停止定时器，不会有后台线程尝试调用已销毁的控件方法，解决了关闭时的异常。

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