Übungsprogramm "Kinokarten"

[Aviator]

Zitat von Luckie:

Was ist denn das? Warum kein einfaches Close? Klar, kann ich beim Auto einfach den Zündschlüssel abziehen. Aber normalerweise bremst man, hält an, leg den ersten Gang ein, zieht die Handbremse, macht dann den Motor aus und zieht den Zündschlüssel ab.

Du hast das Lenkradschloss vergessen.

[Luckie]

Zitat von Aviator:

Zitat von Luckie:

Was ist denn das? Warum kein einfaches Close? Klar, kann ich beim Auto einfach den Zündschlüssel abziehen. Aber normalerweise bremst man, hält an, leg den ersten Gang ein, zieht die Handbremse, macht dann den Motor aus und zieht den Zündschlüssel ab.

Du hast das Lenkradschloss vergessen.

Das lasse ich vom Autodieb selber einrasten.

[EdAdvokat]

Ich will aus der Anweisung Application.terminate keine Ideologie machen, oder gar ein "Autounfall" provozieren. Bislang habe ich auch immer "close" für das Schließen des Programmes benutzt und mußte u.a. auch hier in der DP und in anderen Foren lernen, dass "close" lediglich zum Schließen eines Formulars eingesetzt werden sollte, jedoch nicht zum Schließen des Programms selbst. Dies wurde wie eine Todsünde behandelt. Nun kommt die Rolle rückwärts? Da habe ich doch brav all die Hinweise der Foren berücksichtigt und soll nun also wieder vom Glauben abfallen? Ist es denn so schlimm mit Application.terminate?
Über die anderen Fakten werde ich nachdenken.
Wie ich die Exception besser lösen könnte ist mir nicht klar. Ich weis da nicht so richtig weiter. Einfach nur frei lassen (also den Quatsch mit "on EConvertError do showMessage(Fehler)" weglassen?) und erwarten was für eine Exception-Meldung kommt oder was? Da könnte ich eine Anregung gebrauchen. Danke

[haentschman]

Moin...

Zitat:

und mußte u.a. auch hier in der DP und in anderen Foren lernen, dass "close" lediglich zum Schließen eines Formulars eingesetzt werden sollte

Das hast du nicht von hier... Auch deine Anwendung ist auch ein Formular. ...allerdings das Letzte.
Mit Terminate brichst du das Programm HART ab. (sinngemäß) Beispiel: Datenbanken werden u.u. nicht richtig geschlossen. Das kann zu Problemen führen.

..also immer CLOSE

[Luckie]

Wenn du die eine große Prozedur, wie schon vorgeschlagen, in einzelne zerlegst, kannst du gezielt mögliche Exceptions abfangen.

Und zum Close: Ein Aufruf von Close im Hauptformular bewirkt genau das, dass die Anwendung beendet wird. Und bitte zeig mir den Thread hier im Forum, wo gesagt wird, dass Apüplication.Terminate zum Beendne einer Anwendung vor zu ziehen ist. Klar auch wenn man eine Anwendung/Prozess hart beendet gibt Windows alle Systemressourcen frei, DLls werden entladen, alle offenen Handles werden geschlossen usw. Aber Close ist einfach sauberer.

@haentschman: Sockets und DBs sollte man wohl extra behandeln zu Sicherheit, sprich explizit selber schließen, da würde ich mich nicht auf Windows verlassen.

[EdAdvokat]

Ich gebe zu, mit den Hinweisen war ich gefordert und das Ergebnis (Datei anbei) ist sicher nicht so, wie erwaret. Ich komme nicht weiter mit einer effektiveren Verkleinerung der großen Prozedur "Kartenbestellung". Bei jeder versuchten Neuaufteilung kommt die Zählerei der noch freien Plätze durcheinander. Wenn diese Berechnung ordentlich durchführt wird, gibt er trotz nicht freier Kinoplätze eine Rechnungssumme aus. Ich habe mehrere Varianten ausprobiert, doch meine bescheidenen Kenntnisse...
In der jetzigen Form sollte alles so wie vorgesehen funktionieren.
Noch eine kleine Frage habe ich zum Compilerhinweis besser charlnset zu verwenden bei der Abfrage der Editfelder (Key in...)

[Luckie]

Deswegen das ganze in eine Klasse kapseln.

[Aviator]

Zitat von EdAdvokat:

Noch eine kleine Frage habe ich zum Compilerhinweis besser charlnset zu verwenden bei der Abfrage der Editfelder (Key in...)

Und wie wäre die Frage?

CharInSet solltest du bei neueren Delphi Versionen verwenden. Müsste dann so aussehen: if not CharInSet(Key, ['0'..'9']) then