Forum: Algorithmen, Datenstrukturen und Klassendesign
by DieDolly,
10. Dez 2019
Jetzt mach noch was als Lösung in https://www.delphipraxis.net/1453114-post17.html vorgeschlagen wurde und alles ist sauber.
Forum: Algorithmen, Datenstrukturen und Klassendesign
by DieDolly,
10. Dez 2019
Zeig mal deinen abgeänderten Code. Also alles meine ich was wichtig ist.
Forum: Algorithmen, Datenstrukturen und Klassendesign
by DieDolly,
10. Dez 2019
Die Fehlermeldung sagt nichts über die TObjectList aus.
Forum: Algorithmen, Datenstrukturen und Klassendesign
by DieDolly,
10. Dez 2019
button := ButtonList.Items as TTMSFNCToolBarButton;
Braucht man diesen Cast, wenn ButtonList generisch ist?
Kann ignoriert werden. Im Prozedurkopf steht TObjectList.
Forum: Algorithmen, Datenstrukturen und Klassendesign
by DieDolly,
9. Dez 2019
Was du da in die Liste packst ist ja gut. Aber dann gibt man die Liste doch nicht frei. Man erstellt sie wenn man sie braucht und gibt sie frei, wenn man sie nicht mehr braucht.
Aber wenn man da was reinpackt und später damit weiterarbeiten will, gebe ich die doch nicht sofort frei.
Forum: Algorithmen, Datenstrukturen und Klassendesign
by DieDolly,
9. Dez 2019
Versuch mal System.Generics.Collections.
Ich würde, wie erwähnt, das ganze System komplett neu machen. Es ist absoluter Quatsch erst eine Liste zu befüllen, nur um sie danach sofort wieder freizugeben.
Forum: Algorithmen, Datenstrukturen und Klassendesign
by DieDolly,
9. Dez 2019
Ich würde das ganze Konzept noch einmal überarbeiten.
Man schreibt doch nicht Buttons in eine Liste, nur um sie danach wieder freizugeben. Man arbeitet mit der Liste weiter.
Forum: Algorithmen, Datenstrukturen und Klassendesign
by DieDolly,
9. Dez 2019
Ich würde die Liste als erstes generisch deklarieren. Dann brauchst du das alles mit as nicht mehr.
BList: TObjectList<TTMSFNCToolBarButton>;
Das sieht dann so aus
for i ...
ButtonList.Items.Text :=