Restart

[polymorphism]

To, że działa, nie znaczy, że spełnia założenia. Równie dobrze możesz wywalić tę klasę, a za wywołaniem Close po prostu wstawić wywołania SetEvent i CloseHandle - efekt będzie dokładnie ten sam. Użycie tej klasy miało gwarantować to, że event przejdzie w stan signaled w momencie, gdy aplikacja zostanie zamknięta, tzn. wtedy, gdy wyskoczy z funkcji WinMain.

[Darek_C++]

Ja do restartu programu czyli jego zamkni ecia i ponownego uruchomienia używam takiego prostego kodu:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

int h = Application->MessageBoxA("Wymagane jest ponowne uruchomienie programu", "Uwaga",mbNo);
if(h==1)
{
        Application->Terminate();
        Sleep(400);
        ShellExecute(Application->Handle, "open", Application->ExeName.c_str(), "-r", "", SW_SHOW);
        Beep(600, 100);

}
void __fastcall TForm1::FormCreate(TObject *Sender)
{
        if(ParamStr(1) == "-r")
        {

                StatusBar1->Panels->Items[1]->Text = "Zrestartowano program";
        }
        else
        {
                StatusBar1->Panels->Items[1]->Text = "Gotowy";
        }
}
 

[Cyfrowy Baron]

I już na samym początku mam zastrzeżenie do Twojego kodu:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

int h = Application->MessageBoxA("Wymagane jest ponowne uruchomienie programu", "Uwaga",mbNo);

Jeżeli przekazujesz tylko flagę mbNo, to masz do dyspozycji tylko opcję NIE. Poza tym do Application->MessageBox stosuje się zupełnie inne flagi. Ta której użyłeś jest stosowana raczej przy tworzeniu komunikatów np. za pomocą funkcji CreateMessageBox.
Tak to powinno wyglądać:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

  int h = Application->MessageBoxA("Wymagane jest ponowne uruchomienie programu!\Czy chcesz restartować program?", "Uwaga", MB_YESNO | MB_ICONQUESTION);

 if(h == ID_YES)
 {
        Close();
        Sleep(400);

        ShellExecute(Application->Handle, "open", Application->ExeName.c_str(), L "", L"", SW_SHOW);

        Beep(600, 100);
}

[Darek_C++]

Dlaczego close() [skoro to tyczy zamykania całej aplikacji], a nie Application->Terminate(); ?

[Cyfrowy Baron]

Terminate odnosi się do wszystkich okien aplikacji, więc niezależnie od tego z którego okna wywołasz tą funkcje, to zamknięta zostanie cała aplikacja. Close zamyka tylko okno dla którego została ta funkcja wywołana, jeżeli jest to okno główne aplikacji to zamknięta zostanie cała aplikacja ze wszystkimi oknami.

Close zwalnia wszystkie zasoby aplikacji przed zamknięciem, a Terminate zabija proces i pozostawia w pamięci śmieci po aplikacji. Terminate zamyka bezwarunkowo aplikację i powinno się jej używać tylko gdy w aplikacji istnieje ryzyko wystąpienia błędów i nie da się zakończyć jej działania funkcją Close.

[polymorphism]

(...) a Terminate zabija proces i pozostawia w pamięci śmieci po aplikacji.

Nic z tych rzeczy. Terminate wysyła komunikat WM_QUIT, co powoduje, że aplikacja zostaje zamknięta w ten sam sposób, jakbyś zamknął jej główne okno. Pomyliłeś tę metodę z funkcją TerminateThread.

[Pitek]

Wszystko działa z poprawionym kodem od polimorphism ale dalej pojawia się to o czym mówiłem jeżeli od otwarcia programu do restartu wykonam jakieś działanie to restart nie spełnia swojej funkcji tylko zamyka program. Natomiast jeżeli od otwarcia programu nie robię nic tylko od razu go restartuje to jest ok. Czemu się tak dzieje jaki kod mam pokazać gdzie mogę robić błąd ?

[polymorphism]

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

String nazwa = ExtractFileName(Application->ExeName);

...

        ShellExecute(NULL,"open",(nazwa).t_str(),NULL,NULL,SW_SHOW);
 

Nigdy nie podawaj względnych ścieżek, jeśli nie masz pewności co do aktualnego katalogu.

[Cyfrowy Baron]

Czyli powinieneś zrobić to tak:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

ShellExecute(NULL, NULL, Application->ExeName.t_str(),NULL,NULL,SW_SHOW);

Co zresztą podałem w swoim kodzie.

---

Nic z tych rzeczy. Terminate wysyła komunikat WM_QUIT, co powoduje, że aplikacja zostaje zamknięta w ten sam sposób, jakbyś zamknął jej główne okno.

Sugerowałem się tym fragmentem z pliku pomocy:

By calling Terminate rather than freeing the application object, you allow the application to shut down in an orderly fashion.

Jak rozumiem zamiast zwalniania obiektów aplikacji pozwala zamknąć... i to orderly fashion jest dla mnie trochę niezrozumiałe - sposób uporządkowany, ale co to oznacza.

[polymorphism]

To 'in orderly fashion' rozumiem jako 'we właściwy sposób'. I nie 'obiektów aplikacji', tylko 'application object', co ja przetłumaczyłem jako 'obiekt Application'.

Czyli całe zdanie przetłumaczyłbym tak:

Wywołując Terminate zamiast usuwać obiekt Application, zamykasz aplikację we właściwy sposób.

[Cyfrowy Baron]

Co to jednak oznacza: zamiast usuwać obiekt Application?
Poza tym Close(), też zamyka aplikację prawidłowo, więc w czym tkwi różnica?

[polymorphism]

Co to jednak oznacza: zamiast usuwać obiekt Application?

Domyślam się, że chodzi np. o to:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

Application->Free();
// lub
delete Application;

(...) więc w czym tkwi różnica?

W tym, że Close zamyka konkretne okno, a Terminate zamyka całą aplikację. Jak łatwo się domyśleć, zamknięcie głównego okna aplikacji w konsekwencji spowoduje wywołanie metody Terminate (lub kodu, który robi to samo).

[Pitek]

Bardzo dziękuję wersja bez zmiennej pośredniczącej działa dobrze. Mam jednak jeszcze pytanie jak do tej aktualnej funkcji stworzyć coś co by się z nią nie gryzło i dobrze działało a chodzi mi o blokadę ponownego uruchomienia bo ta blokada o której już tutaj wspomniano wyrzuca przy próbie restartu że nie można uruchomić kopii czyli nie pozwala na restart.

[Cyfrowy Baron]

Ja rozwiązałem ten problem stosując metodę zaproponowaną przez polymorphism. Czyli w klasie restartu nic nie zmieniam, jednak przed dokonaniem restartu zmieniam nazwę okna głównego. Mutex za pomocą funkcji FindWindow szuka programu po nazwie klasy i nazwie okna. Gdy restart zmieni nazwę okna, to mutex nie może go odnaleźć i restartuje okno, natomiast bez restartu nie można uruchomić kopii programu, gdyż mutex to blokuje:

W pliku Project.cpp

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

WINAPI _tWinMain(HINSTANCE, HINSTANCE, LPTSTR, int)
{
 HANDLE hMutex = OpenMutex(MUTEX_ALL_ACCESS, 0, L"Project1.exe_mutex");
 if(!hMutex) hMutex = CreateMutex(0, 0, L"Project1.exe_mutex");
 else
  {
   HWND hWnd = FindWindow(L"TForm1", L"Form1");
   if(hWnd)
   {
    WaitForSingleObject(hMutex, INFINITE);
    ShowMessage("Program jest już uruchomiony");
    Application->Terminate();
   }
 }
        try
        {
                Application->Initialize();
                Application->MainFormOnTaskBar = true;
                Application->CreateForm(__classid(TForm1), &Form1);
                Application->Run();
        }
        catch (Exception &exception)
        {
                Application->ShowException(&exception);
        }
        catch (...)
        {
                try
                {
                        throw Exception("");
                }
                catch (Exception &exception)
                {
                        Application->ShowException(&exception);
                }
        }
        return 0;
}

Restart aplikacji

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

void __fastcall TForm1::Button1Click(TObject *Sender)
{
 String nazwa = ExtractFileName(Application->ExeName);

 Form1->Caption = "restart";

 HANDLE hEvent = CreateEvent(NULL, FALSE, FALSE, nazwa.c_str());
 if(hEvent)
 {
  ShellExecute(NULL, L"open", nazwa.c_str(), NULL, NULL, SW_SHOW);

  static signal_event_at_end tmp(hEvent);
  Close();
 }
}

[polymorphism]

Przecież tu podałem jak to zrobić link.