Do I need to implement a Dispose method?

[wh]

I have a class which when instatiated creates a new XmlDocument object and
loads an xml file into it. It is used as follows:

MyObject obj = new MyObject("books .xml");

If I later have

obj = new MyObject("books 2.xml");

Would all memory referenced by 'obj' (i.e. the XmlDocument) from the first
call be freed or would I be required to implement a Dispose method to
release the memory associated like:

public void Dispose()
{
m_Xml = null;
}

From what I can make out, as XmlDocument is a managed resource (with no
explicit Dispose() or Close() method) it will automatically be deallocated.

Wayne.

[Jon Skeet [C# MVP]]

Re: Do I need to implement a Dispose method?

wh <wayne@nospam.p yesmeadow.com> wrote:[color=blue]
> I have a class which when instatiated creates a new XmlDocument object and
> loads an xml file into it. It is used as follows:
>
> MyObject obj = new MyObject("books .xml");
>
> If I later have
>
> obj = new MyObject("books 2.xml");
>
> Would all memory referenced by 'obj' (i.e. the XmlDocument) from the first
> call be freed or would I be required to implement a Dispose method to
> release the memory associated like:
>
> public void Dispose()
> {
> m_Xml = null;
> }
>
> From what I can make out, as XmlDocument is a managed resource (with no
> explicit Dispose() or Close() method) it will automatically be
> deallocated.[/color]

Setting

obj = new MyObject("books 2.xml"); doesn't free up memory immediately,
but then neither does m_Xml = null;

You just don't free up memory immediately in .NET - the garbage
collector takes care of it. If you've got *unmanaged* resources within
your object, or references to any objects which themselves implement
IDisposable, you should dispose of those appropriately. However, that
should be a fairly rare occurrence, IMO.

--
Jon Skeet - <skeet@pobox.co m>

http://www.pobox.com/~skeet

If replying to the group, please do not mail me too

[wh]

Re: Do I need to implement a Dispose method?

Thank you for your reply.

So can I assume that *if* XmLDocument did implement a Dispose() method then
I should provide a Dispose() method in my class and call the Dispose() of
the XmlDocument?

Wayne.

[wh]

Re: Do I need to implement a Dispose method?

Thank you for your reply.

So can I assume that *if* XmLDocument did implement a Dispose() method then
I should provide a Dispose() method in my class and call the Dispose() of
the XmlDocument?

Wayne.

[Jon Skeet [C# MVP]]

Re: Do I need to implement a Dispose method?

wh <wayne@nospam.p yesmeadow.com> wrote:[color=blue]
> So can I assume that *if* XmLDocument did implement a Dispose() method then
> I should provide a Dispose() method in my class and call the Dispose() of
> the XmlDocument?[/color]

Yes.

--
Jon Skeet - <skeet@pobox.co m>

http://www.pobox.com/~skeet

If replying to the group, please do not mail me too

[Jon Skeet [C# MVP]]

Re: Do I need to implement a Dispose method?

wh <wayne@nospam.p yesmeadow.com> wrote:[color=blue]
> So can I assume that *if* XmLDocument did implement a Dispose() method then
> I should provide a Dispose() method in my class and call the Dispose() of
> the XmlDocument?[/color]

Yes.

--
Jon Skeet - <skeet@pobox.co m>

http://www.pobox.com/~skeet

If replying to the group, please do not mail me too