วัตถุที่ถูกล็อกจะยังคงถูกล็อกอยู่หรือไม่หากเกิดข้อยกเว้นขึ้นภายใน

ในแอพ ac # threading ถ้าฉันจะล็อคอ็อบเจกต์ให้เราพูดคิวและถ้ามีข้อยกเว้นเกิดขึ้นอ็อบเจกต์จะยังคงล็อคอยู่หรือไม่ นี่คือรหัสหลอก:

int ii;
lock(MyQueue)
{
   MyClass LclClass = (MyClass)MyQueue.Dequeue();
   try
   {
      ii = int.parse(LclClass.SomeString);
   }
   catch
   {
     MessageBox.Show("Error parsing string");
   }
}

ตามที่ฉันเข้าใจรหัสหลังจากการจับไม่ได้ดำเนินการ - แต่ฉันสงสัยว่าการล็อกจะถูกปลดปล่อยหรือไม่

อันดับแรก; คุณได้พิจารณา TryParse หรือไม่

in li;
if(int.TryParse(LclClass.SomeString, out li)) {
    // li is now assigned
} else {
    // input string is dodgy
}

การล็อกจะถูกคลายออกด้วยเหตุผล 2 ประการคือ lockประการแรกเป็นหลัก:

Monitor.Enter(lockObj);
try {
  // ...
} finally {
    Monitor.Exit(lockObj);
}

ประการที่สอง; คุณจับและไม่โยนข้อยกเว้นภายในซ้ำดังนั้นจึงlockไม่เห็นข้อยกเว้นจริง แน่นอนว่าคุณกำลังล็อคอยู่ตลอดระยะเวลาของกล่องข้อความซึ่งอาจเป็นปัญหาได้

ดังนั้นมันจะถูกปล่อยออกมาทั้งหมดยกเว้นข้อยกเว้นที่ไม่สามารถกู้คืนได้อย่างหายนะร้ายแรงที่สุด

ฉันทราบว่าไม่มีใครพูดถึงในคำตอบของพวกเขาสำหรับคำถามเก่า ๆ นี้ว่าการปลดล็อกข้อยกเว้นเป็นสิ่งที่อันตรายอย่างยิ่งที่ต้องทำ ใช่คำสั่งล็อคใน C # มีความหมาย "ในที่สุด"; เมื่อตัวควบคุมออกจากล็อคตามปกติหรือผิดปกติล็อคจะถูกคลายออก คุณทุกคนพูดถึงเรื่องนี้เหมือนเป็นเรื่องดี แต่มันเป็นสิ่งที่ไม่ดี! สิ่งที่ควรทำหากคุณมีพื้นที่ที่ถูกล็อกซึ่งทำให้เกิดข้อยกเว้นที่ไม่สามารถจัดการได้คือยุติกระบวนการที่เป็นโรคทันทีก่อนที่จะทำลายข้อมูลผู้ใช้มากขึ้นไม่ใช่ปล่อยล็อกและดำเนินการต่อ

ลองดูแบบนี้สมมติว่าคุณมีห้องน้ำที่ล็อคประตูและมีคนรออยู่ข้างนอก ระเบิดในห้องน้ำดับลงฆ่าคนในนั้น คำถามของคุณคือ "ในสถานการณ์นั้นจะมีการปลดล็อกโดยอัตโนมัติเพื่อให้คนถัดไปเข้าห้องน้ำได้หรือไม่" ใช่! ฉันจะ. นั่นไม่ใช่เรื่องดี ระเบิดลงที่นั่นและฆ่าใครบางคน! ประปาอาจจะถูกทำลายบ้านที่ไม่มีโครงสร้างเสียงและอาจจะมีระเบิดอีกในการมี สิ่งที่ควรทำคือพาทุกคนออกไปให้เร็วที่สุดและรื้อถอนบ้านทั้งหลัง

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

หากคุณล็อคภูมิภาคของรหัสเพื่อดำเนินการกลายพันธุ์ที่ไม่มีหัวข้ออื่นอ่านรัฐในเวลาเดียวกันและการกลายพันธุ์พ่นแล้วถ้าข้อมูลไม่ได้เสียหายก่อนก็แน่ใจว่าอยู่ในขณะนี้ ซึ่งเป็นสิ่งสถานการณ์ที่ล็อคควรจะป้องกัน ตอนนี้รหัสที่รออ่านสถานะนั้นจะได้รับสิทธิ์เข้าถึงสถานะเสียหายทันทีและอาจเกิดปัญหาขึ้นเอง อีกครั้งสิ่งที่ถูกต้องคือยุติกระบวนการ

ไม่ว่าคุณจะเชือดมันเป็นข้อยกเว้นภายในล็อคเป็นข่าวร้าย คำถามที่ถูกต้องที่จะถามไม่ใช่ "ล็อคของฉันจะถูกล้างในกรณีที่มีข้อยกเว้นหรือไม่" คำถามที่ถูกต้องคือ "ฉันจะแน่ใจได้อย่างไรว่าจะไม่มีข้อยกเว้นภายในตัวล็อคและถ้ามีฉันจะจัดโครงสร้างโปรแกรมของฉันอย่างไรเพื่อให้การกลายพันธุ์ถูกย้อนกลับไปสู่สถานะที่ดีก่อนหน้านี้"

ใช่ว่าจะปล่อยอย่างถูกต้อง lockทำหน้าที่เป็นtry/ finallyด้วยMonitor.Exit(myLock)ในที่สุดดังนั้นไม่ว่าคุณจะออกด้วยวิธีใดก็ตามจะถูกปล่อยออกมา ในฐานะที่เป็นด้านโน้ตcatch(... e) {throw e;}จะหลีกเลี่ยงที่ดีที่สุดในขณะที่ความเสียหายที่กองร่องรอยบนe; จะเป็นการดีกว่าที่จะไม่จับมันเลยหรืออีกทางเลือกหนึ่งคือใช้throw;แทนที่จะthrow e;โยนซ้ำ

หากคุณต้องการทราบจริงๆการล็อคใน C # 4 / .NET 4 คือ:

{
    bool haveLock = false;
    try {
       Monitor.Enter(myLock, ref haveLock);
    } finally {
       if(haveLock) Monitor.Exit(myLock);
    }
} 

"คำสั่งล็อกถูกรวบรวมเพื่อเรียก Monitor.Enter จากนั้นลอง ... บล็อกในที่สุดในบล็อกสุดท้าย Monitor.Exit จะถูกเรียก

การสร้างโค้ด JIT สำหรับทั้ง x86 และ x64 ทำให้แน่ใจได้ว่าการยกเลิกเธรดจะไม่เกิดขึ้นระหว่างมอนิเตอร์ป้อนการโทรและบล็อกการลองที่ตามมาทันที "

นำมาจาก: ไซต์นี้

ล็อคของคุณจะได้รับการปลดอย่างถูกต้อง การlockกระทำเช่นนี้:

try {
    Monitor.Enter(myLock);
    // ...
} finally {
    Monitor.Exit(myLock);
}

และfinallyรับประกันว่าบล็อกจะดำเนินการไม่ว่าคุณจะออกจากtryบล็อกอย่างไร

เพียงเพื่อเพิ่มคำตอบที่ยอดเยี่ยมของ Marc เล็กน้อย

สถานการณ์เช่นนี้เป็นสาเหตุที่ทำให้lockคีย์เวิร์ดมีอยู่ ช่วยให้นักพัฒนาตรวจสอบว่าล็อกถูกปลดล็อกในfinallyบล็อก

หากคุณถูกบังคับให้ใช้Monitor.Enter/ Exitเช่นเพื่อรองรับการหมดเวลาคุณต้องแน่ใจว่าได้โทรไปMonitor.Exitที่finallyบล็อกเพื่อให้แน่ใจว่ามีการคลายล็อกอย่างเหมาะสมในกรณีที่มีข้อยกเว้น