| Luckie |
22. Feb 2003 16:03 |
Zitat:
Zitat von MathiasSimmack
- Du hast einen Dialog als Hauptfenster. In dem Dialog steckt aber nur die Listview, alles andere wird im Programm erzeugt. Ich hätte diesen Schritt übersprungen und Fenster und Listview auch gleich im Programm erstellt; also ohne Dialogressource.
|
- Nicht mehr. Mitlerweile ist noch was dazu gekommen.
Zitat:
- Der Quellcode ist für meinen Geschmack zu unübersichtlich. Du hast zwar räumliche Trennungen (durch mehrzeilige Kommentare), aber manchmal stecken Prozeduren in "Sparten", in denen sie, IMHO, eigentlich nichts zu suchen haben.
|
Habe ich gerade die Tage geändert.
Zitat:
- Was mir ebenfalls nicht sonderlich gefällt (persönliche Meinung!), ist das Deklarieren von Variablen an zentraler Stelle. Das mache ich nur bei Texten (Konstanten), die man z.B. übersetzen können soll. Variablen deklariere ich erst dann, wenn sie gebraucht werden; die ganzen Fenster-Handles z.B. erst vor der "WndProc", usw.
Benötige ich so ein Handle dann doch vor seiner Deklaration, dann übergebe ich es der Prozedur (die es braucht) als Parameter. Erst wenn ich das Handle in verschiedenen Prozeduren benötige, ziehe ich es weiter nach vorn.
|
Ist das wirklich so schlimm?
Zitat:
- Schau dir mal bitte den Teil mit dem Dropdown-Menü für den "Drucken"-Button an. Solltest du mal einen Button davor einfügen, oder solltest du evtl. die Toolbar-Anpassung ausprobieren, bei der der Anwender entscheiden kann, welche Buttons er wo sehen will, dann erscheint das Menü unter Garantie unter dem falschen Button.
|
Da kommt nichts mehr dazu. :wink:
Zitat:
- Es gibt einige Optimierungsmöglichkeiten im Code. Als Beispiel sei das Umschalten der Listview-Ansicht (Icon, Report, Liste) genannt. Das ist immer das selbe. Eine Prozedur, aufgerufen mit dem gewünschten Stilattribut (wie im Tutorial), würde den Code verkürzen und die Exe auch wieder ein Stückchen kleiner machen.
|
Jupp. Aber die DB ist schon alt, sehr alt. Sie wurde ständig erweitert und verbessert (was nicht nicht unbedingt für den Code gilt :? )
Zitat:
- Die Konstante "DATFILE" konnte ich übrigens auskommentieren, ohne dass das Auswirkungen gehabt hätte.
|
Ups. Die hatte ich deklariert und habe sie dann in der Aufregung ganz vergessen zu benutzen. :oops:
Zitat:
- Na gut, die Namen von bestimmten Parametern würde ich nicht als Fehler ansehen. Ich persönlich (!) bevorzuge nur eben das "richtige" Casten und vermeide daher Variablennamen, die mit irgendwelchen Typen identisch sind (HWND, WPARAM, LPARAM, HBITMAP, ...) - obwohl letztlich auch bloß LONGINTS usw. hinter solchen Typen stecken.
|
Ansichtssache.
Aber das ganze noch mal neu schreiben würde viel Arbeit bedeuten. Das einzigste, was ich eventuell machen werde ist es, den Quellcode komplett zu überarbeiten. Ein Anfang ist schon gemacht, ich habe alles mal sortiert. Das lag aber daran, dass der Code-Explorer von den GExperts die Prozeduren nur in alphabetischer Rehenfolge anzeigt, nicht wie sie im Code stehen. :( |
