C# winforms: Ist meine Trennung von GUI und Logik korrekt?

Question

bearbeiten

Basierend auf Anregungen unter der logischen Code GUI Delegierten zu schicken, kam ich mit Code wie folgt auf:

Action ClearFunction = new Action(()=>Invoke(new Action(delegate() { ResultsBox.Clear(); }))); 

Ist es möglich, dies zu verkürzen?

---

Hier ist ein Teil meiner C# Windows Forms Programm.

Als ich anfing, den Code in einen anderen Thread zu konvertieren, begann er sich sehr unhandlich zu fühlen, als ich Threads erzeugte und öffentliche Methoden in Delegaten erstellte, sodass der Logikcode tatsächlich die GUI verwenden konnte.

Bitte geben Sie Vorschläge für bessere Idiome oder Verbesserungen der Architektur. Vielen Dank.

// form1.cs 
    public void ClearResultsBox() 
    { 
     ResultsBox.Clear(); 
    } 

    public void PrintResults(string s) 
    { 
     ResultsBox.AppendText(s); 
    } 

    private void SearchButton_Click(object sender, EventArgs e) 
    { 
     var t = new Thread(() => SearchCore.Execute(DirectoryBox.Text, SearchBox.Text, this)); 
     t.Start(); 
    } 

    // logic.cs 
class SearchCore 
{ 
    delegate void ClearFunction(); 
    delegate void AppendFunction(string a); 

    static ClearFunction clear; 
    static AppendFunction print; 

    public static void Execute(string path, string searchterm, MainForm form) 
    { 
     clear = new ClearFunction(() => form.Invoke(new ClearFunction(form.ClearResultsBox))); 
     print = new AppendFunction(s => form.Invoke(new AppendFunction(form.PrintResults), s)); 

     clear(); 

Answer 1

Ich hätte keinen kreisförmigen Verweis zwischen dem Formular und SearchCore. Warum lässt du die Suchlogik nicht durch einen Callback zurück zur Form bringen? Auf diese Weise muss die Suche nicht über die Form informiert werden und ist einfacher zu testen.