C# 图形不在需要的地方绘制矩形答案

【问题标题】：C# Graphics not drawing rectangles where requiredC# 图形不在需要的地方绘制矩形
【发布时间】：2019-03-28 12:37:59
【问题描述】：

我正在创建一个图形表单，其中坐标为 x,y 的对象被绘制到图形中。它适用于小的 x 和 y，但是当我想在不同的位置（例如 x = 500，y = 300）绘制它们时，它们会消失。

 public WindowHandler()
        {
            dc = this.CreateGraphics();

            this.Size = new Size(sizeX, sizeY); // 800x600
            startSimulation = new Button
            {
                // button properties
            };

            this.Controls.Add(startSimulation);
            startSimulation.Click += new EventHandler(StartSimulationClick);

        }

        private void CreationsMethods()
        {
            creations.PaintAllAnimals(dc);
        }

        public void PaintAllAnimals(Graphics g)
        {
            foreach (var animal in ecoStructure.world.animals)
            {
                animal.PaintAnimal(g);
            }
        }

        public void PaintAnimal(Graphics graphics)
        {
            Rectangle rectangle = new Rectangle(x, y, 3, 3);
            Pen pen = new Pen(colour);
            graphics.DrawRectangle(pen, rectangle);
            graphics.FillRectangle(colour, rectangle);
        }

我想把所有的对象都放到窗口上。有没有办法让图形“更大”？我需要再做一个吗？或者我应该使用不同的工具来绘制矩形？

【问题讨论】：

你在哪里画画？
我使用我在代码中发布的所有内容。 dc 是 Graphics，我用它来绘制（PaintAnimal 方法）。我也使用System.Drawing; using System.Windows.Forms;
看起来像这样：link。如您所见，右侧矩形在中间被“切割”，因为它应该以这个特定的坐标 (x,y) 绘制。它们也应该有更多（总共 15 个），但它们正在从这个图形中删除。
问题是你使用了CreateGraphics，你不应该这样做。 Winform 应用程序中的所有绘制都应在OnPaint 覆盖或Paint 事件处理程序中完成。下次 Windows 决定重新绘制窗口时，您使用 CreateGraphics 进行的任何绘制都将被覆盖。顺便说一句，您正在创建一个 Pen 对象并且永远不会处理它。这是句柄泄漏。
好的！非常感谢。我会阅读并更改它。

标签： c# graphics size

【解决方案1】：

感谢@Chris Dunaway 在评论中发布答案。所以我删除了 CreateCraphics，而我现在使用的是 OnPaint 方法。它工作缓慢，但有效。所以我会尽可能快地做到这一点。现在，我刚刚创建了这个。 NextStepClick 是我使用 OnPaint 绘制矩形的方法。

private void CreationsMethods(object sender, PaintEventArgs e)
        {
            dc = e.Graphics;
            base.OnPaint(e);
            creations.PaintAllAnimals(dc);
        }
private void NextStepClick(object sender, EventArgs e)
        {
            this.Refresh();
            picBox.Paint += new System.Windows.Forms.PaintEventHandler(CreationsMethods);
        }

【讨论】：

picBox.Paint +=.. 请勿多次挂机！！！（至少没有再次解开它。） Click 事件似乎是一个可疑的地方..
如何“解开”它？
picBox.Paint -=.. 但是，正如我所说，最好将它连接一次，就像您执行任何其他事件一样，例如鼠标点击或 selectionchanged 等。如果它被连接多次，它将运行多次，这可能是您感觉缓慢的原因..
@TaW hooking 或 unhooking 的正确术语是 subscribing 和 unsubscribing。 @LauraAlice 似乎对unhook这个词感到困惑@
哦，是的，我就是这么理解的。感谢大家的建议，现在运行顺利！