Mal so ein paar Tipps:
Um nur mal so ein paar Sachen zu nennen.
Wieso? Weil wir dir Tipps geben und versuchen, zu helfen? Auch wenn du 40 Jahre in der Softwareprogrammierung tätig warst und es dich vielleicht ankotzt, von einem 58 Jahre jüngeren Menschen Tipps zu empfangen, kannst du dir das wenigstens mal angucken / versuchen, die Tipps umzusetzen. Wir machen das ja nicht, weil wir böse sind
- Keine Deutschen Namen für Variablen/Methoden/Eigenschaften/Irgendetwas nehmen
- Deutliche Namen (ay, ax usw. sind eher undeutlich)
- https://www.vb-paradise.de/index.php/Thread/61310-B%C3%B6ses-aus-VB6-VB2003-und-die-richtigen-VB-NET-Alternativen/ So ziemlich alle Funktionen, die mit einem großen
C
anfangen (CInt, CSng, CDbl, ...) kannst du rausschmeißen. Außerdem sindJoin()
undSplit()
ebenfalls absolut nicht mehr zu verwenden - Wieso
Threading.Thread.Sleep(25)
? Application.DoEvents()
ist ein No go! Wieso?- Die Methode ist 258 Zeilen lang! Das ist selbst für manche Klassen ziemlich viel. Unbedingt mehr abkapseln / abstrahieren!
- Pfade mit Path.Comine verbinden.
AndAlso
stattAnd
Um nur mal so ein paar Sachen zu nennen.
drschef schrieb:
Ich werde mich mal aus dem Thema auskoppeln.
Wieso? Weil wir dir Tipps geben und versuchen, zu helfen? Auch wenn du 40 Jahre in der Softwareprogrammierung tätig warst und es dich vielleicht ankotzt, von einem 58 Jahre jüngeren Menschen Tipps zu empfangen, kannst du dir das wenigstens mal angucken / versuchen, die Tipps umzusetzen. Wir machen das ja nicht, weil wir böse sind