TFileStream speichert nur Unsinn

[dGeek]

Ich habe folgende Funktion, welche eigentlich einen simplen String mit einer Uhrzeit an eine Textdatei anhängen soll.

Delphi-Quellcode:

var
 aFileStream: TFileStream;
 aLogStr: String;
 aFileName: String;
begin
 aLogStr := DateTimeToStr(Now) + aSystemLogStr + sLineBreak;
 aFileName := vLogFile;

 if FileExists(aFileName) then
  aFileStream := TFileStream.Create(aFileName, fmOpenReadWrite)
 else
  aFileStream := TFileStream.Create(aFileName, fmCreate);

 try
  aFileStream.Seek(0, soEnd);
  aFileStream.WriteBuffer(Pointer(aLogStr)^, Length(aLogStr));
 finally
  aFileStream.Free;
 end;
end;

Das problem ist nun, dass der übergebene String erstens nicht komplett abgespeichert wird ud zweitens in der Datei überall NUL steht.

Bei folgender Variante verschwinden die NUL's, aber der übergebene String wird gar nicht gespeichert:

Delphi-Quellcode:

var
 aFileStream: TFileStream;
 aLogStr: String;
 aFileName: String;
begin
 // aStr z.B. == 'Programm gestartet".

 aLogStr := DateTimeToStr(Now) + '   |   ' + aStr + sLineBreak;
 aFileName := vLogFile;

 if FileExists(aFileName) then
  aFileStream := TFileStream.Create(aFileName, fmOpenReadWrite)
 else
  aFileStream := TFileStream.Create(aFileName, fmCreate);

 try
  aFileStream.Seek(0, soEnd);
  aFileStream.WriteBuffer(Pointer(aLogStr)^, Length(aLogStr));
 finally
  aFileStream.Free;
 end;
end;

Was stimmt da nicht?

[Benedikt Magnus]

Warum nimmst du nicht einfach eine Stringlist? Die scheint mir dafür geeigneter bzw. einfacher.

Delphi-Quellcode:

var
  aStringList: TStringList;
  aLogStr: String;
  aFileName: String;
begin
  aLogStr := DateTimeToStr(Now) + aSystemLogStr;
  aFileName := vLogFile;

  aStringList := TStringList.Create;
  try
    if FileExists(aFileName) then aStringList.LoadFromFile(aFileName);
    aStringList.Add(aLogStr);
    aStringList.SaveToFile(aFileName);
  finally
    aStringList.Free;
  end;
end;

[dGeek]

Daran hatte ich auch schon gedacht. Nur ist die nicht viel langsamer, oder ist das mittlerweile in Zeiten von SSDs usw ein Irrglaube?

[Der schöne Günther]

Dieses komische Pointer/Buffer-Gedöns ist mir unheimlich. Was gewinnt man damit?

Was spricht gegen TFile.AppendAllText(..) ?

Delphi-Quellcode:

uses System.IoUtils, System.Classes, System.SysUtils;
const
   fileName = 'x:\myFile.abc';
var
   aLogStr: String;
begin
   aLogStr := DateTimeToStr(Now) + 'derp' + sLineBreak;
   TFile.AppendAllText(fileName, aLogStr);
end.

Wenn man es unbedingt mit einem TStream machen möchte: Dann nimm doch einfach WriteData und wandel deinen Text in ein Byte-Array (TBytes) um:

Delphi-Quellcode:

   asBytes := TEncoding.ASCII.GetBytes(aLogStr);
   aFileStream.WriteData(asBytes, Length(asBytes));

[Uwe Raabe]

Dir ist aber schon bewusst, daß string ein Unicode-String ist, in dem jedes Zeichen zwei Byte groß ist?

Im Gegensatz dazu erwartet WriteBuffer die tatsächliche Byte-Größe des zu schreibenden Buffers.

[Benedikt Magnus]

Zitat von dGeek:

Daran hatte ich auch schon gedacht. Nur ist die nicht viel langsamer, oder ist das mittlerweile in Zeiten von SSDs usw ein Irrglaube?

Selbst wenn (müsste ich jetzt auch testen), ist dein Log so zeitkritisch?

[dGeek]

Nein natürlich nicht

[p80286]

Siehe Uwe sowie
mystring[1],length(mystring)*charsize Je nachdem wie groß die bestehende Datei ist, könnte die Stringlist etwas langsam sein.

Gruß
K-H

[dGeek]

Ja das stimmt. Hätte ich mal die Lönge der String nachgezählt, welche abgespeichert wurden, wäre mir das bestimmt aufgefallen. Aber in dem Moment kommt man nicht drauf

[Sherlock]

Persönliche Meinung (verursacht durch Softwarefehler in allen Betriebssystemen und mit allen Programmiersprachen):
Pointer sind immer der letzte Ausweg.
Wenn man mit Pointern hantiert, sollte man vorher bereits mindestens einen halben Tag andere Lösungen versucht haben.

Es gibt nur noch ganz seltene Anwendungsfälle, die die Verwendung von Pointern notwendig machen. Dazu gehören WinAPI calls und Assembler. Wenn man also in einer VCL-Anwendung plötzlich Pointer verwenden möchte, sollte man kurz innehalten und nochmal darüber nachdenken. Pointer sind schlicht nicht zeitgemäß und zu fehleranfällig.

Sherlock