Ze29

Als Mocking Framework würde ich nicht NMock nehmen sondern ein moderneres Framework das mit Lamdas arbeitet. Anbieten würden sich da z.B. Moq oder Rhino Mocks. NSubstitute sieht ebenfalls sehr gut aus. Die Mocking Frameworks arbeiten unter der Haube mit dynamischen Proxy Klassen. Was ganz anderes als dein Ansatz.

Also das mit der globalen Klasse knick mal lieber, siehe Klotzkopps Kommentar Was für ein Problem willst du überhaupt lösen? Denk am besten erstmal darüber nach, warum du von einem Form auf das andere willst und fange da an das Problem an der Wurzel zu packen. Wenn du einmal damit anfängst auf den Parent zuzugreifen und rumzucasten dann gehts ganz schnell weiter mit "2 forms hoch, 3 rechts und dann wieder einen runter".

Folgende Hinweise: 1) Limitiere deine Methoden eine Länge von maximal 10 Zeilen - versuche sie so um ~5 Zeilen lang zu halten. 2) Zerlege ein komplexes Problem in kleine, einfachere Probleme (brauchen dann auch kürzere Methoden) Schreib damit das ganze nochmal neu und du wirst Fehler schneller finden, als in so einem riesen Wust von if else if if else else if else...

Wichtiger ist, dass generische Listen Typensicher sind und damit der Compiler mehr Fehler abfangen kann. In eine Array Liste kannste alles reinschmeißen und es knallt dann ggf. beim Casten zur Laufzeit. Das passiert dir mit ner generischen Liste nicht.

a) wie bereits gesagt sollten statt ArrayList generische Listen verwendet werden. Ansonsten lässt sich das ganze auch als Dreizeiler ausdrücken: public List<string> Split(string input) { var ignoredChars = new[] {"!", ".", ",", "?", "\n"}; var words = input.Split(new[] {" "}, StringSplitOptions.RemoveEmptyEntries); return words.Where(x => !ignoredChars.Contains(x)).ToList(); } Die String.Split Methode kann selbstständig leere Einträge herausfischen und die Where Methode sammelt dann alles auf, was nicht einem der ignorierten Chars entspricht. Falls es unbedingt ne Array List sein _muss_ kann man das Ergebnis der Methode auch in ne ArrayList stecken - return new ArrayList(words.Where(...));