TStringList.Capacity Zuweisung?

[Morphie]

Zitat von DeddyH:

Die Alternative wäre gewesen, die Methode abstract zu deklarieren, aber dann muss jede abgeleitete Klasse sie implementieren. So kann jede Ableitung selbst entscheiden, ob sie das tut oder eben nicht.

Ich bin ja immer dafür, in einer Klasse (auch in einer Basisklasse) nur das zu deklarieren, was auch Sinn macht.
Wenn in der Basisklasse eine Methode keinen Sinn macht, sondern nur in einer speziellen geerbten Klasse, dann würde ich es in der Basisklasse einfach gar nicht deklarieren.

Das wäre so, als wenn ich in einer Klasse TKraftfahrzeug die Methode "Schwimmen" anbiete, obwohl das nur in der Klasse TAmphibienfahrzeug Sinn macht. Alle Kraftfahrzeuge haben jetzt die Methode Schwimmen, doch bei den meisten funktioniert diese eben nicht... Irgendwer kommt dann auf die Idee, mit einem TMotorrad in einen See zu fahren, weil es ja die Methode Schwimmen besitzt. (übertriebenes Beispiel )

Man könnte sogar so weit gehen, und noch eine zweite Klasse dazwischen hängen, falls es SetCapacity in mehreren Nachfahren geben soll...
TStrings > TGrowStrings > TStringList, TOtherStrings,...

Ob es da irgendeine Eigenheit bei ObjectPascal gibt, damit das keinen Sinn machen könnte, weiß ich nicht, aber in strengen OOP-Sprachen würde ich das so machen...

[Bummi]

@Morphie

Ich sehe das genau umgekehrt, ein paar zusätzliche virtuelle Methoden und vor allem mehr Methoden virtuell statt statisch implementiert würde einem oft viel "gefrickel" ersparen.
Das von Dir gewählte (etwas extreme) Beispiel würde ich auch nicht implementiert haben wollen.

[DeddyH]

Dir ist aber schon aufgefallen, dass es sich um den Setter der Property Capacity handelt? Da kannst Du wenn überhaupt höchstens die ganze Property in der Basisklasse weglassen, zumindest fällt mir auf die Schnelle keine saubere Alternative ein. Irgendwer bei (wahrscheinlich noch) Borland wird sich dabei schon etwas gedacht haben.

[Morphie]

Zitat von DeddyH:

Dir ist aber schon aufgefallen, dass es sich um den Setter der Property Capacity handelt?

Jo, ist mir bewusst... wenn ich von SetCapacity schreibe, meine ich natürlich die gesamte Property, die in der Basisklasse aber sonst nirgends genutzt wird, daher für mich überflüssig...

[Bernhard Geyer]

Ich denke hier ist es nicht sinnvoll ein capacity-property schon in der Basisklasse einzuführen. Diese Property macht dort keinen Sinn da sowas zu sehr an die Speicherorganisation von TStringList (Dumme Pointerlist statt performanterer Tree oder Hashmap) angelehnt ist.

Willst du mit Variablen vom Typ TStrings arbeiten aber wenn es nötig ist die Capacity (vor-)belegen wäre ein setzen mit den RTTI-Funktionen möglich. Haben uns selbst SetIntProperty-Funktionen gebastelt um nur bei Klassen mit entsprechenden Properties diese zu setzen und bei fehlen einfach nix zu machen.

[PeterPanino]

Der Grund meiner Frage war folgender: Wenn man in einer StringList die Capacity z.B. auf den Wert 10 einstellt, sollten beim StringList.LoadFromFile maximal 10 Zeilen gelesen werden (auch wenn die Datei etwa 100 Zeilen hat). Wäre eine solche Vorgangsweise möglich? Oder müsste man dazu eher die Methode LoadFromFile überschreiben?

[Bernhard Geyer]

Zitat von PeterPanino:

Der Grund meiner Frage war folgender: Wenn man in einer StringList die Capacity z.B. auf den Wert 10 einstellt, sollten beim StringList.LoadFromFile maximal 10 Zeilen gelesen werden (auch wenn die Datei etwa 100 Zeilen hat). Wäre eine solche Vorgangsweise möglich? Oder müsste man dazu eher die Methode LoadFromFile überschreiben?

Ich würde hier eine angepasste LoadFromFile-Methode schreiben.
Das Capacity-Property ist ein Property das nur die interne Speicherverwaltung steuert. Normalerweise wächst die Capacity entsprechend der schon vorhandenen größe immer um ein bestimmten Prozentsatz. Das führt bei großen Listen dazu das häuft die größe Zeitintensiv angepasst werden muss.

[Popov]

Zitat von PeterPanino:

Der Grund meiner Frage war folgender: Wenn man in einer StringList die Capacity z.B. auf den Wert 10 einstellt, sollten beim StringList.LoadFromFile maximal 10 Zeilen gelesen werden (auch wenn die Datei etwa 100 Zeilen hat). Wäre eine solche Vorgangsweise möglich? Oder müsste man dazu eher die Methode LoadFromFile überschreiben?

Ich denke Capacity ist für etwas anderes Gedacht. Bevor man mit Add einen String anfügt, wird vorher der Speicher der Liste z. B. um 1 erhöht, so dass nun ein String angefügt werden kann. Allerdings ist das nicht optimal für jeden String den Speicher separat zu erhöhen, also wird mit Capacity etwas Speicher auf Vorrat reserviert (Capacity ist also immer Count plus Etwas). StringList macht das eigentlich automatisch, ich glaube 4, 8, 16 , usw. Wenn also vier Strings in StringList drin sind, wird für acht weitere Strings der Speicher reserviert. Sind die voll, für die nächsten 16 Strings usw. Wie gesagt, dass passiert automatisch, so dass man Capacity kaum selbst nutzt. Interessant wird das erst wenn man vorher weiß, dass man z. B. 1000 Strings einfügt. Statt also, dass TStringList alle paar Strings eine Pause machen muss um die Kapazität zu erhöhen, kann man mit Capacity gleich selbst für 1000 Strings den Speicher reservieren. Dafür ist es gedacht. Es wird eigentlich automatisch genutzt, man kann es unter bestimmten Bedingungen aber auch selbst nutzen.