Datei mit FileStream auslesen

[shmia]

Musst du die Datei bloss einlesen oder muss die Datei später auch verändert geschrieben werden?
Falls die Datei nur gelesen werden muss, könntest du Dir viel Arbeit sparen, wenn du eine Adapterklasse einsetzt.

Hier mal als Vergleich zwischen deiner alten Technik und meinem neuen Vorschlag:

Delphi-Quellcode:

Read(ds.FileID,sizeof(word)); // ALT
ds.FileID := h.ReadWord;      // NEU

// ALT - 3 Zeilen
Read(ds.string_length_comment[i],sizeof(word));
Setlength(ds.string_comment[i], ds.string_length_comment[i]);
Read(ds.string_comment[i][1],ds.string_length_comment[i]);

// NEU - nur eine Zeile
ds.string_length_comment[i] := h.ReadShortString;

Beim Einlesen eines Words ist der Unterschied noch recht gering,
aber beim Einlesen von den Strings zeigt sich doch, dass die neue Technik viel angenehmer ist.

Ausserdem kannst du Dir damit sämtliche Längenangaben in dem Record sparen
und dich voll auf die Nutzdaten konzentrieren.

Hier wäre mal die Deklaration der Adapterklasse:

Delphi-Quellcode:

  TStreamAdapter = class(TObject)
  protected
    FStream : TStream;
  public
    constructor Create(AStream:TStream;AOwnsStream: Boolean = False);
    destructor Destroy;override;
    function ReadBoolean: Boolean;
    function ReadChar: Char;
    function ReadCurrency: Currency;
    function ReadDateTime: TDateTime;
    function ReadDouble: Double;
    function ReadExtended: Extended;
    function ReadInt64: Int64;
    function ReadInteger: Integer;
    function ReadCString: string;
    function ReadShortString: string;
    function ReadSingle: Single;
    function ReadSizedString: string;
    function ReadWord:Word;
    procedure WriteBoolean(Value: Boolean);
    procedure WriteChar(Value: Char);
    procedure WriteCurrency(const Value: Currency);
    procedure WriteDateTime(const Value: TDateTime);
    procedure WriteDouble(const Value: Double);
    procedure WriteExtended(const Value: Extended);
    procedure WriteInt64(Value: Int64); overload;
    procedure WriteInteger(Value: Integer); overload;
    procedure WriteStringDelimitedByNull(const Value: string);
    procedure WriteShortString(const Value: ShortString);
    procedure WriteSingle(const Value: Single);
    procedure WriteSizedString(const Value: string);
    procedure WriteWord(const Value:Word);
  end;

Bei Interesse kann ich dir noch mehr zu der Adapterklasse schreiben.

[delphinewbie]

Hi shmia,
vielen Dank für deinen Vorschlag. Ja, die Datei muss nur ausgelesen werden. Veränderungen daran sind sogar unerwünscht !!!
Muss leider zugeben, dass ich mit Adapterklassen nichts anfangen kann, da ich ja auch erst anfange, mich mit delphi zu beschäftigen. Eins ist mir jedoch an deinem Beispiel aufgefallen :

Delphi-Quellcode:

// NEU - nur eine Zeile
ds.string_length_comment[i] := h.ReadShortString;

Ich lese doch eigentlich zwei Werte aus: 1. Länge der folgenden Zeichenkette - 2. Zeichenkette selbst

Bei dir in der einen Zeile kann ich aber nur erkennen, dass die Länge der Zeichenkette ausgelesen wird. Mir ist jetzt nicht klar, wie deine eine Zeile meine drei Zeilen ersetzt.

Auch wenn ich mir dadurch 'ne Menge Arbeit ersparen würde, stellt sich mir aber die Frage, ob damit der Fehler behoben wäre, der nach jedem Programmende auftritt.
Ich gehe bisher davon aus, dass das Auslesen der Datei richtig funktioniert, weil auch die korrekten Kurven usw. entstehen. Nur muss halt danach noch etwas unsauber programmiert sein (fehlende Speicherfreigaben oder ungültige Speicherbereiche...)
Ich habe die Hoffnung noch nicht aufgegeben, das Problem zu lösen und setze auch weiterhin ganz stark auf euer immenses Wissen
Tschüß und

[himitsu]

Er meinte es wohl eher so
ds.string_comment[i] := h.ReadShortString; .
bzw. so

Delphi-Quellcode:

ds.string_comment[i] := h.ReadShortString;
ds.string_length_comment[i] := Length(ds.string_comment);

Das ReadShortString ließt wohl das LängenByte aus und darauf folgend entsprechend viele Bytes/AnsiZeichen in einen String, welcher dann zurückgegeben wird.


PS: Nur um die Verwirrungen komplett zu machen.
Man ließt hier eine feste Anzahl an Byte in einen String,
nur daß ab Delphi 2009 der String ein UnicodeString ist und somit jedes Zeichen darin 2 Byte groß wäre.
Und dann behaupte man wieder ich hätte nie davor gewarnt, daß man keine generischen Typen (wie den String) mit festen ByteGrößen gleichsetzen soll.

[delphinewbie]

Hi,
das würde heißen, wenn ich die Stringlänge eigentlich nicht weiter benötige, genügt diese Zeile

ds.string_comment[i] := h.ReadShortString; ?

Funzt das nur, wenn die Stringlänge in einem Byte steht ? Was ist, wenn die Länge aber in 2 Bytes steht, wie bei mir ?

L_of_Filecomment : word;
Filecomment : string;

Zitat:

PS: Nur um die Verwirrungen komplett zu machen.

Das hat auf jeden Fall geklappt !!!

Ich glaube, ich laß lieber die Finger davon... Es muss auch mit FileStream gehen, wenn auch mit etwas mehr Aufwand.

[himitsu]

Zitat von delphinewbie:

Funzt das nur, wenn die Stringlänge in einem Byte steht ? Was ist, wenn die Länge aber in 2 Bytes steht, wie bei mir ?

Wenn diese Funktion so erstellt wurde, daß sie mit 2 Byte als Länte arbeitet, dann kann man es auch nehmen.

Und genau deswegen hatte ich doch schonmal gefragt.
Diese 2 Byte sind auch in der Datei vorhanden und nicht nur in der Record-Deklaration?

Nja, selbst wenn du die länge selber ausließt, muß sie nicht unbegingt mit in dem Record drinstehn.

Zitat von delphinewbie:

Ich glaube, ich laß lieber die Finger davon... Es muss auch mit FileStream gehen, wenn auch mit etwas mehr Aufwand.

Auch bei dem FileStream mußt du aufpassen.
Deklariere einfach deine Strings als AnsiString und gut ist.

Du kannst gerne mal so ab etwa hier ( http://www.delphipraxis.net/154168-n...ml#post1046616 ) dir die unicode- und integerbezogenen Postings durchlesen.

[delphinewbie]

Jawoll,
diese zwei Bytes sind in der Datei vorhanden. Ich muss immer wieder darauf hinweisen : Das Auslesen der Datei funktioniert. Wäre die Dateilänge bspw. nur in einem Byte abgelegt statt wie von mir angenommen in zweien, müßte es doch bei der Darstellung der Daten (z.B. in einer Tabelle) ein riesen Durcheinander geben. Zusätzlich habe ich noch den Vergleich mit einer Textdatei, die durch das Programm, mit dem die Messwerte gewonnen werden aus der Binärdatei konvertiert wird. Und ich muss sagen, dass ich hier eine exakte Übereinstimmung feststellen kann.
Nun werde ich trotzdem mal auf AnsiString umstellen, vllt. bewirkt es ja was....

[p80286]

Entschuldigt,
aber wenn ich soetwas als Beschreibung von Daten in einer Datei lese, fühle ich mich doch sehr unwohl:
formula : array[1..255] of string; was verbirgt sich hinter "String"?
Handelt es sich um den "klassischen" Pascal-String Längenbyte,Nutzdaten[1]..Nutzdaten[längenbyte] oder ist es ein LängenWort(16Bit)/(32Bit), sind für jeden String 255 Bytes reserviert oder sind die Längenangaben/Strings mit oder ohne Trennzeichen hintereinander "geklatscht"??????
Unicode und 16Bit-Chars sparen wir uns einmal.

Und dann gibt es noch die Frage ob die Zahlen Leastsignifikant (Intel) gespeichert wurden, denn wenn dies nicht der Fall ist, dann kann man kein Word/Integer oder was auch immer sondern nur Bytes lesen, die dann erst einmal "richtig" sortiert werden müssen.

(Falls man das Format einer Datei nicht genau kennt, dann geht kein Weg an Filestream vorbei)

Gruß
K-H