FormatINI

[EWeiss]

Kleine procedure um INI Dateien zu formatieren.

Delphi-Quellcode:

function LeftStr(const Str: string; Size: Word): string;
begin
  Result := Copy(Str, 1, Size)

end;

Delphi-Quellcode:

type
  PParseFile = ^TParseFile;
  TParseFile = record
    Nr: Integer;
    Str: string;
    Ptr: PParseFile;
    Max: Integer;
  end;

Delphi-Quellcode:

var
  LineStart: PParseFile;
  FPBuffer: PParseFile;
  FToPBuffer: PParseFile;
  nReading: Integer;

Delphi-Quellcode:

procedure FormatINI(Filename: string);
var
  sBuffer: string;
  ParseFile: TextFile;
  StringList: TStringList;
  StringParse: string;

procedure AppendToLinkedList(nReading: Integer; sBuffer: string);
begin

  New(FPBuffer);

  if nReading = 0 then
  Begin
    New(FToPBuffer);
    LineStart := FToPBuffer;
    LineStart^.Nr := 0;
  end;

  FPBuffer^.Nr := nReading;
  FPBuffer^.Str := sBuffer;
  LineStart^.Max := FPBuffer^.Nr;
  FToPBuffer^.Ptr := FPBuffer;
  FToPBuffer := FPBuffer;
end;

begin
  StringList := TStringList.Create;

  Assignfile(ParseFile, Filename);
  reset(ParseFile);

  try
    try
      while not eof(ParseFile) do
      begin
        ReadLN(ParseFile, sBuffer);
        AppendToLinkedList(nReading, sBuffer);
        inc(nReading);
      end;
    except
      raise Exception.Create(SysErrorMessage(GetLastError));
    end;
  finally
    nReading := 0;
    CloseFile(ParseFile);
  end;

  FPBuffer := LineStart;
  while (FPBuffer.Nr <= LineStart.Max - 1) do
  begin
    FPBuffer := FPBuffer.Ptr;
    StringParse := LeftTrim(FPBuffer.Str);
    if Length(StringParse) <> 0 then
      StringList.Add(FPBuffer^.Str);

    if FPBuffer.Nr > 0 then
      if LeftStr(FPBuffer^.Str, 1) = '[' then
        StringList.Insert(StringList.IndexOf(StringParse), '');
  end;

  StringList.SaveToFile(Filename);
  StringList.Free;

  Dispose(FPBuffer);
  FPBuffer := nil;
  LineStart := nil;

end;

Mich ärgert immer wenn die Dateien zu unaufgeräumt sind.
Die Ini wird so formatiert das vor jedem neuen Eintrag "[..]" eine Leerzeile eingefügt wird.

Wer's braucht.. bitte schön.

gruss

[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