MessageDlg wird dopellt angezeigt - Solle er aber nicht

[Muetze1]

Zitat von marabu:

Die pauschale Entfernung von Form1 führt zu Problemen in den nicht objekt-gebundenen Funktionen und Prozeduren. Entweder man macht diese zu Methoden der Form (hier empfehlenswert) oder man übergibt Parameter um globale Zugriffe zu vermeiden.

Asche über mein Haupt. Ich habe die klassenlosen Funktionen glatt übersehen und mich hauptsächlich auf den unteren Teil konzentriert, wo die ganzen Methoden stehen. Bei den klassenlosen Funktionen ist das natürlich fatal, Form1. pauschal zu entfernen. Aber ich denke mal, hier könnte man leicht das ganze zu einer Methode wandeln, wobei ich hier keine Nachteile sehen würde.

Ansonsten: Verbesserungen könnte man an dem Code noch eine Menge machen, weil die StringReplace() Funktionen bisher recht träge von statten gehen sollten. Schliesslich wird ständig aus einer Property gelesen und dieser wieder zugewiesen. Über die gesamten Aufrufe könnte man mit einer lokalen Variable arbeiten und nur zu Beginn die Property auslesen und am Ende wieder setzen.