检查此代码：
.. 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();
}
}
位图是ICloneable,IDis可以在C#中使用.
为了避免内存泄漏,对于Disposable对象,通常使用,那么无论你的代码有多么错误,对象都会被系统自动处理.
在我的示例中,我不能使用,因为我不想处理对象,我以后需要它(整个类也将自己处理,因为它的IDisposable也是如此.
我的问题是：我有一个imageObject对象,然后我使用它Clone()方法克隆一个新对象并将其提供给旧的对象变量.这会导致一个(克隆的或原始的)对象无处可去,永远不会被处理,内存泄漏.
[编辑]
似乎大多数意见都是Clone造成的附加对象,旧的应该是Dispose()
这是新代码：
public void ImageCrop(int X,int H)
{
// We have 1 object: imageObject
using (Bitmap croppedImage = imageObject.Clone(new Rectangle(X,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
}
它应该适当处理资源.
解决方法
避免资源泄漏或过早丢弃错误的关键是确保每个IDisposable对象始终只有一个明确定义的所有者负责处理它.有时,对象将公开一种方法,通过该方法,它将承担传入对象的所有权.如果对象的所有者将其传递给此类方法,则该对象的原始所有者不应将其丢弃.否则,对象的所有者必须在销毁对象的最后一个引用之前处置该对象.
如果someClass拥有ImageObject,那么它应该在销毁对象之前处置该对象.另一方面,如果一个对象持有对另一个对象的唯一引用,为了重新分配原始引用而克隆被保持的对象看起来有点像代码味道.我不知道最初如何分配ImageObject,但它似乎应该在您的对象中创建,或者基于传入的图像对象进行克隆.在任何一种情况下,您都应该能够对传入图像的类型进行足够的控制,以选择可以裁剪而无需(重新)克隆的类型.
