Diskusia o FreeAndNil - nemal by som ho používať

[Stanislav Hruška]

Tu je zaujímavá debata na: https://en.delphipraxis.net/topic/7035-freeandnil-the-great-delphi-developer-debate/ Rozumiem len niečomu, ale záver je, žeby som sa mal čo najviac vyhýbať použitiu FreeAndNil.
Škoda, že neviem pochopiť prečo. Niečo tam je vysvetlené, ale ako celok mi to uniká   Môj pohľad na to:

• Ak použijem lokálnu inštanciu, tak mám použiť .Free + nil(?)
• Ak vytvorím inštanciu v konštruktore a ničím ju v deštruktore, tiež mám použiť .Free + nil

Osobne iné prípady nemám.
Ten nil som dal s otáznikom z jedného dôvodu: Pozrel som sa na Free. Volá Destroy. Funkcia Destroy tam je prázdna. Preto neviem či sa ukazovateľ nastaví na nil !!!

[Radek Červinka]

Kód: Delphi [Vybrat]

1. procedure TObject.Free;
2. begin
3.   if Self <> nil then
4.     Destroy;
5. end;
6.  

IMHO lokalní proměnné stačí Free, pokud se jedna o globalni nebo proměnné třídy tak vždy FreeAndNil.

[Daniel_Andrascik]

No dnes som prvykrat po 20tich rokoch programovania narazil tiez na AV sposobenu vdaka pouzitiu FreeAndNil. Ako nakodil som narychlo len taku utilitku tak som si moc nerobil hlavu s pouzivanim globalnych premennych. Zvycajne ich v projekte mam absolutne minimum. Neviem ake vsetky "side efects" FreeAndNil vie vyvolat, ale jeden z problemov je ze FreeAndNil najprv do globalnej premennej zapise adresu nill a potom vola free. A v mojom pripade ten globalny objekt vo svojom destruktore este likvidoval dalsiu plejadu objektov a niektore z nich potrebovali este nejake doplnkove volania do povodneho globalneho objektu, ale tato premenna uz bola koli funkcii FreeAndNil "znillovana" takze dosla AV 🤷. Dost blbo sa to hlada a stacilo to rozpisat na dva riadky, najprv zavolat free a potom rucne priradit nill. Ale ako hovorim, 20 rokov sa mi to nestalo, pretoze takto hulvatsky vacsinou neprogramujem 

[Stanislav Hruška]

Ešte je zaujímavá iná skutočnosť.
Na DelphiPraxis sa vyjadrili, že volať pred FreeAndNill test na Assigned() je nezmysel. Lebo samotné FreeAndNil začína týmto testom.
Lenže zásadne ak dám FreeAndNil(MyQuery), pričom MyQuery už prešlo touto procedúrou, tak dostávam AV.

[Radek Červinka]

Ešte je zaujímavá iná skutočnosť.
Na DelphiPraxis sa vyjadrili, že volať pred FreeAndNill test na Assigned() je nezmysel. Lebo samotné FreeAndNil začína týmto testom.
Lenže zásadne ak dám FreeAndNil(MyQuery), pričom MyQuery už prešlo touto procedúrou, tak dostávam AV.

Tak tam máš nějaký odkaz na tu proměnnou v destructoru => mas to blbe napsane.

[pf1957]

Lenže zásadne ak dám FreeAndNil(MyQuery), pričom MyQuery už prešlo touto procedúrou, tak dostávam AV.

A je nejaky problem vtrasovat do destructoru a zjistit, co tam mas za bordel?

[Stanislav Hruška]

Nejde o ten bordel. Ten som odstránil. Ide o to, že FreeAndNill nemá čo vyhadzovať AV. Alebo sa mýlim?

[pf1957]

Nejde o ten bordel. Ten som odstránil. Ide o to, že FreeAndNill nemá čo vyhadzovať AV. Alebo sa mýlim?

Pokud se nejedna o multihreading (neni to atomicka operace) nebo se nejedna o nejakou prasarnu, jak psal Daniel a promenna ma hodnotu nil, tak samozrejme ne, protoze jak ti psali, bezna public metoda TObject.Free() zacina testem na nil.

Jinak samozrejme ano, vcetne nepredvidatelneho chovani pri operaci s dangling pointerem

[Stanislav Hruška]

Nič z toho neprichádza do úvahy   Je to obyčajný TQuery.

[Radek Červinka]

Nič z toho neprichádza do úvahy   Je to obyčajný TQuery.

Tak si to preloz s debug dcu a trasuj dovnitr.

[pf1957]

Nič z toho neprichádza do úvahy   Je to obyčajný TQuery.

Nejde o TQuery, ale o promennou, ktera obsahuje odkaz na instanci, pokud to neni automaticka promenna v zasobniku, tak jedna z moznosti je, ze sis nekde pod prd*li uvolnil sam sebe a dobiha ti metoda v objektu, jehoz instance uz byla zrusena -> pak dereference na member variable muze vest k AV, zrovna tak si nekde neco muzes prepisovat v pameti, pokud neco alokujes dynamicky v pameti.

Pokud to neni automaticka promenna, takze zivotni cyklus promenne/instance je omezen na scope subroutiny, tak jak psal Radek, nekde ve sprave zivotniho cyklu mas neco hooodne spatne

[Stanislav Hruška]

Je to pole triedy. Dá sa povedať, že som tam porušil, asi nepísané pravidlo, a premennú som uvoľnil po prvom a zároveň poslednom použití. Nech nezaberá miesto  . A to robilo problémy.
Presunul som to Destroy a je pokoj. Ešte mám možnosť to definovať ako lokálnu premennú. To sa mi nechce.

[František]

narazil som na toto este

https://dalijap-blogspot-com.translate.goog/2022/07/freeandnil-debate-thread-safety.html?_x_tr_sl=auto&_x_tr_tl=cs&_x_tr_hl=sk