在窗体指定位置上闪烁灯，就是画一个红色的圈，再画一个白色的圈

Question

 protected override void OnPaint(PaintEventArgs e)
        {
            base.OnPaint(e);
            Graphics dc = e.Graphics ;
            int i;
            i = 2;
            while (true)
            {
                if (2 == i)
                {
                    Pen whitePen = new Pen(Color.White, 2);
                    dc.DrawEllipse(whitePen, 0, 50, 50, 50);
                    DateTime start = DateTime.Now;
                    //最多循环10秒
                    while ((DateTime.Now - start).Seconds < 2)
                    {
                        Application.DoEvents();
                    }
                    i = 1;
                }
                else
                {
                    Pen redPan = new Pen(Color.Red, 2);
                    dc.DrawEllipse(redPan, 0, 50, 50, 50);
                    DateTime start = DateTime.Now;
                    //最多循环10秒
                    while ((DateTime.Now - start).Seconds < 2)
                    {
                        Application.DoEvents();
                    }
                    i = 2;
                }
            }
这段代码有什么问题？

Answer 1

whitePen ,redPan 都没释放。看你命名变量是懂gdi的，居然没有释放资源的好习惯。
在OnPaint中DoEvents，如果此时被遮挡或是引发重绘，软件就会死掉了。很容易死循环的做法。

Answer 2

自己发出来看下protected override void OnPaint(PaintEventArgs e)
  {
  base.OnPaint(e);
  Graphics dc = e.Graphics ;
  int i;
  i = 2;
  while (true)
  {
  if (2 == i)
  {
  Pen whitePen = new Pen(Color.White, 2);
  dc.DrawEllipse(whitePen, 0, 50, 50, 50);
  DateTime start = DateTime.Now;
  //最多循环10秒
  while ((DateTime.Now - start).Seconds < 2)
  {
  Application.DoEvents();
  }
  i = 1;
  }
  else
  {
  Pen redPan = new Pen(Color.Red, 2);
  dc.DrawEllipse(redPan, 0, 50, 50, 50);
  DateTime start = DateTime.Now;
  //最多循环10秒
  while ((DateTime.Now - start).Seconds < 2)
  {
  Application.DoEvents();
  }
  i = 2;
  }
  }

Answer 3

try: exp: 
       private void button1_Click(object sender, EventArgs e)
        {
            Thread thr = new Thread(new ThreadStart(Run));
            thr.IsBackground = true;
            thr.Start();
        }        private void Run()
        {
            int i = 0;
            while (true)
            {
                Graphics g = this.CreateGraphics();
                Pen whitePen = new Pen(Color.White, 2);
                Pen redPan = new Pen(Color.Red, 2);
                if (i % 2 == 0)
                {
                    g.DrawEllipse(whitePen, 0, 50, 50, 50);
                }
                else
                {
                    g.DrawEllipse(redPan, 0, 50, 50, 50);
                }
                g.Dispose();
                i++;
                Thread.Sleep(1000);
            }
        }

Answer 4

如楼上所说。执行pen对象的Dispose方法.因为OnPaint是在窗体收到WM_PAINT的时候调用的，而你的Application.DoEvents会执行消息循环，此时如果遮住屏幕又会触发OnPaint，你实际会嵌套很多次，尽量不要这么做，把这些内容放在一个定时器中实现。

Answer 5

我单步调试的时候，发现第一次DoEvents的时候又执行OnPaint了，我没有挡住窗体啊，我在最后dispose(),还是不行，还是提示有问题。

Answer 6

额。对，抱歉。
我写了释放语句。但是在程序停止的时候还是出错System.Runtime.InteropServices.ExternalException

Answer 7

要实时的释放资源，就是把X.Dispose（）放在timer控件的运行中