C # графика мерцает

Question

Я работаю над программой рисования, но у меня проблема с мерцанием при перемещении курсора мыши при рисовании линии с резинкой. Я надеюсь, что вы можете помочь мне убрать это мерцание строки, вот код:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace GraphicsTest
{
    public partial class Form1 : Form
    {
        int xFirst, yFirst;
        Bitmap bm = new Bitmap(1000, 1000);
        Graphics bmG;
        Graphics xG;
        Pen pen = new Pen(Color.Black, 1);
        bool draw = false;

        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            bmG = Graphics.FromImage(bm);
            xG = this.CreateGraphics();
            bmG.Clear(Color.White);
        }

        private void Form1_MouseDown(object sender, MouseEventArgs e)
        {
            xFirst = e.X;
            yFirst = e.Y;
            draw = true;
        }

        private void Form1_MouseUp(object sender, MouseEventArgs e)
        {
            bmG.DrawLine(pen, xFirst, yFirst, e.X, e.Y);
            draw = false;
            xG.DrawImage(bm, 0, 0);
        }

        private void Form1_MouseMove(object sender, MouseEventArgs e)
        {
            if (draw)
            {
                xG.DrawImage(bm, 0, 0);
                xG.DrawLine(pen, xFirst, yFirst, e.X, e.Y);
            }
        }

        private void Form1_Paint(object sender, PaintEventArgs e)
        {
            xG.DrawImage(bm, 0, 0);
        }
    }
}

Answer 1

Во-первых, не используйте CreateGraphics(), если только вам это не нужно. Привязать обработчик событий к OnPaint и вызвать Invalidate(), когда вы хотите обновить поверхность.

Если вы не хотите, чтобы он мерцал, вам нужно будет дважды буферизовать поверхность рисования. Самый простой способ сделать это - установить для свойства DoubleBuffered вашей формы значение True.

Я очень рекомендую, если вы планируете расширить это, чтобы сделать ваш рисунок для элемента управления PictureBox. PictureBox по умолчанию имеет двойную буферизацию и позволяет намного проще управлять областью рисования.

В коде:

public partial class Form1 : Form
    {
    int xFirst, yFirst;
    Bitmap bm = new Bitmap(1000, 1000);
    Graphics bmG;
    Pen pen = new Pen(Color.Black, 1);
    bool draw = false;

    public Form1()
    {
        InitializeComponent();
    }

    private void Form1_Load(object sender, EventArgs e)
    {
        bmG = Graphics.FromImage(bm);
        bmG.Clear(Color.White);
    }

    private void Form1_MouseDown(object sender, MouseEventArgs e)
    {
        xFirst = e.X;
        yFirst = e.Y;
        draw = true;
    }

    private void Form1_MouseUp(object sender, MouseEventArgs e)
    {
        bmG.DrawLine(pen, xFirst, yFirst, e.X, e.Y);
        draw = false;
        Invalidate();
    }

    private void Form1_MouseMove(object sender, MouseEventArgs e)
    {
        if (draw)
        {
            Invalidate();
        }
    }

    private void Form1_Paint(object sender, PaintEventArgs e)
    {
        if (draw) {
            e.Graphics.DrawImage(bm, 0, 0);
            e.Graphics.DrawLine(pen, xFirst, yFirst, e.X, e.Y);
        } else {
            e.Graphics.DrawImage(bm, 0, 0);
        }
    }
}

Edit:

Еще одна проблема, вы создаете частного Pen участника. Ручки (и кисти, а также многие объекты GDI +) представляют собой дескрипторы неуправляемых объектов, которые необходимо утилизировать, иначе ваша программа утечет. Либо заключите их в операторы using (предпочтительный и безопасный для исключений способ), либо явным образом избавьтесь от них в методе Dispose формы.

В качестве альтернативы в System.Drawing вы можете получить доступ к некоторым предварительно созданным ручкам и кистям, которые не нужно (и не следует) утилизировать. Используйте их как:

    private void Form1_Paint(object sender, PaintEventArgs e)
    {
        if (draw) {
            e.Graphics.DrawImage(bm, 0, 0);
            e.Graphics.DrawLine(Pens.Black, xFirst, yFirst, e.X, e.Y);
        } else {
            e.Graphics.DrawImage(bm, 0, 0);
        }
    }

Answer 2

Причина, по которой он мигает, заключается в том, что вы рисуете фон (который сразу отображается на экране, стирая линию), а затем накладываете линию. Таким образом, линия продолжает исчезать и появляться, давая мерцающий дисплей.

Лучшее решение для этого называется двойной буферизацией. То, что вы делаете, - рисуете все изображение в «закадровом» растровом изображении и показывает его на экране только после его завершения. Поскольку вы отображаете только законченное изображение, эффект мерцания отсутствует. Вы должны просто установить this.DoubleBuffered = true, чтобы WinForms выполнила всю тяжелую работу за вас.

NB. На самом деле вы не должны рисовать вне вашего обработчика рисования - в идеале вы должны Invalidate () область, которая требует перерисовки, а затем ваш обработчик рисования будет перерисовывать только эту область (с любыми наложенными линиями и т. Д. По мере необходимости) .

Answer 3

Фиксированный и рабочий код.

public partial class Form1 : Form
{
    int x1, y1, x2, y2;
    bool drag = false;

    Bitmap bm = new Bitmap(1000, 1000);
    Graphics bmg;


    public Form1()
    {
        InitializeComponent();
    }

    private void Form1_Load(object sender, EventArgs e)
    {
        bmg = Graphics.FromImage(bm);
    }

    private void pictureBox_MouseDown(object sender, MouseEventArgs e)
    {
        drag = true;
        x1 = e.X;
        y1 = e.Y;
    }

    private void pictureBox_MouseUp(object sender, MouseEventArgs e)
    {
        drag = false;

        bmg.DrawLine(Pens.Black, x1, y1, e.X, e.Y);
        pictureBox.Invalidate();
    }

    private void pictureBox_MouseMove(object sender, MouseEventArgs e)
    {
        if (drag)
        {
            x2 = e.X;
            y2 = e.Y;
            pictureBox.Invalidate();
        }
    }

    private void pictureBox_Paint(object sender, PaintEventArgs e)
    {
        if (drag) {
            e.Graphics.DrawImage(bm, 0, 0);
            e.Graphics.DrawLine(Pens.Black, x1, y1, x2, y2);            
        }
        else {
            e.Graphics.DrawImage(bm, 0, 0);
        }
    }
}