C# と VB.NET の質問掲示板

Visual Studio 2010 C# .NET 4.0 Client Profile な Form App を作っています。

通信処理の WorkerThread の中から画面更新をしたいので
void Form1_ReceivedEventHandler(object sender, ReceiveEventArgs arg) {
    Invoke((Action)(()=> { label1.Text = arg.packet.ToString(); }));
}
のように Invoke 経由で UI スレッドを呼ぶコードを作ってみました。

Form1_Load で Form1_ReceivedEventHandler が呼ばれるよう delegate += していて
Form1_FormClosing で Form1_ReceivedEventHandler を呼ばないよう delegate -= しています。
が、それだけでは Form1 を閉じるタイミングで System.ObjectDisposedException が出ます。
複数 Form な App なので Form1 を閉じても通信処理は依然有効なままでなければなりません。

理由は既にわかっていて、ユーザが閉じる操作をしたときたまたま Invoke 中だと
- Invoke 中に FormClosing -> FormClosed の順に処理が進み
- FormClosed の時点で UI 部品は Dispose されているので
- Invoke したい lambda が実行される時点で label1.Text はもう処分済み
なわけですが、どう対処するとよいと思いますか？

try { Invoke...; }
catch (System.ObjectDisposedException e) { 何もしない }
というのは既に試していて問題なさそうなのですが、いかにもださいので代案募集したいです。

ん、こっちの方が良いかな
EventHandler<ReceiveEventArgs> act = (s, e) => { };
EventHandler<ReceiveEventArgs> act1 = (s, e) => this.Invoke(new EventHandler<ReceiveEventArgs>(this.OnReceived), s, e);
act += act1;
hoge.Received += new EventHandler<ReceiveEventArgs>((s, e) => act(s, e));
f.FormClosing += (s, e) => act -= act1;

追記
ああ全然駄目ですねこれ。

■No75173 (774RR さん) に返信
> try { Invoke...; }
> catch (System.ObjectDisposedException e) { 何もしない }
> というのは既に試していて問題なさそうなのですが、いかにもださいので代案募集したいです。

本来あるべきところではなく、隅っこをつくようでアレですが、念のため。

Invoke 呼び出す時点で Control.IsDisposed == true の状況（厳密には !IsHandleCreated）があり得た場合、飛んでくる例外は InvalidOperation になります。
（FormClosing でイベントを外していれば、これの発生確率は非常に低くなるとは思いますが）
http://referencesource.microsoft.com/#System.Windows.Forms/winforms/Managed/System/WinForms/Control.cs,7620

※昔あった Invoke メソッドの呼び出しでタイミング依存でハングするという問題は、Reference Source を見る限り、すでに修正されているように見えました。参考までに。
http://bbs.wankuma.com/index.cgi?mode=al2&namber=6843&page=20&KLOG=18
http://referencesource.microsoft.com/#System.Windows.Forms/winforms/Managed/System/WinForms/Control.cs,4329

-----
それだけでは難ですので一案として。
いっそのこと、static メソッドなどで「ださい」部分を隠蔽してしまうとか。


public partial class Form1 : Form
{
    public void Received(object sender, EventArgs e)
    {
        this.SafeInvoke(() => { /* 実行したい処理 */ });
        
        // 拡張メソッドが避ける場合：
        // ControlExtension.SafeInvoke(this, () => { /* 実行したい処理 */ });
    }
}

public static class ControlExtension
{
    public static void SafeInvoke(this Control control, Action action)
    {
        try
        {
            control.Invoke(action);
        }
        catch (ObjectDisposedException)
        { }
        catch (InvalidOperationException)
        { }
    }
}

■No75191 (774RR さん) に返信

役に立つかわかりませんが、こんな感じのはどうでしょう？

    Private _dowork As Boolean
    Private _action As Action = Nothing
    Private _tsk As Threading.Tasks.Task

    Private Sub Form1_Load(sender As Object, e As System.EventArgs) Handles Me.Load
        _dowork = True
        _tsk = New Threading.Tasks.Task(AddressOf TaskMain)
        _tsk.Start()
    End Sub

    Private Sub TaskMain()
        Do While _dowork
            Invoke(Sub()
                       Label1.Text = Date.Now.ToString("HH:mm:ss")
                   End Sub)
        Loop
        If _action IsNot Nothing Then _action()
    End Sub

    Private Sub Form1_FormClosing(sender As Object, e As System.Windows.Forms.FormClosingEventArgs) Handles Me.FormClosing
        If _dowork Then
            _dowork = False
            _action = Sub() Invoke(Sub() Me.Close())
            e.Cancel = True
        End If
    End Sub

■No75193 (れい さん) に返信
> 修正はいったという話は来ていないですし.Net4でもダメだったように思います。
> デッドロックの頻度は減っているように感じますが調査していません。

過去のスレッドで指摘されていた、「スレッド ID の取得・比較のアプローチがダメ」という部分は .NET 4 で変わっていたため、直っているとみていました。
（IsDisposed で判定するようになっている）
ほかの部分でハングする要因が残っているのかな？


> いつ死ぬかわからないWindowとの通信でInvokeはダメパターンで使ってはいけない。
> と、私のメモ帳には書いてあります。

これは私も同意見です。

代案をあえて出すなら、SynchronizationContext.Send/Post はマシかもしれません。
確か、特定の Form に依存せず、ウィンドウを持っているはずなので。

-----
一案？

public partial class Form1 : Form
{
    private readonly SafeInvoker _safeInvoker;

    public Form1()
    {
        InitializeComponent();
        _safeInvoker = new SafeInvoker(this);
    }

    private void Received(object sender, EventArgs e)
    {
        _safeInvoker.Invoke(() => { /* 実行したい処理  */ });
    }
}

public class SafeInvoker
{
    private readonly Control _referenceControl;
    private readonly SynchronizationContext _synchronizationContext;

    public SafeInvoker(Control referenceControl)
    {
        _referenceControl = referenceControl;
        _synchronizationContext = SynchronizationContext.Current;
    }

    public void Invoke(Action action)
    {
        _synchronizationContext.Send(o =>
        {
            if (_referenceControl.IsDisposed) return;
            action();
        }, null);
    }
}

■No75207 (Azulean さん) に返信
あ、いえ言葉足らずですみません。
私は単にInvoke()された先のUIスレッドでのIsDisposedチェックを意図してました。

public static void SafeInvoke(this Control control, Action action)
{
    Action wrapper = ()=>{
        if(control.IsDisposed)
            return;

        action();
    };

    control.Invoke(wrapper);
}

が、やはり汎用的にするにはおっしゃるとおりInvokeの外側で例外チェックが必要ですね。
適当な事を書いてしまった気がします。
ごめんなさい。＞ALL