c#Pen抛出了OutOfMemoryException
一个带有面板和框的表单，用户点击一个框，一个标签出现片刻，说“框命中” 我有一个工作版本的代码，但我试图重新安排它，使其更有效的实验，我发现我目前得到这个例外c#Pen抛出了OutOfMemoryException,c#,winforms,visual-studio,memory,idisposable,C#,Winforms,Visual Studio,Memory,Idisposable,一个带有面板和框的表单，用户点击一个框，一个标签出现片刻，说“框命中” 我有一个工作版本的代码，但我试图重新安排它，使其更有效的实验，我发现我目前得到这个例外 using System; using System.Collections.Generic; using System.ComponentModel; using System.Data; using System.Drawing; using System.Linq; using System.Text; using System.T
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Drawing.Drawing2D;
namespace WindowsFormsApplication17
{
public partial class Form1 : Form
{
Label lbl1;
Panel panel;
Timer timer1;
Point[][] points = new Point[9][];
GraphicsPath[] gps = new GraphicsPath[9];
public Form1()
{
InitializeComponent();
}
private void Form1_Load(object sender, EventArgs e)
{
panel = new Panel();
lbl1 = new Label();
timer1 = new Timer();
this.Controls.Add(panel);
Label alias = lbl1;
alias.Text = "box clicked";
alias.Visible = false;
panel.BackgroundImageLayout = ImageLayout.Stretch;
this.Height = 500; this.Width = 500;
panel.Controls.Add(lbl1);
panel.MouseDown += p_MouseDown;
timer1.Tick += timer1_Tick;
panel.BackColor = Color.White;
panel.Height = 400;
panel.Width = 400;
for (int i = 0; i < 9; i++) points[i] = new Point[4];
points[0][0] = new Point(19, 262);
points[0][1] = new Point(28, 257);
points[0][2] = new Point(27, 284);
points[0][3] = new Point(16, 285);
points[1][0] = new Point(52, 253);
points[1][1] = new Point(62, 250);
points[1][2] = new Point(61, 277);
points[1][3] = new Point(49, 278);
points[2][0] = new Point(87, 249);
points[2][1] = new Point(100, 248);
points[2][2] = new Point(99, 275);
points[2][3] = new Point(86, 274);
points[3][0] = new Point(126, 250);
points[3][1] = new Point(140, 252);
points[3][2] = new Point(139, 279);
points[3][3] = new Point(126, 277);
points[4][0] = new Point(164, 257);
points[4][1] = new Point(175, 260);
points[4][2] = new Point(175, 287);
points[4][3] = new Point(164, 284);
points[5][0] = new Point(197, 265);
points[5][1] = new Point(209, 269);
points[5][2] = new Point(209, 295);
points[5][3] = new Point(198, 292);
points[6][0] = new Point(228, 273);
points[6][1] = new Point(241, 275);
points[6][2] = new Point(240, 300);
points[6][3] = new Point(229, 300);
points[7][0] = new Point(262, 274);
points[7][1] = new Point(274, 273);
points[7][2] = new Point(275, 300);
points[7][3] = new Point(262, 301);
points[8][0] = new Point(297, 272);
points[8][1] = new Point(308, 268);
points[8][2] = new Point(311, 295);
points[8][3] = new Point(298, 296);
panel.Paint += thepanel_Paint;
// I can see that I can remove all the braces, but anyhow.
for (int i = 0; i < 9; i++)
{
using (gps[i] = new GraphicsPath())
using (Pen pen = new Pen(Color.Black, 1)) //this pen is not for drawing. but for the logic re graphicspath
{
gps[i].AddPolygon(points[i]);
}
}
}
void p_MouseDown(object sender, EventArgs e)
{
MouseEventArgs mea = (MouseEventArgs)e;
//GraphicsPath[] gps = new GraphicsPath[9];
bool boxhit = false;
for (int i = 0; i < 9; i++)
{
using (Pen pen = new Pen(Color.Black, 1)) //this pen is not for drawing. but for the logic re graphicspath
{
bool cond1=false;
bool cond2=false;
cond1 = gps[i].IsOutlineVisible(new Point(mea.X, mea.Y), pen);
cond2 = gps[i].IsVisible(new Point(mea.X, mea.Y));
if (cond1 == true ||
cond2 == true)
{
boxhit = true;
lbl1.Visible = true;
timer1.Enabled = true;
}
}
} //for
// if (boxhit == false) MessageBox.Show("no box hit");
}
private void timer1_Tick(object sender, EventArgs e)
{
lbl1.Visible = false; timer1.Enabled = false;
}
private void thepanel_Paint(object sender, PaintEventArgs e)
{
GraphicsPath[] gps = new GraphicsPath[9];
Graphics gg = panel.CreateGraphics();
for (int i = 0; i < 9; i++)
using (gps[i] = new GraphicsPath())
{
gps[i].AddPolygon(points[i]);
gg.DrawPath(new Pen(Color.Blue, 1), gps[i]);
}
}
}
}
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Drawing.Drawing2D;
namespace WindowsFormsApplication17
{
public partial class Form1 : Form
{
Label lbl1;
Panel panel;
Timer timer1;
Point[][] points = new Point[9][];
GraphicsPath[] gps = new GraphicsPath[9];
public Form1()
{
InitializeComponent();
}
private void Form1_Load(object sender, EventArgs e)
{
panel = new Panel();
lbl1 = new Label();
timer1 = new Timer();
this.Controls.Add(panel);
Label alias = lbl1;
alias.Text = "box clicked";
alias.Visible = false;
panel.BackgroundImageLayout = ImageLayout.Stretch;
this.Height = 500; this.Width = 500;
panel.Controls.Add(lbl1);
panel.MouseDown += p_MouseDown;
timer1.Tick += timer1_Tick;
panel.BackColor = Color.White;
panel.Height = 400;
panel.Width = 400;
for (int i = 0; i < 9; i++) points[i] = new Point[4];
points[0][0] = new Point(19, 262);
points[0][1] = new Point(28, 257);
points[0][2] = new Point(27, 284);
points[0][3] = new Point(16, 285);
points[1][0] = new Point(52, 253);
points[1][1] = new Point(62, 250);
points[1][2] = new Point(61, 277);
points[1][3] = new Point(49, 278);
points[2][0] = new Point(87, 249);
points[2][1] = new Point(100, 248);
points[2][2] = new Point(99, 275);
points[2][3] = new Point(86, 274);
points[3][0] = new Point(126, 250);
points[3][1] = new Point(140, 252);
points[3][2] = new Point(139, 279);
points[3][3] = new Point(126, 277);
points[4][0] = new Point(164, 257);
points[4][1] = new Point(175, 260);
points[4][2] = new Point(175, 287);
points[4][3] = new Point(164, 284);
points[5][0] = new Point(197, 265);
points[5][1] = new Point(209, 269);
points[5][2] = new Point(209, 295);
points[5][3] = new Point(198, 292);
points[6][0] = new Point(228, 273);
points[6][1] = new Point(241, 275);
points[6][2] = new Point(240, 300);
points[6][3] = new Point(229, 300);
points[7][0] = new Point(262, 274);
points[7][1] = new Point(274, 273);
points[7][2] = new Point(275, 300);
points[7][3] = new Point(262, 301);
points[8][0] = new Point(297, 272);
points[8][1] = new Point(308, 268);
points[8][2] = new Point(311, 295);
points[8][3] = new Point(298, 296);
panel.Paint += thepanel_Paint;
// I can see that I can remove all the braces, but anyhow.
for (int i = 0; i < 9; i++)
{
using (gps[i] = new GraphicsPath())
using (Pen pen = new Pen(Color.Black, 1)) //this pen is not for drawing. but for the logic re graphicspath
{
gps[i].AddPolygon(points[i]);
}
}
}
void p_MouseDown(object sender, EventArgs e)
{
MouseEventArgs mea = (MouseEventArgs)e;
//GraphicsPath[] gps = new GraphicsPath[9];
bool boxhit = false;
for (int i = 0; i < 9; i++)
{
using (Pen pen = new Pen(Color.Black, 1)) //this pen is not for drawing. but for the logic re graphicspath
{
bool cond1=false;
bool cond2=false;
cond1 = gps[i].IsOutlineVisible(new Point(mea.X, mea.Y), pen);
cond2 = gps[i].IsVisible(new Point(mea.X, mea.Y));
if (cond1 == true ||
cond2 == true)
{
boxhit = true;
lbl1.Visible = true;
timer1.Enabled = true;
}
}
} //for
// if (boxhit == false) MessageBox.Show("no box hit");
}
private void timer1_Tick(object sender, EventArgs e)
{
lbl1.Visible = false; timer1.Enabled = false;
}
private void thepanel_Paint(object sender, PaintEventArgs e)
{
GraphicsPath[] gps = new GraphicsPath[9];
Graphics gg = panel.CreateGraphics();
for (int i = 0; i < 9; i++)
using (gps[i] = new GraphicsPath())
{
gps[i].AddPolygon(points[i]);
gg.DrawPath(new Pen(Color.Blue, 1), gps[i]);
}
}
}
}
cond1=gps[I].isooutlinevisible（新点（mea.X，mea.Y），pen）上得到相同的异常p\u MouseDownusing System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
namespace blah
{
public partial class Form1 : Form
{
Pen p;
public Form1()
{
InitializeComponent();
}
private void Form1_Load(object sender, EventArgs e)
{
MessageBox.Show("sadf"); //displays
MessageBox.Show(p.Color.ToString()); // doesn't display a messagebox at all!
}
}
}
除了初始化
画笔using图形。那么当您使用时会发生什么呢。。。是的，对象已被处理
相反，尝试创建您的路径，例如
for (int i = 0; i < 9; i++)
{
gps[i] = new GraphicsPath();
gps[i].AddPolygon(points[i]);
}
您正在Paint事件处理程序中泄漏图形对象。也许还有更多的例外，一旦操作系统确定您泄漏太多，并且不允许您再创建更多的异常，那么这些异常就是不可靠的。使用任务管理器的“进程”选项卡，并为GDI对象和用户对象添加列，这样您就可以看到泄漏的增加。@HansPassant谢谢。。即使使用添加到绘制事件处理程序中的图形对象，我仍然会在同一行（在p_MouseDown过程中）上获得相同的异常。我在我的问题中添加了一个注释。@HansPassant刚刚检查了用户对象和GDI对象，当我运行它时，它们都是32和37，在出现异常时。@barlop可能是因为我全局声明了一个。。我想在内存方面更好的设计是，如果我只有一个图形，并改变了点。一个有趣的稍微相关的问题是，我不应该使用使用-如果我按照你的建议，使用删除，那么对象就不会被处理。但是应该处理IDisposable对象。此外，在工作版本和非工作版本中，我都有“使用（gps[I]=new GraphicsPath（）”，因此它不能像“使用关键字创建GraphicsPath[因此出现问题]”那样简单好的，我在表单加载过程中看到，您提到的for循环可以工作，因为它不会处理对象。。因此，它可用于mousedown过程。但是，当您不再需要对象时，不处理您所处理的对象可能是一种不好的做法。在这种情况下，这是没有意义的。谢谢，我会查出来的
for (int i = 0; i < 9; i++)
{
gps[i] = new GraphicsPath();
gps[i].AddPolygon(points[i]);
}
using System;
using System.Collections.Generic;
using System.Drawing;
using System.Drawing.Drawing2D;
using System.Linq;
using System.Threading;
using System.Threading.Tasks;
using System.Windows.Forms;
namespace WindowsFormsApplication1
{
// It's preferred to use established naming conventions,
// such as prefixing private fields with "_".
// Declaring access modifiers explicitly, even default modifier
// "private", is advised because it makes reading thousands lines of code
// that much easier (one can expect first keyword to be access modifier)
public partial class Form1 : Form
{
// For convenience and readability,
// I'd prefer Lists and List-of-Lists over arrays
private readonly List<List<Point>> _points = new List<List<Point>>();
private readonly List<GraphicsPath> _graphicsPaths = new List<GraphicsPath>();
public Form1()
{
InitializeComponent();
// Points to create paths from
_points.AddRange(new[]
{
new List<Point>
{
new Point(19, 62),
new Point(28, 57),
new Point(27, 84),
new Point(16, 85)
},
new List<Point>
{
new Point(52, 53),
new Point(62, 50),
new Point(61, 77),
new Point(49, 78)
},
new List<Point>
{
new Point(87, 49),
new Point(100, 48),
new Point(99, 75),
new Point(86, 74)
}
});
// Create GDI graphics paths
foreach (List<Point> points in _points)
{
GraphicsPath path = new GraphicsPath();
path.AddPolygon(points.ToArray());
_graphicsPaths.Add(path);
}
}
private void Form1_Load(object sender, EventArgs e)
{
// Adjust form
Height = 200;
Width = 100;
// Create label and panel
// No need for these to be private fields
Label label = new Label
{
Text = @"Border clicked",
Visible = false
};
// Create panel
Panel panel = new Panel
{
Height = 400,
Width = 400,
BackColor = Color.White,
BackgroundImageLayout = ImageLayout.Stretch
};
// Paint event handler.
// Personally I prefer inline anonymous methods
// over named methods when logic is simple
// and it's not being reused
panel.Paint += (o, args) =>
{
// 'using' because we want to get rid of Graphics
// and Pen when we are done drawing paths
using (Graphics graphics = panel.CreateGraphics())
{
using (Pen pen = new Pen(Color.Blue, 3))
{
foreach (GraphicsPath path in _graphicsPaths)
graphics.DrawPath(pen, path);
}
}
};
// Mouse (down) event handler.
panel.MouseDown += (o, args) =>
{
// Get mouse point
Point mousePoint = new Point(args.X, args.Y);
// Again, we want to dispose Pen
using (Pen pen = new Pen(Color.Transparent, 0F))
{
// Get first path under mouse pointer
GraphicsPath path = _graphicsPaths.FirstOrDefault(p =>
p.IsOutlineVisible(mousePoint, pen));
if (path == null)
return;
// If found, "flash" our informative label
// in non-blocking way
Task.Run(() =>
{
label.Invoke((Action)(() => label.Visible = true));
Thread.Sleep(500);
label.Invoke((Action)(() => label.Visible = false));
});
}
};
// Add controls to containers
panel.Controls.Add(label);
Controls.Add(panel);
}
private void Form1_FormClosing(object sender, FormClosingEventArgs e)
{
// This could be more reasonable place to dispose
// GDI Graphics path created earlier?
foreach (GraphicsPath path in _graphicsPaths)
path.Dispose();
_graphicsPaths.Clear();
}
}
}