Embarcadero príklad

[František]

http://docwiki.embarcadero.com/CodeExamples/Tokyo/en/Generics_Collections_TDictionary_(Delphi)

ja viem že je to len example
ale

Kód: Delphi [Vybrat]

1. City := TCity.Create;

v úvode 3x po sebe, nemá žiadne Free

je to košér?

lebo ak je to príklad, mal by isť príkladom, či?

[JaroB]

Vlastní ho Dictionary a to si ho uvolní ve svém Clear/Free, ne?

[Delfin]

je to košér?

Neni, protoze TDictionary se nestara o lifetime klicu ci hodnot. To umi TObjectDictionary.

[Stanislav Hruška]

v úvode 3x po sebe, nemá žiadne Free

Na samom konci je jedno uvoľnenie. To asi nepostačuje. Nejako som to neštudoval.

[Delfin]

Neni, protoze TDictionary se nestara o lifetime klicu ci hodnot. To umi TObjectDictionary.

...a protoze se nikde v kodu neobjevuje extrakce instance (hodnoty slovniku) s jejim naslednym uvolnenim. Celkove je ten priklad pomerne nepekny.

[František]

takže kedy budú uvoľnené vytvorené objekty city, ktoré boli pridané do Dictionary?

[vandrovnik]

takže kedy budú uvoľnené vytvorené objekty city, ktoré boli pridané do Dictionary?

Při běhu toho programu nebudou uvolněné (memory leak). Po ukončení aplikace je celá ta paměť vrácena operačnímu systému.

[František]

takže ak to chcem využívať počas celého behu programu, tak to nemusím teda vôbec riešiť?

[vandrovnik]

Normální životní cyklus by měl být takový, že vytvořím, držím si odkaz a používám, používám, používám, a když už používat nechci, tak uvolním. Já jsem zvyklý to mít zhruba takto:

Kód: Delphi [Vybrat]

1. m:=tMemoryStream.Create;
2. try
3.  ...
4.  ...
5. finally
6.  FreeAndNil(m);
7. end;

[Stanislav Hruška]

Prakticky áno. Ak to chceš mať z programátorskeho hľadiska čisté, tak to pri ikončení aplikácie uvoľníš. Taký FMM4 by ti kričal, že máš memory leak. Len taká podotázočka: používaš ho?

[vandrovnik]

Prakticky áno. Ak to chceš mať z programátorskeho hľadiska čisté, tak to pri ikončení aplikácie uvoľníš. Taký FMM4 by ti kričal, že máš memory leak. Len taká podotázočka: používaš ho?

Správná připomínka - aby "to křičelo", dá se přidat do .dpr souboru, nejlépe hned za begin:

Kód: Delphi [Vybrat]

1.  {$IFDEF DEBUG}
2.   System.ReportMemoryLeaksOnShutdown:=true;
3.  {$ENDIF}
4.  

[Stanislav Hruška]

FMM4 je, aspoň podľa návodu čo som čítal o ReportMemoryLeaksOnShutdown, pružnejší. Vypíše memory leak už pri uzavretí formulára. To značne uľahčuje prácu.

[Radek Červinka]

FMM4 je, aspoň podľa návodu čo som čítal o ReportMemoryLeaksOnShutdown, pružnejší. Vypíše memory leak už pri uzavretí formulára. To značne uľahčuje prácu.

Jelikož ReportMemoryLeaksOnShutdown je testována až ve finalization sekci FastMM4.pas tak to jednoznačně není pravda, navíc nechápu jak by to mělo fungovat, protože i z různých formulářů můžeš alokovat třeba globální proměnné nebo cokoliv jiného a ten memory manager by to v tvém případě reportoval. Prostě je to nesmysl, možná si to pleteš se zavřením posledního formuláře.

[František]

takže keď v OnCretae naplním v cykle Dictionary s TCity, musím/mal by som to pri OnClose uvolniť? alebo stačí uvolniť Dictionary (a sním pôjde aj city)?

[vandrovnik]

Když něco vytvoříš v OnCreate, tak to obvykle chceš uvolnit v OnDestroy. V OnClose bych to nedělal, protože zavřený formulář je možné opakovaně zobrazit - někdy se to hodí, protože se zobrazí rychleji, než když se musí nejdřív vytvořit.

tDictionary se o uvolnění nepostará, ale tObjectDictionary to umí - při volání Create mu předáváš jako parametr, zda se má starat o život klíčů a život hodnot.