Ошибка при рисовании линии в Graphics.DrawLine C# Visual Studio
private void Form1_Paint(object sender, PaintEventArgs e)
{
Graphics l = e.Graphics;
Pen p = new Pen(Color.Black, 1);
float angle = 0;
float len = 100;
PointF ori = new PointF(Width/2, 0);
PointF bob = new PointF(Width/2, len);
while(true)
{
bob.X = ori.X + len * (float)Math.Sin(angle);
bob.Y = ori.Y + len * (float)Math.Cos(angle);
angle += 0.001F;
l.DrawLine(p, ori.X, ori.Y, bob.X, bob.Y);
l.DrawEllipse(p, bob.X - 15, bob.Y, 30, 30);
if (angle == 360)
{
break;
}
l.Dispose();
}
}
Строка ошибки - l.DrawLine (p, ori.X, ori.Y, bob.X, bob.Y). Тип ошибки: System.ArgumentException. Сообщение об ошибке: параметр недействителен.
Сначала я думал, что проблема связана с поплавками, но DrawLine допускает такие типы данных. Он повторяется, если кажется, что ошибка возникает, когда угол> 0. Его масштабы, похоже, не проблема. Любая помощь приветствуется. Заранее спасибо. [ОБНОВЛЕНИЕ] Похоже, ошибка связана с l.Dispose
@JohnnyMopp, возможно, вы захотите добавить это в качестве ответа
Не Dispose - l.Dispose(); - экземпляр у вас не создавай
Да, я только что понял это. Но теперь предыдущий рисунок не удаляется. Любые идеи?
Вызовите Clear на экземпляре Graphics.
Вам также следует сильно рассмотреть возможность замены этого цикла while на цикл for.
Ответы 2
Проблема в неправильном вызове Dispose:
private void Form1_Paint(object sender, PaintEventArgs e)
{
Graphics l = e.Graphics;
// Pen is IDisposable, that's why why wrap it into "using": it's you who created it
using (Pen p = new Pen(Color.Black, 1)) {
float angle = 0;
float len = 100;
PointF ori = new PointF(Width/2, 0);
PointF bob = new PointF(Width/2, len);
while(true)
{
bob.X = ori.X + len * (float)Math.Sin(angle);
bob.Y = ori.Y + len * (float)Math.Cos(angle);
angle += 0.001F;
l.DrawLine(p, ori.X, ori.Y, bob.X, bob.Y);
l.DrawEllipse(p, bob.X - 15, bob.Y, 30, 30);
// angle is float, that's why == is not recommended:
// (you can well have 359.99999999999999) and thus == will never be true
if (angle >= 360)
break;
// l.Dispose(); // <- Wrong: it's not you who've created it
// (let system Dispose it)
}
}
}
В качестве дополнения к ответу @Dmitry я могу предложить этот совет:
Как правило, вы не должны вызывать Dispose для объекта, время жизни которого вы не контролируете. Вы получаете существующий экземпляр Graphics из события Paint. Элемент управления, вызвавший событие Paint, является тем, что создал этот объект Graphics, поэтому он отвечает за вызов Dispose для него, когда это будет сделано с ним, а не вы.
Когда вы вызываете Dispose для объекта, который вы не контролируете, вы фактически «вырываете почву из-под» этого кода. Это плохо, потому что этот код мог ожидать, что экземпляр все еще жив, чтобы он мог выполнять с ним другие операции. Утилизируя его, вы не даете ему такого шанса.
Однако вы делаете правильные вещи со своим экземпляром Pen, p. Вы создаете его во время заявления using, поэтому вы несете за это ответственность. Using обрабатывает это, автоматически вызывая Dispose, когда выполнение выходит из блока.
Если бы вы создали Graphicsinstance yourself, with something likeGraphics.FromImage`, тогда, вы должны были бы нести ответственность за его очистку.
l.Dispose();делает объектGraphicsнедействительным.