Werte aus einer Ini Datei Lesen funktioniert nicht so ganz

[Dalai]

Zitat von p80286:

Das begeistert zwar nicht, aber ist wohl nur für die Testausgabe.

Naja, kann man machen, aber es gibt auch so schicke Komponenten, in die man Stringlisten speichern lassen kann. Für eine Testausgabe ist es sicher benutzbar. Dennoch kann man auf den Punkt der hartkodierten Pfade nicht oft genug hinweisen. Selbst heutzutage sehe ich noch Software, die hartkodierte Pfade nutzt, die Adminrechte voraussetzt oder ähnliche Geschichten. Das kotzt mich aus Administratorensicht einfach nur an, denn die Admins müssen dann die Scheiße wegräumen oder drumrumlaufen, die ihnen die Programmierer (oder solche, die behaupten, welche zu sein) hinterlassen haben.

Sorry für OT.

MfG Dalai

[Popov]

Zitat von Dalai:

Das ist doch Käse. Erzeuge die Listen ini, Sektionen & Co einmalig vor der Schleife und leere sie, sofern nötig, mit TStringList.Clear innerhalb der for-Schleife.

Muss er nicht. Jeder Aufruf von ReadSections und ReadSection löscht die vorherige, also die alte Liste.

[Back2Code]

Zitat von Uwe Raabe:

Übrigens: der Sinn von Abschnitte erschließt sich nicht ganz. ReadSectionValues liest immer Key-Value Paare ein.

Überhaupt bleibt deine eigentliche Absicht bei dem Code verborgen. Einzig der Kommentar am Ende bringt etwas Licht, stimmt aber nicht mit dem Code überein.

Also im Grunde genommen lese ich ein Verzeichnis ein, hole mir dort alle *Ini Dateien und beginne die einzulesen. Da will ich jeweils pro Ini Datei nochmal in verschiedene Stringlisten die Sektion, Abschnitt sowie Wert unterteiln, da ich diese dann später wiederum in eine Datenbank übertragen möchte.

[Dalai]

Zitat von Popov:

Zitat von Dalai:

Das ist doch Käse. Erzeuge die Listen ini, Sektionen & Co einmalig vor der Schleife und leere sie, sofern nötig, mit TStringList.Clear innerhalb der for-Schleife.

Muss er nicht. Jeder Aufruf von ReadSections und ReadSection löscht die vorherige, also die alte Liste.

Jeder Aufruf von ReadSection(s) leert die Liste, ja. Damit spart man sich das Clear (deswegen schrieb ich ja "sofern nötig"). Dennoch ist das wiederholte Erzeugen der Listen innerhalb der Schleife unnötig und verursacht Speicherlecks, zumal es überhaupt kein zugehöriges sl.Free gibt.

MfG Dalai

[Popov]

Habe ich auch nichts anderes behauptet. Im Code wird doppelt gemoppelt, Speicherlecks erstellt und immer wieder aufs neue unnötig TStringLists erstellt.

EDIT:

Übrigens, finde nur ich das etwas sonderbar oder ist es neuerdings, also in neuen Delphiversionen möglich Umlaute in Bezeichnern zu verwenden?

Schlüssel := TStringlist.Create;

[DeddyH]

In aktuellen Delphis kein Thema, auch wenn ich so etwas nach wie vor vermeide.

[p80286]

[OT]

Zitat von Dalai:

Zitat von p80286:

Das begeistert zwar nicht, aber ist wohl nur für die Testausgabe.

Naja, kann man machen, aber es gibt auch so schicke Komponenten, in die man Stringlisten speichern lassen kann. Für eine Testausgabe ist es sicher benutzbar. Dennoch kann man auf den Punkt der hartkodierten Pfade nicht oft genug hinweisen. Selbst heutzutage sehe ich noch Software, die hartkodierte Pfade nutzt, die Adminrechte voraussetzt oder ähnliche Geschichten. Das kotzt mich aus Administratorensicht einfach nur an, denn die Admins müssen dann die Scheiße wegräumen oder drumrumlaufen, die ihnen die Programmierer (oder solche, die behaupten, welche zu sein) hinterlassen haben.

Sorry für OT.

MfG Dalai

Gruß
K-H

[Sir Rufo]

Verrate doch mal was du eigentlich mit der ganzen Aktion bezweckst.

Kann es sein, dass du die Ini-Dateien zusammenführen möchtest?