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在C＃中为ICloneable，IDisposable。

为了避免内存泄漏，对于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

using只需在Dispose块中调用finally 只要您在所有代码路径中调用Dispose 某处，就可以了。

如果您不致电Dispose，GC将最终为您处理，但这可能会导致资源争用。

在这种特殊情况下，你应该在克隆它之后处理它，因为看起来你再也不会使用它了。

Answer 2

我不能肯定地说，但是如果你害怕它可能，为什么不将图像克隆到新变量，处理原始图像，然后重新分配：

public bool ImageCrop(int X, int Y, int W, int H)
{     
    Bitmap croppedImage = imageObject.Clone(new Rectangle(X, Y, W, H), imageObject.PixelFormat);
    imageObject.Dispose();
    imageObject = new Bitmap(croppedImage);
    croppedImage.Dispose();
}

Answer 3

是的，这可能是泄密。

如果要制作一次性对象的副本（在您的情况下为Bitmap），则应立即处置不再需要的实例。

使用关键字只是手动拥有try-finally并调用Dispose（）的一个方便之处。如果在你的情况下你不能使用'使用'，那么只需使用try-finally块并确保清理悬空资源。

Answer 4

内存不会在托管代码中泄漏，但您可能会导致资源泄漏。位图是Windows中较低级别对象的包装器，它是一次性的，以便正确清理较低级别的对象。如果您将物品放置在外面，它们通常应该在一段时间后由垃圾收集器处理，但不能保证它实际上会被丢弃。

克隆图像会创建一个应该自行处理的新对象。当克隆被原始图像替换时，您应该将原始图像处理掉。您可以使用using关键字：

public bool ImageCrop(int X, int Y, int W, int H) {
  using (Bitmap original = imageObject) {
    imageObject = original.Clone(new Rectangle(X, Y, W, H), imageObject.PixelFormat);
  }
}

Answer 5

假设Clone（）正常工作，它将为您提供2个Disposable对象来管理。两者都需要Disposed（）。

所以我认为它不会解决你的问题。

方法返回IDisposable对象的情况并不少见，您只需要确保更高级别的（异常安全）资源管理。这么仔细。

Answer 6

避免资源泄漏或过早丢弃错误的关键是确保每个IDisposable对象始终只有一个明确定义的所有者负责处理它。有时，对象将公开一种方法，通过该方法，它将承担传入对象的所有权。如果对象的所有者将其传递给此类方法，则该对象的原始所有者应该不处置它。否则，对象的所有者必须在销毁对象之前处置它。

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

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

6 个答案: