TIniFile umbiegen zu TIniMemFile

[Benedikt Magnus]

Zitat von Fritzew:

Warum das normale verhalten ändern?
Echt das musst Du mir erklären.
Was ist an Deiner Lösung besser?

Weil das normale Verhalten geändert wurde. So ist es nunmal sauberer.

Zitat von Fritzew:

Achte mal auf die Fragestellung.
Ich binde eine andere Unit ein und fertig.
Du musst jeden Aufruf von free durch Deine Routine ersetzen.
Wenn das besser ist dann sei es so.

Wie du an meiner Antwort sehen kannst, habe ich zwei Vorschläge gemacht, oder? Und mein Erster ist ein anderer.

Aber ich höre jetzt auf, darüber zu diskutieren. Das erscheint mir nicht sehr sinnvoll und wird langsam Off-Topic...

[Daniel]

Zitat von Fritzew:

Achte mal auf die Fragestellung.

Achte bitte ein wenig darauf, wie sich Deine Beiträge für Dritte darstellen könnten. Du wirkst ausgesprochen schroff - auch wenn dies möglicherweise nicht so gemeint ist.

[Fritzew]

Dein erster Vorschlag war ja auch in Ordnung.

Ich habe einfach ein Problem mit "unerwartetem" Verhalten von Klassen.
Und eine Procedure/ Function die eine Klasse freigibt ist für mich unerwartet. Dafür gibt es nämlich genau eine Lösung
den Destructor.

Aber nichts für ungut.

[Uwe Raabe]

Zitat von Jim Carrey:

Aber gibt es eine Möglichkeit die Deklaration von TIniFile so umzubiegen, dass TMemIniFile benutzt wird und zusätzlich bei MeineIniDatei.Free; dann .UpdateFile + Free ausgeführt wird?

Ich habe noch nicht so recht den Sinn hinter dieser Aktion verstanden, aber in der Regel verwendet man im Programm überall TCustomIniFile (der gemeinsame Vorfahre von TIniFile und TMemIniFile ). Bei der Erzeugung wird dann entweder eine TIniFile - oder TMemIniFile -Instanz oder sonst eine Ableitung von TCustomIniFile erzeugt. Das automatische UpdateFile wird (ab Berlin) durch ein AutoSave := true realisiert. Das schreibt auch nur, wenn sich wirklich was verändert hat.

[Fritzew]

Zitat von Daniel:

Zitat von Fritzew:

Achte mal auf die Fragestellung.

Achte bitte ein wenig darauf, wie sich Deine Beiträge für Dritte darstellen könnten. Du wirkst ausgesprochen schroff - auch wenn dies möglicherweise nicht so gemeint ist.

War bestimmt nicht so gemeint.
Sollte ich jemandem zu nahe getreten sein, tut es mir leid.
Mea culpa

[Lemmy]

Zitat von Benedikt Magnus:

Ganz einfach weil eine Prozedur das machen sollte, was ihre Bezeichnung aussagt. Und eine weitere Funktion an den Destruktor zu hängen finde ich unsauber.

Tipp: Schau dir mal BeforeDestruction an, da musst Du weder den Destructor überschreiben noch eine eigene Methode einführen.

[Benedikt Magnus]

Zitat von Lemmy:

Zitat von Benedikt Magnus:

Ganz einfach weil eine Prozedur das machen sollte, was ihre Bezeichnung aussagt. Und eine weitere Funktion an den Destruktor zu hängen finde ich unsauber.

Tipp: Schau dir mal BeforeDestruction an, da musst Du weder den Destructor überschreiben noch eine eigene Methode einführen.

Danke!
Das löst aber nicht das Problem (weshalb ich ursprünglich auf die neue Methode kam), dass auf diese Weise etwas mitunter Unerwartetes beim Freigeben des Objektes passiert, das ich auch nicht mehr unterbinden kann.
Die von Uwe Raabe angesprochene Lösung mit AutoSave := true , wie sie in Berlin realisiert wurde, scheint mir daher die eleganteste zu sein. Nichts Unerwartetes, keine neuen Methoden.

[Fritzew]

Um das für mich abzuschließen

Delphi-Quellcode:

unit myini;

interface

uses
   sysutils,
   IniFiles;

type
   tinifile = class(tmeminifile)
   private
      fautosave: boolean;

   public
      constructor Create(const FileName: string; const autosave: boolean = true); overload;
      constructor Create(const FileName: string; const Encoding: TEncoding; const autosave: boolean = true); overload;
      destructor Destroy; override;
   end;

implementation

{ tinifile }

constructor tinifile.Create(const FileName: string; const Encoding: TEncoding; const autosave: boolean = true);
begin
   inherited Create(FileName, Encoding);
   fautosave := autosave;
end;

constructor tinifile.Create(const FileName: string; const autosave: boolean = true);
begin
   inherited Create(FileName);
   fautosave := autosave;
end;

destructor tinifile.Destroy;
begin
   if fautosave then
      UpdateFile;
   inherited;
end;

end.

Diese Unit überall da einbinden wo das neue verhalten gewünscht ist.

[hoika]

Hallo,
dieser Thread scheint der Nachfolger von diesem hier zu sein
http://www.delphipraxis.net/190773-p...ml#post1352810

Und im aktuellen Thread geht es um den Austausch von TIniFile durch TMemIniFile,
ohne den Quellcode zu ändern.
Die neue Klasse und das Einbinden als letzte Unit löst das ja am Besten,
Ob so eine Vorgehensweise gut ist, sei dahingestellt.

Wenn die Code-Stellen abzählbar sind,
kann ich ja gleich TMemIniFile schreiben.

[Rollo62]

Nur mal zum Verständnis:
Wenn du mit "umbiegen" meinst das diese neue TIniFile Klasse dann IMMER statt der alten aufgerufen wird,
egal wo und wann ...
Das wird nicht funktionieren.

Du musst immer deine neue Unit einbinden, also wozu dann den Namen gleich machen ?
Da würde ich schon eher ein IIniFile Interface anlegen, das würde mehr Sinn machen.

Rollo