การเข้าถึงการปิดดัดแปลง (2)


101

นี้เป็นส่วนขยายของคำถามจากการเข้าถึงปิดดัดแปลง ฉันแค่ต้องการตรวจสอบว่าสิ่งต่อไปนี้ปลอดภัยเพียงพอสำหรับการใช้งานจริงหรือไม่

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

ฉันดำเนินการผ่านด้านบนเพียงครั้งเดียวต่อการเริ่มต้นระบบ ตอนนี้ดูเหมือนว่าจะทำงานได้ดี ดังที่จอนได้กล่าวไว้เกี่ยวกับผลการต่อต้านในบางกรณี ฉันต้องระวังอะไรที่นี่? จะเป็นไรไหมถ้ารายการถูกเรียกใช้มากกว่าหนึ่งครั้ง


18
ขอแสดงความยินดีตอนนี้คุณเป็นส่วนหนึ่งของเอกสาร Resharper แล้ว confluence.jetbrains.net/display/ReSharper/…
Kongress

1
อันนี้ยุ่งยาก แต่คำอธิบายข้างต้นทำให้ชัดเจนสำหรับฉัน: สิ่งนี้อาจดูเหมือนจะถูกต้อง แต่ในความเป็นจริงแล้วจะใช้เฉพาะค่าสุดท้ายของตัวแปร str เมื่อใดก็ตามที่คลิกปุ่มใด ๆ เหตุผลก็คือ foreach unrolls เข้าสู่ while loop แต่ตัวแปรการวนซ้ำถูกกำหนดไว้นอกลูปนี้ ซึ่งหมายความว่าเมื่อคุณแสดงกล่องข้อความค่าของ str อาจถูกทำซ้ำเป็นค่าสุดท้ายในคอลเล็กชันสตริงแล้ว
DanielV

คำตอบ:


159

ก่อนหน้า C # 5 คุณต้องประกาศตัวแปรภายใน 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" สี่ครั้ง

เนื่องจากข้อกำหนดภาษา (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
    }
}

ยกเลิกการสมัครอีกครั้ง; หากคุณต้องการยกเลิกการสมัครตัวจัดการที่ไม่ระบุตัวตนอย่างจริงจังเคล็ดลับคือจับตัวจัดการเอง:

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


หากเป็นเช่นนั้นตัวแปรชั่วคราวจะอยู่ในหน่วยความจำจนกว่าแอปจะปิดเพื่อให้บริการแก่ผู้รับมอบสิทธิ์และไม่แนะนำให้ทำเช่นนั้นสำหรับลูปขนาดใหญ่มากหากตัวแปรใช้หน่วยความจำมาก ฉันถูกไหม?
ผิดพลาด

1
มันจะอยู่ในความทรงจำตลอดระยะเวลาที่มีสิ่งต่างๆ (ปุ่ม) อยู่กับเหตุการณ์ มีวิธียกเลิกการสมัครสมาชิกแบบครั้งเดียวเท่านั้นซึ่งฉันจะเพิ่มในโพสต์
Marc Gravell

2
แต่เพื่อให้มีคุณสมบัติตรงประเด็นของคุณ: ใช่ตัวแปรที่จับได้สามารถเพิ่มขอบเขตของตัวแปรได้ คุณต้องระวังอย่าจับสิ่งที่คุณไม่คาดคิด ...
Marc Gravell

1
โปรดอัปเดตคำตอบของคุณเกี่ยวกับการเปลี่ยนแปลงข้อมูลจำเพาะ C # 5.0 ได้หรือไม่ เพียงเพื่อให้เป็นเอกสารวิกิที่ยอดเยี่ยมเกี่ยวกับ foreach loops ใน C # มีคำตอบที่ดีอยู่แล้วเกี่ยวกับการเปลี่ยนแปลงในคอมไพเลอร์ C # 5.0 ที่ปฏิบัติต่อ foreach ลูปbit.ly/WzBV3Lแต่ไม่ใช่แหล่งข้อมูลแบบวิกิ
Ilya Ivanov

1
@Kos ใช่forไม่เปลี่ยนแปลงใน 5.0
Marc Gravell
โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.