[解決済み] モディファイドクロージャーへのアクセス(2)

質問

の質問を拡張したものです。 変更されたクロージャへのアクセス . 私はちょうど次のものが実際に生産的な使用のために十分に安全であるかどうかを確認したいです。

List<string> lists = new List<string>();
//Code to retrieve lists from DB    
foreach (string list in lists)
{
    Button btn = new Button();
    btn.Click += new EventHandler(delegate { MessageBox.Show(list); });
}

起動時に1回だけ上記を実行しています。今のところ、問題なく動作しているようです。Jonが言ったように、場合によっては直感に反する結果になることがあります。ここで気をつけるべきことは何でしょうか？リストが複数回実行されても大丈夫なのでしょうか？

どのように解決するのですか？

C# 5 より前のバージョンでは、変数を再宣言する必要があります。 の内部 さもなければ、変数が共有され、すべてのハンドラが最後の文字列を使用することになります。

foreach (string list in lists)
{
    string tmp = list;
    Button btn = new Button();
    btn.Click += new EventHandler(delegate { MessageBox.Show(tmp); });
}

重要なのは、C# 5以降、これが変更されたことに注意してください。 の場合、特に foreach の場合は、もうこれを行う必要はありません：質問のコードは期待通りに動作します。

この変更なしでは動作しないことを示すために、以下を考えてみましょう。

string[] names = { "Fred", "Barney", "Betty", "Wilma" };
using (Form form = new Form())
{
    foreach (string name in names)
    {
        Button btn = new Button();
        btn.Text = name;
        btn.Click += delegate
        {
            MessageBox.Show(form, name);
        };
        btn.Dock = DockStyle.Top;
        form.Controls.Add(btn);
    }
    Application.Run(form);
}

上記を実行する をC# 5より先に実行する。 を実行すると、各ボタンは異なる名前を表示しますが、ボタンをクリックすると "Wilma"が4回表示されます。

これは、言語仕様 (ECMA 334 v4, 15.8.4) (C# 5 より前) で定義されているためです。

foreach (V v in x) embedded-statement が展開されます。

{
    E e = ((C)(x)).GetEnumerator();
    try {
        V v;
         while (e.MoveNext()) {
            v = (V)(T)e.Current;
             embedded-statement
        }
    }
    finally {
        … // Dispose e
    }
}

なお、変数 v (これはあなたの list が宣言されています。 の外側 の外側で宣言されています。したがって、キャプチャされた変数の規則によって、リストのすべての反復はキャプチャされた変数ホルダーを共有することになります。

C# 5以降では、これが変更されます。反復変数( v ) がスコープされ の内側 というループになります。仕様の参考資料がないのですが、基本的にはこうなります。

{
    E e = ((C)(x)).GetEnumerator();
    try {
        while (e.MoveNext()) {
            V v = (V)(T)e.Current;
            embedded-statement
        }
    }
    finally {
        … // Dispose e
    }
}

Re unsubscribe; anonymous handlerを積極的にunsubscribeしたい場合、ハンドラ自身をキャプチャするのがコツです。

EventHandler foo = delegate {...code...};
obj.SomeEvent += foo;
...
obj.SomeEvent -= foo;

同様に、一回限りのイベントハンドラ（Loadなど）が必要な場合。

EventHandler bar = null; // necessary for "definite assignment"
bar = delegate {
  // ... code
  obj.SomeEvent -= bar;
};
obj.SomeEvent += bar;

これは現在、自己購読を解除しています ;-p