Abfragemenü

[s.h.a.r.k]

Evtl. sollte man noch dazu sagen, dass die Methode MessageDlg() nicht threadsafe ist und man diese in der Unit Dialogs findet.

[Dunkelbunt27]

FormCanClose ist ja nicht als Ereignis vordefiniert (bei mir zumindest nicht)
reicht es also aus die Procedure anzulegen
procedure TForm1.FormCanClose(Sender: TObject; var CanClose: Boolean);; und in type zu definieren?

EDIT:
Es gibt ein OnCloseQuery, procedure TForm1.FormCloseQuery(Sender: TObject;
var CanClose: Boolean);
Das ist identisch.

[Dunkelbunt27]

Wie kann ich es machen, dass wenn ich erst auf JA klicke (also ich möchte speichern)
und dann im Speicherdialog abbreche, sich das Programm nicht schließt?

ich hab es jetzt so:

Delphi-Quellcode:

  case MR of
    mrYes   : begin
               CanClose := False;
               if not FileExists(Filename) then
               begin
                 SpeicherDialog := TSaveDialog.Create(self);
                 Speichern_mit_Dialog;
                  //if SpeicherDialog.Execute = False  //Falsch, da so der Dialog erneut aufgerufen wird.
                  //   CanClose := False;
               end
               else
                begin
                  Speichern(Filename);
                  CanClose := True;
                  Exit;
                end;
              end;
    mrNo    : begin
              //leer
              end;
    mrCancel: begin
               CanClose := False;
               exit;
              end;
  end;

<Edit> Doppelpost -.- </Edit>

[SirTwist]

Sorry, natürlich heißt das Event CloseQuery....

am einfachsten definierst Du Deine Speichern()-Routine als Funktion, die einen Boolean zurückliefert, und zwar True genau dann, wenn alles gut gegangen ist.

Dann hättest Du nur noch mrYes: CanClose := Speichern();

Wobei natürlich die File-Auswahlbox in der Speichern()-Routine aufgerufen werden muss.

[Dunkelbunt27]

Okay,
Danke.

[s.h.a.r.k]

Delphi-Quellcode:

procedure TForm1.OnCloseQuery(Sender: TObject; var CanClose: Boolean);
begin
  CanClose := True;
  case MR of
    mrYes : begin
      SD := TSaveDialog.Create();
      try
        // Set dialog properties
        // ...
  
        if (SD.Execute) then
        begin
          // Save document
        end
        else begin
          // Do NOT close application because user cancelled
          // SaveDialog.
          CanClose := False;
        end;
      finally
        SD.Free();
      end;
    end;
    
    // Nothing to do here
    mrNo : ;
  
    // Application should NOT be closed
    mrCancel : CanClose := False;
  end;
end;

Sinnvoller wäre es den Teil unter mrYes in eine eigene Methode auszulagern, allein schon wegen der Übersichtlichkeit, aber eben auch aufgrund der möglichen Wiederverwendbarkeit -- vorausgesetzt du schreibst diese Save()-Methode ausreichend allgemein.

[Dunkelbunt27]

Zitat:

Sinnvoller wäre es den Teil unter mrYes in eine eigene Methode auszulagern

Ja, das hab ich auch so gemacht, sonst ist es unübersichtlich.

Frage am Rande, wie kann man in einer eigenen procedure oder function einen SaveDialog erstellen?
Er meckert immer über den Parameter self,
SpeicherDialog := TSaveDialog.Create(self); .

[Luckie]

Warum willst du ihn selber erstellen? Zieh die Komponente auf das Formular und rufe ihn auf.

[Luckie]

Zitat von Dunkelbunt27:

Er meckert immer über den Parameter self,

Tja, clever wäre es, wenn du uns jetzt noch die Fehlermeldung mitteilen würdest.