C＃winforms：GUIとロジックの分離が正しく行われていますか？

Question

編集ロジックコードGUIの代表を送信するために、以下の提案に基づいて

、私はこのようなコードを思い付いた：

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

がこれを短縮することが可能ですか？

---

ここは私のC＃windowsフォームプログラムの一部です。

別のスレッドを使用するようにコードを変換し始めたとき、ロジックコードが実際にGUIを使用できるように、スレッドを生成して代理人にラップされたパブリックメソッドを作成すると、

より良いイディオムやアーキテクチャの改善についてご提案ください。ありがとうございました。

// 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

フォームとsearchCoreの間に循環参照はありません。 検索ロジックをコールバックを通じてフォームに戻すのはなぜですか？その方法では、検索はフォームについて知る必要はなく、テストするのが簡単です。