MVPは

Question

iが

public interface IProgressView 
{ 
    string Status { set; } 
    void SetProgress(int percentageDone); 

    void Display(); 
    void Close(); 

    event Action Closing; 
} 

class ProgressPresenter 
{ 
    private IProgressView m_view; 
    private ILongRunningTask m_task; 
    private bool m_completed; 

    public Progress(IProgressView view) 
    { 
     m_view = view; 
    } 

    public virtual void Display(ILongRunningTask task, string taskName) 
    { 
     m_task = task; 

     m_view.Status = taskName " is running"; 

     m_view.Closing += OnClosing; 
     task.ProgressChanged += UpdateProgress; 
     task.Completed += Completed; 

     task.StartAsync(); 

     m_view.Display(); 

     m_view.Closing -= OnClosing; 
     task.ProgressChanged -= UpdateProgress; 
     task.Completed -= Completed; 
    } 

    protected virtual void UpdateProgress(object sender, ProgessEventArgs e) 
    { 
     m_view.SetProgress(e.AlreadyDone * 100/e.Total); 
    } 

    protected virtual void Completed() 
    { 
     m_completed = true; 
     m_view.Status = "Completed"; 
     m_view.Close(); 
    } 

    private virtual void OnClosing() 
    { 
     if (!m_completed) m_downloader.Cancel(); 
    } 
} 

私の問題は、タスクが別のスレッドで実行されていることで、簡単なProgressPresenterとビューへの各呼び出し（Formとして実装）スロー​​を実装WinFormsの。私はそれが別のスレッドから呼び出すことができ念の

public string Status 
{ 
    set { Invoke(new Action(() => progressLabel.Text = value)); } 
} 

状に各メソッドをラップする必要がありますか？またはプレゼンターに欠陥がありますか？

アドバイスありがとうございます

Answer 1

はい、そうしてください。 私は他のどのライブラリを使っているのか分かりませんが、それを行うためにすべてのビューにアスペクトを追加することをお勧めします。

また、基本ビューでいくつかの便利なメソッドを追加する価値があります。例えば。解答のアスペクト実装チェックhere