Zamykanie formy (Form2)

[Corvis]

jaki błąd screen daj

[Miroo_007]

Oto zdjęci wyskakującego błędu

bez tytułu.JPG

[Corvis]

spakuj projekt mailem i mi wyślij placek2002 małpa wp.pl bo normalnie nie wierze

[Miroo_007]

Witam
W formie Form1 mam (w ciele lokalnej funkcji):

Kod: Zaznacz cały

TForm2 *form_2 = new TForm2(this);
form_2->ShowModal();
delete form_2;
form_2 = NULL;

W formie Form2 mam:

Kod: Zaznacz cały

this->Free();
this->Close();

Wyskakuje mi błąd pamięci przy zamknięciu Form2. Jak można zakończyć instrukcje form_2->ShowModal() i przejść do następnej linijki?

[Cyfrowy Baron]

musisz usunąć z Form2 polecenia this->Close() i this->Free.
Umieść w zdarzeniu OnClose dla Form2 polecenie

Kod: Zaznacz cały

void __fastcall TForm1::FormClose(TObject *Sender, TCloseAction &Action)
{
Action = caFree;
}


[banita]

nie do konca poniewaz polecenie:

Action = caFree;

zawarte w zdarzeniu OnClose powoduje ze formatka po zamknieciu sama po sobie sprzata, czyli zwalnia pamiec.
co za tym idzie po wyjsciu z funkcji ShowModal nie mozesz drugi raz zwalniac pamieci po formatce.
musisz zdecydowac sie na jedno z tych rozwiazan.

osobiscie gdy wywoluje formatke modalnie to jawnie wywoluje delete xyz, natomiast gdy wywoluje ja niemodalnie to kozystam z samozwalniania pamieci w zdarzeniu onClose.

co do wywolania this->Free() to wczesniej Witold cytowal fragment manuala na ten tamat. gdybys czytal co userzy ci odpisuja nie mialbys tego problemu.

[Miroo_007]

Zacznę od początku. Żadna z powyższych odpowiedzi nie jest poprawna. Poprawne jest coś takiego:

Kod: Zaznacz cały

//Unit1.cpp
TForm2 *form_2 = new TForm2(this);
form_2->ShowModal();
delete form_2;
form_2 = NULL;
//Unit2.cpp
this->Close();

Podpowiedź autorstwa Corvisa.

[Corvis]

Tak jak ci w mailu napisałem ten caly projekt ma źle zrobione zarządzanie formatkami. W miare możliwośći postaram ci sie to poprawić

[Miroo_007]

Dynamiczne zarządzanie formatkami poprawiłem. Jest jakiś program do badania wycieków pamięci?

[polymorphism]

Kod: Zaznacz cały

...
delete form_2;
form_2 = NULL; <---

A to po co?

[Corvis]

Dynamiczne zarządzanie formatkami poprawiłem. Jest jakiś program do badania wycieków pamięci?

Project Optrions C++ Compiler Debugging zaznacz Enable CodeGuard options

wtedy jeżeli ci pamięć wycienie dostaniesz komunikat z informacją gdzie się to stało

Miroo_007 napisał(a):

Kod: Zaznacz wszystko
...
delete form_2;
form_2 = NULL; <---


A to po co?

Przyłączam się do pytania

[banita]

Zacznę od początku. Żadna z powyższych odpowiedzi nie jest poprawna. ....

chyba czytasz inny temat skoro tak twierdzisz

[Miroo_007]

Kod: Zaznacz cały

...
delete form_2;
form_2 = NULL; <---

A to po co?

http://www.yevol.com/en/bcb/Lesson18.htm. Temat: Customizing Dynamic Forms

Furthermore, to avoid any memory leak, you can reclaim the dynamically allocated memory by assigning NULL to the deleted object:

Kod: Zaznacz cały

//---------------------------------------------------------------------------
void __fastcall TForm1::FormDblClick(TObject *Sender)
{
   TForm1 *Cosmos = new TForm1(this);
   Cosmos->ShowModal();

   delete Cosmos;
   Cosmos = NULL;
}
//---------------------------------------------------------------------------

[polymorphism]

Chyba nie do końca przeczytałeś tutora:

In reality, since you can use your judgment to decide when the object would be displayed, when the form that owns it is closed, before deleting the dynamic object, you can first find out if the object was really created. This can be performed with a simple if statement.



Kod: Zaznacz cały

void __fastcall TForm1::FormDestroy(TObject *Sender)
{
   if( Fake )
   {
      delete Fake;
      Fake = NULL;
   }
}


Czyli to zerowanie wskaźnika ma jakiś sens. W przypadku twojego kodu jest to zbędne. Tworzysz obiekt lokalnie, więc wskaźnik form_2 po przejściu tego fragmentu kodu zawsze będzie wskazywał na obiekt, który został usunięty z pamięci. A wszelkie odwoływanie się do tego wskaźnika w dalszej części kodu, bez ponownej inicjalizacji, jest błędem (którego nawet ten NULL nie załatwi ).

[usermaw]

jesli Forma jest modalna, to sama sie niszczy przy zamykaniu i nie ma potrzeby wstawiania delete wskaznik