web-dev-qa-db-ja.com

変更されたクロージャーへのアクセス（2）

これは、 Access to Modified Closure からの質問の拡張です。以下が実際に本番で使用するのに十分安全かどうかを確認したいだけです。

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); });
}

私はスタートアップごとに一度だけ上記を実行します。現時点では、問題なく動作しているようです。ジョンがいくつかのケースで直観に反する結果について述べたように。だから私はここで何に注意する必要がありますか？リストが複数回実行されても問題ありませんか？

c#.netresharperclosures
101
faulty

C＃5より前は、変数insideをforeachで再宣言する必要があります-そうでない場合は共有され、すべてのハンドラーは最後の文字列を使用します。

_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）のスコープはinsideループです。仕様のリファレンスはありませんが、基本的には次のようになります。

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

再購読解除;匿名ハンドラーを積極的にサブスクライブ解除したい場合の秘訣は、ハンドラー自体をキャプチャーすることです。

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

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

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

これは現在、登録を解除しています;-p

159
Marc Gravell