UIスレッドにメソッドを呼び出すと、スレッドはWindowsメッセージキューによって検索され、その後リエントラントである必要はありませんか？UIスレッドでマーシャリングされたメソッドはスレッドセーフである必要があります
private void CalledFromWorkerThread()
{
//changed from 'InvokeRequired' Anti-Pattern
this.Invoke((Action) (() => _counter++; /* Is this ok? */));
}
明確化：_counterにアクセスするUIスレッドのみです。
_counterはUIスレッドによってのみアクセスされるものとします。
2つのスレッドがCalledFromWorkerThreadを呼び出す場合、_counterは適切にインクリメントされ、スレッドセーフです。
これは、同じスレッドからの呼び出しを委譲しますが、必ずしもその関数内で行うすべてのことがスレッドセーフであるとは限りません。
これがそのフィールドを変更する唯一のコードだとします。スレッドセーフですか？ –
それにアクセスする唯一のスレッドです（スレッドセーフである必要はありません）。そう、はい、それは安全です。 – ghimireniraj
説明に基づいて、UIスレッドのみが_counterにアクセスしているため、ロックは必要ありません。私は私の例を更新しました。私は必要なチェックを呼び出す場合に余分なことを避けるために、この方法でコーディングする方が好きです。
private void CalledFromWorkerThred()
{
this.Invoke((Action) (() => _counter++;));
}
+1質問を更新しました。乾杯、耳。 –
まだカウンターをロックしていない場合は、カウンターを使用する必要があります。 –
こちらもご覧ください：http://www.yoda.arachsys.com/csharp/threads/winforms.shtml –
「InvokeRequired」のテストは反パターンです。 'Control.Invoke'はすでにこのようなチェックを含んでいます。 –