一次性对象克隆是否会导致C#内存泄漏？

Question

检查此代码:

.. class someclass : IDisposable{
    private Bitmap imageObject;
    public void ImageCrop(int X, int Y, int W, int H)
    {
        imageObject = imageObject.Clone(new Rectangle(X, Y, W, H), imageObject.PixelFormat);
    }
    public void Dispose()
    {
        imageObject.Dispose();
    }
}

Bitmap是ICloneable,IDisposable在C#中.

为了避免内存泄漏,对于Disposable对象,通常使用using,无论代码有多么错误,对象都将由系统自动处理.

在我的示例中,我无法使用,using因为我不想处理对象,我以后需要它(整个类也将自己处理它自己IDisposable.

我的问题是:我有一个imageObject对象,然后我用它Clone()方法克隆一个新对象并将其赋予旧的对象变量.这会导致一个(克隆的或原始的)对象无处可去,永远不会被处理,内存泄漏.

[编辑]

似乎大多数意见是Clone造成额外的对象,旧的应该是Dispose()

这是新代码:

    public void ImageCrop(int X, int Y, int W, int H)
    {
            // We have 1 object: imageObject
            using (Bitmap croppedImage = imageObject.Clone(new Rectangle(X, Y, W, H), imageObject.PixelFormat))
            {
                    // We have 2 objects: imageObject and croppedImage
                    imageObject.Dispose(); // kill one, only croppedImage left
                    imageObject = new Bitmap(croppedImage); // create one, now 2 objects again
            } // the using() will kill the croppedImage after this
            // We have 1 object: imageObject
    }

它应该适当处理资源.

Answer 1

避免资源泄漏或过早处置错误的关键是确保每个 IDisposable 对象始终都有一个明确定义的所有者负责处置它。有时，对象会公开一个方法，通过该方法它将获得传入对象的所有权。如果对象的所有者将其传递给这样的方法，则该对象的原始所有者不应处置它。否则，对象的所有者必须在销毁对象的最后一个引用之前释放该对象。

如果someClassowns ImageObject，那么它可能应该在销毁对该对象的引用之前处置该对象。另一方面，如果一个对象持有对另一个对象的唯一引用，则为了重新分配原始引用而克隆所持有的对象似乎有点代码味道。我不知道 ImageObject 最初是如何分配的，但似乎它应该在您的对象中创建，或者基于传入的图像对象进行克隆。无论哪种情况，您都应该能够对传入图像的类型进行足够的控制，以选择可以裁剪而无需（重新）克隆的类型。