TString führt "pop" nicht aus

[jaenicke]

Ich denke eher, dass pop durchaus ausgeführt wird. Funktioniert denn nach dem Klicken von Button2 noch Button3, wenn du nur einen Waggon hinzugefügt hattest?

Das reine Entfernen aus deinem Stack führt aber nicht dazu, dass du auch etwas siehst. Denn was du in deinem Fenster siehst ist nur die Listbox, und die änderst du ja nicht beim Entfernen.
Du musst also beim Ändern des Stack auch deine Listbox ändern. Beim Hinzufügen fügst du den Eintrag auch in die Listbox hinzu. Was musst du also beim Entfernen machen? Den ersten Eintrag auch aus der Listbox löschen würde ich vorschlagen. (Wobei du vorher prüfen solltest, ob der Stack leer ist.)

Dann noch eine Bemerkung am Rande:
In der vorgegebenen Unit mStack heißt der Stack TStack. Das T ist dabei ein Kennzeichen dafür, dass es sich um eine Klasse handelt. Dann kann man besser erkennen, dass es die Klasse ist (der Bauplan für einen Waggon) und kein konkret erzeugtes Objekt (das einen existierenden Waggon repräsentiert). Deshalb solltest du auch deine Klasse Waggon besser TWaggon nennen.

[HiWieGehts]

Hallo,

ich hätte gesagt, es gibt nur einen Waggon. In diesen wird der neue Wert eingetragen und der alte überschrieben. Beim Lesen wird dann immer der zuletzt eingetragene Wert angezeigt. Pop ist OK.
Und noch so ein Gedanke: bei dir ist Zug = Waggon - Zug: Waggon. Ich kenne die Aufgabenstellung nicht, aber logisch wäre aus meiner Sicht, wenn ein Zug aus mehreren Waggons bestehen könnte. Es kann aber auch sein, dass in einem Waggon mehrere Ladungen transportiert werden. Dann müsste aber der Waqgon anders definiert sein.
Benutze mal den Debugger - dann wird sich das Problem rasch lösen.

[jaenicke]

Ja, stimmt, es wird auch nur ein Waggon beim Programmstart erzeugt. Statt in den einen Wert zu setzen soll sicher auch im Button1 ein neuer Waggon erzeugt werden.

[DeddyH]

BTW: TObject.Destroy

Zitat:

Rufen Sie Destroy nicht direkt auf. Verwenden Sie stattdessen Free. Die Methode Free überprüft, ob die Objekt-Referenz nicht bereits nil ist und ruft Destroy nur bei Bedarf auf.

[DerVicko]

Es lag anscheinend wirklich an dem felhenden T vor Waggon
Jetzt funktioniert alles. (und daran haben mein Lehrer und ich eine ganze Unterrichtseinheit verballert...)
@ jaenicke
Das ich die Änderung in meiner Listbox nicht sehen kann ist mir bewusst, dafür habe ich ja Button3 genutzt.
Ich wollte erstmal den absoluten Grundsatz haben, das ganze Drumherum mit Schleifen, Ausgabe etc. kommt später.
@ HiWieGehts
Der Name der Klasse ist relativ belanglos, die Aufgabenstellung sieht nur vor, dass sich Zahlen von StackA nach StackC bewegen, da fand ich Zug und Waggon ganz passend.

Damit ist dann meine Frage beantwortet (ich weiß ja nicht ob ein Mod hier Threads schließen muss oder nicht)

[HiWieGehts]

Eines ist sicher, an dem T vor Waggon lag es nicht! Da hat sich sonst was verändert. Ob Waggon oder TWaggon ist grundsätzlich egal. Dabei handelt es sich nur um eine Definition, die dem Programmierer die Lesbarkeit des Codes vereinfachen soll. Für Delphi spielt das keine Rolle.