C# データバインディングされたプロパティの更新

質問

C#の質問になります。

INotifyPropertyChangedを実装したプロパティをラベルにバインディングして、プロパティの値を表示しています。
ボタンを押下するとプロパティの値を更新するプログラムを追加しました。プロパティの値が変わるとラベルの表記も自動で更新されています。

ある時、プロパティに収める値を演算する処理に3秒掛かるようになりました。結果がラベルに反映されるまでの時間は重要ではないため、この部分をスレッド化しようと思い、下記のようなプログラムを作成しました。

結果として、下記のプログラムはInvalidOperationExceptionを発生し動作しません。
別スレッドからラベルにバインディングしているプロパティを変更したためですが、正しい？（一般的な）解決法がわかりません。

そもそも設計が間違っているという考えも大いにあり得るのですが、その場合、どの様に実装すべきかご教示いただければと思います。

public partial class Form1 : Form
{
private SampleViewModel svm = new SampleViewModel();

public Form1()
{
InitializeComponent();
}

private void Form1_Load(object sender, EventArgs e)
{
label1.DataBindings.Add(new Binding("Text", svm.Data, "MyProperty", true, DataSourceUpdateMode.OnPropertyChanged));
}

private void button1_Click(object sender, EventArgs e)
{
// データ更新
svm.Update();
}
}

internal sealed class SampleViewModel
{
public SampleModel Data
{
get { return this._Data; }
set { this._Data = value; }
}
private SampleModel _Data = new SampleModel();

public void Update()
{
// 重い処理がなかったときは下記の1行だった
//Data.MyProperty = DateTime.Now.ToLongTimeString();

// 重い処理をスレッドで処理
// label1にMyPropertyをバインディングしているからInvalidOperationExceptionが発生した
Task.Factory.StartNew(() =>
{
// 重い処理ダミー
System.Threading.Thread.Sleep(3000);

return DateTime.Now.ToLongTimeString();
}
).ContinueWith(task =>
{
Data.MyProperty = task.Result;
});
}
}

// 以下お決まりの実装
internal sealed class SampleModel : INotifyPropertyChanged
{
public event PropertyChangedEventHandler PropertyChanged;
public string MyProperty
{
get { return this._MyProperty; }
set
{
this._MyProperty = value;
if (PropertyChanged != null)
{
PropertyChanged(this, new PropertyChangedEventArgs("MyProperty"));
}
}
}
private string _MyProperty = DateTime.Now.ToLongTimeString();
}

回答No.1

amyaad · ベストアンサー

UIを更新できるのはUIスレッドであり、別スレッドから更新しようとすると、InvalidOperationExceptionが発生します。参考URLそのままなのですが、 Update()で、
var syncContext = SynchronizationContext.Current;としておき、
ContinueWith(task =>
{
Data.MyProperty = task.Result;
});

の部分を

.ContinueWith(task =>

{

Data.MyProperty = task.Result;

},TaskScheduler.FromCurrentSynchronizationContext());

とかすればよいです。

http://ufcpp.wordpress.com/2012/04/26/%E9%9D%9E%E5%90%8C%E6%9C%9F%E5%87%A6%E7%90%86%E3%81%A8%E3%83%87%E3%82%A3%E3%82%B9%E3%83%91%E3%83%83%E3%83%81%E3%83%A3%E3%83%BC/

お礼

ありがとうございます。
無事に解決できました。

自分ではＧＵＩを直接変更していないからとＧＵＩスレッドの考えが無くなっていました。
InvalidOperationExceptionエラーなのだからもっとよく考えれば良かったです。