WinForms PictureBox绘制直线避免重复线条问题求助

我来帮你搞定这个问题！你遇到的重复线条问题，核心原因是直接在MouseMove事件里用CreateGraphics绘制，每次鼠标移动都会叠加新的线条，而且没有遵循WinForms的正确绘图机制。咱们一步步来修正：

原代码的问题分析 #

• 用pictureBox1.CreateGraphics()直接绘制的内容是临时的，控件刷新后就会消失，但你每次MouseMove都画一次，导致拖动时不断叠加新的线条，看起来像是重复了。
• 没有处理Pen的资源释放，长期运行会造成内存泄漏。
• Invalidate和Refresh的调用时机不对，没有起到正确的刷新作用。

正确的解决方案思路 #

WinForms的绘图应该通过控件的Paint事件来完成，这样每次控件刷新都会重新绘制所有内容，不会出现叠加问题。我们需要：

• 维护一个列表来保存已经完成的线条。
• 在MouseDown时记录起点，MouseMove时记录当前临时终点，MouseUp时把临时线条加入列表。
• 在Paint事件中统一绘制所有已完成的线条，以及当前正在拖动的临时线条。

修改后的完整代码 #

首先定义一个结构体来存储线条的起点和终点：

public struct Line
{
    public Point Start { get; set; }
    public Point End { get; set; }
}

// 保存所有已完成的线条
private List<Line> _drawnLines = new List<Line>();
// 当前绘制的起点和临时终点
private Point _startPoint;
private Point _currentEndPoint;
// 是否正在绘制的标记
private bool _isDrawing;
// 你原代码中的计数器变量
private int _counter = 0;
private int _i = 0;
private double _angle;

然后修改鼠标事件：

private void pictureBox1_MouseDown(object sender, MouseEventArgs e)
{
    _isDrawing = true;
    _startPoint = e.Location;
    _currentEndPoint = e.Location; // 初始化临时终点为起点
}

private void pictureBox1_MouseMove(object sender, MouseEventArgs e)
{
    if (_isDrawing)
    {
        _currentEndPoint = e.Location;
        // 计算角度
        _angle = GetAngle(_startPoint.X, _startPoint.Y, e.X, e.Y);
        // 触发Paint事件刷新绘图
        pictureBox1.Invalidate();
    }
}

private void pictureBox1_MouseUp(object sender, MouseEventArgs e)
{
    if (_isDrawing)
    {
        _isDrawing = false;
        // 将完成的线条加入列表
        _drawnLines.Add(new Line { Start = _startPoint, End = e.Location });
        _counter++;
        
        // 更新文本框信息
        textBox_coordinates.Text = $"Index: {_i}{Environment.NewLine}X: {e.X}{Environment.NewLine}Y: {e.Y}{Environment.NewLine}Angle: {_angle}";
        _i++;
        // 刷新最终线条
        pictureBox1.Invalidate();
    }
}

最后添加PictureBox的Paint事件（记得在设计器里绑定，或者在Form构造函数里注册事件）：

private void pictureBox1_Paint(object sender, PaintEventArgs e)
{
    // 使用using确保Pen资源被正确释放
    using (Pen bluePen = new Pen(Color.Blue, 1))
    {
        // 绘制所有已完成的线条
        foreach (var line in _drawnLines)
        {
            e.Graphics.DrawLine(bluePen, line.Start, line.End);
        }
        
        // 如果正在绘制，绘制临时线条
        if (_isDrawing)
        {
            e.Graphics.DrawLine(bluePen, _startPoint, _currentEndPoint);
        }
    }
}

关键改进点说明 #

1. 统一在Paint事件绘图：所有绘图操作都放在Paint事件里，每次Invalidate都会触发重新绘制，不会出现叠加的重复线条。
2. 资源管理：用using语句包裹Pen，确保使用后及时释放资源，避免内存泄漏。
3. 状态管理：用列表保存已完成的线条，确保控件刷新时不会丢失历史绘制内容；用_isDrawing标记控制临时线条的绘制。
4. 高效刷新：MouseMove只更新临时终点并触发刷新，不会直接绘制，拖动时只会显示当前的临时线条，不会叠加。

额外提醒 #

• 确保你已经给pictureBox1绑定了Paint事件（可以在设计器的属性窗口里找到事件列表，选择Paint并绑定到上面的方法）。
• 你原代码中的GetAngle函数如果逻辑没问题，可以直接继续使用。

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