"IsValidValue". Aber das SetText selbst ist ja eh schon ein Validator-Event und der Aufruf einer funktion in diesem Event, was eh nur eine Sache macht, bringt mir keinen Mehrwert an Leserlichkeit.
Ich kann es natürlich ändern wie es der Reviewer möchte, aber es geht ja um das Grundsätzliche...
Gedanken dazu ?
Ich würde als Grenze klar die
Testbarkeit empfehlen. Also wenn du es so wie es ist gut testen kannst: Passt schon.
Wenn du aber dafür Verrenkungen machen musst, weil dein Form eine Instanz von TSettings benötigt, du musst das Form instanziieren damit du die Methode aufrufen kannst...
auslagern
Die Anekdote dazu: Stell dir vor du gehst zum Arzt, der sagt dir: Du brauchst einen Schrittmacher. Du fragst den Arzt: "Haben sie den denn schon getestet?" Antwort: "Nein, wir haben ein neues Design genutzt, aber das ist leider nicht testbar."
Ergo: Je einfacher die Logik zu testen ist, desto besser ist die Architektur
Und meine Meinung dazu: Logik ohne Test = Es ist offenbar egal ob es funktioniert oder nicht.