FormatINI

[Medium]

Gibt es einen Unterschied im Ergebnis zu diesem etwas knapperen hier?

Delphi-Quellcode:

procedure FormatINI(aFilename: String);
var
  f: TStringlist;
  i: Integer;
begin
  f := TStringlist.Create;
  f.LoadFromFile(aFilename);

  // Remove blank lines
  i := 0;
  while i < f.Count do
  begin
    if Length(Trim(f[i]))=0 then
      f.Delete(i)
    else
      inc(i);
  end;
  // Add blank lines before section headers
  i := f.Count-1;
  while i>0 do
  begin
    if (Length(Trim(f[i]))>0) and (Trim(f[i])[0]='[') then
      f.Insert(i, '');
    dec(i);
  end;

  f.SaveToFile(aFilename);
  f.Free;
end;

[himitsu]

Warum eigentlich umständlich AssignFile+ReadLn und nicht LoadFromFile?

Delphi-Quellcode:

StringList.LoadFromFile(Filename);
StringList.Text := ReplaceStr(StringList.Text, sLineBreak + '[', sLineBreak + sLineBreak + '[');
StringList.SaveToFile(Filename);

Delphi-Quellcode:

// vielleicht auch sicherheitshalber so?
StringList.LoadFromFile(Filename);
StringList.Text := ReplaceStr(ReplaceStr(StringList.Text, sLineBreak + sLineBreak, sLineBreak), sLineBreak + '[', sLineBreak + sLineBreak + '[');
StringList.SaveToFile(Filename);

[EWeiss]

Zitat von himitsu:

Warum eigentlich umständlich AssignFile+ReadLn und nicht LoadFromFile?

Delphi-Quellcode:

StringList.LoadFromFile(Filename);
StringList.Text := ReplaceStr(StringList.Text, sLineBreak + '[', sLineBreak + sLineBreak + '[');
StringList.SaveToFile(Filename);

Delphi-Quellcode:

// vielleicht auch sicherheitshalber so?
StringList.LoadFromFile(Filename);
StringList.Text := ReplaceStr(ReplaceStr(StringList.Text, sLineBreak + sLineBreak, sLineBreak), sLineBreak + '[', sLineBreak + sLineBreak + '[');
StringList.SaveToFile(Filename);

War ein Vorschlag für Verbesserungen bin ich immer offen.
Zumal sie in dem Fall auch kürzer sind.
Auf der anderen Seite warum erst speichern .. dann laden und anschließend wieder speichern.
Macht irgendwie keinen Sinn den die Daten sind ja vorhanden.

Ich wollte halt meine Ini Formatieren damit sie besser zum lesen ist.
Und das Schnipsel wollte ich veröffentlichen falls es jemand braucht.

Letztendlich führen viele Wege zum ziel.

Zitat:

Gibt es einen Unterschied im Ergebnis zu diesem etwas knapperen hier?

Delphi-Quellcode:

procedure FormatINI(aFilename: String);
var
   f: TStringlist;
   i: Integer;
begin
   f := TStringlist.Create;
   f.LoadFromFile(aFilename);

   // Remove blank lines
   i := 0;
   while i < f.Count do
   begin
     if Length(Trim(f[i]))=0 then
       f.Delete(i)
     else
       inc(i);
   end;
   // Add blank lines before section headers
   i := f.Count-1;
   while i>0 do
   begin
     if (Length(Trim(f[i]))>0) and (Trim(f[i])[0]='[') then
       f.Insert(i, '');
     dec(i);
   end;

   f.SaveToFile(aFilename);
   f.Free;
end;

JO warum soll ich erst leere Zeilen löschen ist unnötig.
Wird bei mir in einem rutsch erledigt.
bzw.. werden ignoriert wenn es leere Zeilen gibt.
Und auch hier unnötige zugriffe auf die Datei warum nochmalig öffnen wenn sie schon geöffnet ist?

gruss

[himitsu]

Zitat von EWeiss:

Letztendlich führen viele Wege zum ziel.

Auch für Schleifenfetischisten gibt's Vieles.

Delphi-Quellcode:

StringList.LoadFromFile(Filename);
for i := StringList.Count - 1 downto 1 do
  if StartsStr('[', StringList[i]) and (StringList[i - 1] <> '') then  // ich hoffe die Parameter bei StartsStr sind richtigrum
    StringList.Insert(i, '');
StringList.SaveToFile(Filename);

[EWeiss]

Zitat von himitsu:

Zitat von EWeiss:

Letztendlich führen viele Wege zum ziel.

Auch für Schleifenfetischisten gibt's Vieles.

Delphi-Quellcode:

StringList.LoadFromFile(Filename);
for i := StringList.Count - 1 downto 1 do
  if StartsStr('[', StringList[i]) and (StringList[i - 1] <> '') then  // ich hoffe die Parameter bei StartsStr sind richtigrum
    StringList.Insert(i, '');
StringList.SaveToFile(Filename);

Du übersiehst ganz einfach das die Datei schon vorhanden (geöffnet ist).
Nenne mir einen plausiblen Grund warum ich sie dann schließen, erneut laden und anschließend wieder speichern soll.
Ich soll also zweimal eine Datei öffnen und zweimal schließen nur um ein paar Zeilen Code zu sparen?
Welche Logik steckt dahinter?


gruss

[himitsu]

Neeeeeee, FormatINI öffnet die ja erst, also warum soll man sich selber die Mühe machen das zeilenweise manuell auszulesen?

[EWeiss]

Zitat von himitsu:

Neeeeeee, FormatINI öffnet die ja erst, also warum soll man sich selber die Mühe machen das zeilenweise manuell auszulesen?

Neee bei mir macht das INI := TIniFile.Create...
Dann werden die Daten geschrieben und anschließend direkt aufgeräumt.
Ohne nach dem Schreiben die INI zu schließen anschließend wieder mit LoadFromFile zu öffnen und wieder zu schreiben
und sie anschließend abermals zu schließen.

Das war eigentlich der Hintergrund des ganzen.
Natürlich geht es auch auf deinem weg aber mit unnötig vielen zugriffen auf die Datei selbst.
Ich habe mir also schon etwas dabei gedacht warum ich es so mache.

gruss