8

我有以下代码,它将OutOfMemoryException在运行时生成:

public partial class MainWindow : Window
{
    private DrawingVisual myVisual = new DrawingVisual();

    public MainWindow()
    {
        InitializeComponent();
    }

    private void Window_Loaded(object sender, RoutedEventArgs e)
    {
        myVisual = GetVisual();
        graphicsCanvas.AddVisual(myVisual);
    }

    private void Graphics_Canvas_MouseMove(object sender, MouseEventArgs e)
    {
        //get the data visual:
        DrawingVisual tempVisual = GetVisual();

        //first clear the current display data:
        graphicsCanvas.RemoveVisual(myVisual);

        //get the data visual:
        myVisual = tempVisual;

        graphicsCanvas.AddVisual(myVisual);

        //GC.Collect();
    }

    private DrawingVisual GetVisual()
    {
        double width = graphicsCanvas.ActualWidth;
        double height = graphicsCanvas.ActualHeight;

        DrawingVisual dV = new DrawingVisual();

        Rect clipRect = new Rect(0, 0, width, height);

        dV.Clip = new RectangleGeometry(clipRect);

        using (DrawingContext dC = dV.RenderOpen())
        {
            RenderTargetBitmap rTB = new RenderTargetBitmap((int)width, (int)height, 96, 96, PixelFormats.Pbgra32);

            if (rTB.CanFreeze)
            {
                rTB.Freeze();
            }

            dC.DrawImage(rTB, clipRect);
        }

        return dV;
    }
}

其中 aGraphics_Canvas定义如下:

class Graphics_Canvas : Canvas
{
    private List<DrawingVisual> visuals = new List<DrawingVisual>();

    protected override int VisualChildrenCount
    {
        get { return visuals.Count; }
    }

    protected override Visual GetVisualChild(int index)
    {
        return visuals[index];
    }

    public void AddVisual(DrawingVisual visual)
    {
        visuals.Add(visual);

        base.AddVisualChild(visual);
        base.AddLogicalChild(visual);
    }

    public bool ContainsVisual(DrawingVisual visual)
    {
        return visuals.Contains(visual);
    }

    public bool HasVisuals
    {
        get { return visuals.Count > 0; }
    }

    public void RemoveAllVisuals()
    {
        for (int i = 0; i < visuals.Count; i++)
        {
            RemoveFromLogicalTree(visuals[i]);
        }

        visuals.Clear();
    }

    private void RemoveFromLogicalTree(Visual visual)
    {
        RemoveLogicalChild(visual);
        RemoveVisualChild(visual);
    }

    public void RemoveLastVisual()
    {
        if (visuals.Count > 0)
        {
            int index = visuals.Count - 1;

            RemoveFromLogicalTree(visuals[index]);
            visuals.Remove(visuals[index]);
        }     
    }

    public void RemoveVisual(DrawingVisual visual)
    {
        RemoveFromLogicalTree(visual);
        visuals.Remove(visual);            
    }
}

创建的 XAMLWindow是这样的:

<Window
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:local="clr-namespace:csMemoryLeakTestProject" x:Class="csMemoryLeakTestProject.MainWindow"
    Title="MainWindow" 
    Background="Gray"
    Height="600" Width="800"
    WindowStartupLocation="CenterScreen"
    Loaded="Window_Loaded">
    <Grid>
        <local:Graphics_Canvas Margin="12" Background="White" x:Name="graphicsCanvas" MouseMove="Graphics_Canvas_MouseMove"/>
    </Grid>
</Window>

现在,这只是一个例子来说明我的观点,即我不明白这里使用垃圾收集器的替代方法是什么......

如果您运行该程序,并继续将鼠标Graphics_Canvas移到OutOfMemoryException. 如果您在GC.Collect()我注释掉的地方添加,这不会发生(尽管内存确实增加了少量,原因超出我的范围,并希望与我的问题有关),并且程序继续运行。

那么,为什么不GC启动并清除此内存并停止发生异常?如果我犯了一些非常基本的错误,我会很高兴有人向我指出,这样我就可以超越这一点。

我在这个网站上多次看到程序员的其他建议说“永远不要使用垃圾收集器”。我想遵循最佳实践,但在这种情况下我看不到我还能做什么。

4

2 回答 2

10

GC程序无法管理太多内存不是错。在您的程序中,您可以:

private void Graphics_Canvas_MouseMove(object sender, MouseEventArgs e)
{
   ....
    //ADD ELEMENTS ON EVERY MOVE !
    graphicsCanvas.AddVisual(myVisual);

    //GC.Collect();
}

Tou 为每次鼠标移动添加一个元素,因此增加集合大小。你会期望发生什么?

因此,在90%的情况下,这类问题的解决方案是重新架构您的代码。

例子:

几乎不可能每次都在 MouseMove 上将新元素添加到儿童视觉集合中。可能是重复使用已经存在的情况。

GC建议显式使用 of ,但有时我们需要使用它。但是,我再说一遍,我几乎不敢相信在这种情况下您需要以这种方式管理程序。

于 2013-03-25T09:30:33.920 回答
-1

我认为这描述了可能使您的对象保持活力的原因;

http://msdn.microsoft.com/en-us/library/bb613565.aspx

简而言之; 事件处理程序中的引用可能仍然存在。

于 2013-03-25T09:51:41.433 回答