Dispose a Finalize – .NET – Fórum – Programujte.com

Zdravim,

jsem celkem novy v ASP.NET, tak prosim o jednu radu.

Vytvarim zkusebni tridu na zapis do log souboru. Rad bych ji udelal se vsim vsudy, takze i Dispose a Finalize metody. Je tento kod v poradku, nebo je neco zbytecne navic / neco chybi ke spravnemu vytvoreni a zniceni tridy.

    class LogIt: IDisposable

    {

        FileStream fsLog;

        StreamWriter swLog;



        public LogIt(string logFile)

        {

            fsLog = new FileStream(logFile, FileMode.Append);

            swLog = new StreamWriter(fsLog);

        }



        public void Dispose()

        {

            swLog.Close();

            swLog.Dispose();

            swLog = null;



            fsLog.Dispose();

            fsLog = null;



            GC.SuppressFinalize(this);

        }

        



        ~LogIt()

        {

            this.Dispose();

        }



        

    }

using (LogIt myLog = new LogIt("LogIt.txt"))

{

  necoDelej();

}

dal jsem do Googlu "microsoft patterns IDisposable" a docela poucne jsou hned prvni dva odkazy http://msdn.microsoft.com/en-us/library/fs2xkftw.aspx a http://msdn.microsoft.com/en-us/magazine/cc163392.aspx
Jinak se zda, ze ses vydal v celku dobrou cestou. Ale pozor muze se stat, ze pokud bude cesta k souboru nejaky nesmysl, tak program vyhodi IOException a fsLog i swLog zustanou null a pak to vyhodi jeste NullReferenceException v Dispose...

ja by som si vytvoril:

public static class Logger

{

	static bool active;

	static Stream file;



	public static void Activate(string filepath);

	public static void Deactivate();

	public static void Add(string whatyouwant);

}

Diky za reakce.

Me slo predevsim o to, aby ten objekt byl za vsech okolnosti spravne odstranen z pameti. To, ze to nevo loguje do souboru neni az tak podstatne. Spis to beru jako nejakou kostru, jak dobre udelat finalize a dispoze.

Na tom prvnim http://msdn.microsoft.com/en-us/library/fs2xkftw.aspx od rudolf_franek je to prave delany trochu jinak.

Jednak je pouzita promenna _disposed. je to nutny? Mohlo by se stat, ze se to bude disposovat dvakrat a hodi to nejakou chybu?

Test na nenullovost u me urcite chybi.


Dalsi dotaz z druheho linku:

public class DisposableClass : IDisposable

{

    ~DisposableClass()

    {

        Dispose(false);

    }



    public void Dispose()

    {

        Dispose(true);

        GC.SuppressFinalize(this);

    }



    protected virtual void Dispose(bool disposing)

    {

        if (disposing)

        {

            // Clean up all managed resources

        }

            

        // Clean up all native resources

    }

}

1) Unmanaged resource by ve vyse uvedenem pripade byl prave uvedeny Stream. Managed by mohla byt nejaka dalsi vlastni trida implementujici IDisposable. Nebo napriklad Image, ktery ma v sobe unmanaged resource - referenci na soubor, ze ktereho obrazek ctu.
2) To ze si nekam zaznamenam, ze probehnul korektne dispose je OK. Nejde o to, ze by podruhe v Dispose doslo k chybe, ale pokud je dobre napsany, pak by melo byt zbytecne volat ho dvakrat. I kdyz soucasne pokud je dobre napsany, tak by melo byt mozne ho volat kolikrat chces a nedojde k zadne chybe.
Z jine stranky jde ale take o to, ze nekdy se zavola explicitne Dispose (napriklad ve finally), nekdy je pouzit using a pokud nenastane ani jeden pripad, tak je nezbytne zavolat Dispose ve finalizaci - ale melo by byt zbytecne volat Dispose vickrat nez jednou (proto se pouziva mj. to GC.SuppressFinalize(this))
3) Proc se pri finalizaci odstranuji jen unmanaged resources ? Kdyz zavola program explicitne Dispose(), pak je na miste se zbavit vseho. Pokud se tak nestane a prestanu "pouze" objekt pouzivat - a Dispose(bool) se zavola az na popud garbage collectoru, pak je samozrejme treba uvolnit unmanaged resources - abych nekde na neznamem miste v pameti nedrzel napriklad otevreny soubor - ale managed resources by pravdepodobne mel GC zvladnout uvolnit sam.

To rudolf_franek :

Diky za odpovedi.

Takze kdyz tam ten boolovskej parametr nedam a uvolnim vzdycky vsechno, tak je to OK. Jen tim da se rict usnadnim praci GC, kterej to pak nebude muset dohledavat jako nepouzivany resources a rusit nasledne. Je to tak?