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

本文关键字：内存 泄漏 对象 一次性 | 更新日期: 2023-09-27 18:19:42

检查此代码：

.. 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。

为了避免内存泄漏，对于一次性对象，通常使用using，无论您的代码有多错误，系统都会自动处理该对象。

在我的示例中，我不能使用using，因为我不想处理对象，我稍后需要它（整个类也会处理它的IDisposable）。

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

[EDIT]

似乎大多数意见都是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
    }

并且应该适当地处置资源。

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

using只调用finally块中的Dispose
只要在所有代码路径中的某个位置调用Dispose，就可以了。

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

在这种特殊的情况下，您可能应该在克隆原始文件后处理它，因为看起来您再也不用它了。

我不能肯定，但如果你担心它可能会发生，为什么不将图像克隆到一个新的变量，处理原始图像，然后重新分配：

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();
}

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

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

是的，这可能是泄漏。

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

使用关键字只是手动尝试finally并调用Dispose（）的一种方便方法。如果在您的情况下，您不能使用"using"，那么只需使用try finally块，并确保悬挂的资源已清理干净。

托管代码中的内存不会泄漏，但可能会导致资源泄漏。位图是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);
  }
}

假设Clone（）正常工作，它将为您提供2个可处理对象进行管理。两者都需要Dispose（）。

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

方法返回IDisposable对象并不罕见，您只需确保在更高级别上进行（异常安全）资源管理即可。请小心操作。

相关文章：