การทำให้รูปแบบรหัส InvokeRequired เป็นแบบอัตโนมัติ

ฉันได้รับรู้ถึงความเจ็บปวดอย่างถี่ถ้วนว่าบ่อยครั้งที่เราต้องเขียนรูปแบบโค้ดต่อไปนี้ในโค้ด GUI ที่ควบคุมโดยเหตุการณ์

private void DoGUISwitch() {
    // cruisin for a bruisin' through exception city
    object1.Visible = true;
    object2.Visible = false;
}

กลายเป็น:

private void DoGUISwitch() {
    if (object1.InvokeRequired) {
        object1.Invoke(new MethodInvoker(() => { DoGUISwitch(); }));
    } else {
        object1.Visible = true;
        object2.Visible = false;
    }
}

นี่เป็นรูปแบบที่น่าอึดอัดใจใน C # ทั้งที่ต้องจำและพิมพ์ มีใครมากับทางลัดบางอย่างหรือสร้างสิ่งนี้โดยอัตโนมัติในระดับ? มันจะเจ๋งถ้ามีวิธีแนบฟังก์ชั่นกับวัตถุที่ทำการตรวจสอบนี้โดยไม่ต้องทำงานพิเศษทั้งหมดเช่นobject1.InvokeIfNecessary.visible = trueทางลัดประเภท

คำตอบก่อนหน้านี้ได้กล่าวถึงการทำไม่ได้เพียงแค่เรียก Invoke () ทุกครั้งและแม้กระทั่งไวยากรณ์ Invoke () นั้นไม่มีประสิทธิภาพและยังไม่สะดวกที่จะจัดการ

ดังนั้นมีใครคิดว่าทางลัดใด ๆ ?

วิธีการของลีสามารถทำให้ง่ายขึ้นอีก

public static void InvokeIfRequired(this Control control, MethodInvoker action)
{
    // See Update 2 for edits Mike de Klerk suggests to insert here.

    if (control.InvokeRequired) {
        control.Invoke(action);
    } else {
        action();
    }
}

และสามารถเรียกได้ว่าเป็นแบบนี้

richEditControl1.InvokeIfRequired(() =>
{
    // Do anything you want with the control here
    richEditControl1.RtfText = value;
    RtfHelpers.AddMissingStyles(richEditControl1);
});

ไม่จำเป็นต้องผ่านการควบคุมเป็นพารามิเตอร์ไปยังผู้รับมอบสิทธิ์ C # จะสร้างปิด

อัปเดต :

ตามที่ผู้โพสต์อื่น ๆControlสามารถทั่วไปเป็นISynchronizeInvoke:

public static void InvokeIfRequired(this ISynchronizeInvoke obj,
                                         MethodInvoker action)
{
    if (obj.InvokeRequired) {
        var args = new object[0];
        obj.Invoke(action, args);
    } else {
        action();
    }
}

DonBoitnott ชี้ให้เห็นว่าแตกต่างจากอินเตอร์เฟซที่ต้องใช้อาร์เรย์วัตถุสำหรับวิธีการเป็นรายการพารามิเตอร์สำหรับControlISynchronizeInvokeInvokeaction

อัพเดท 2

การแก้ไขที่แนะนำโดย Mike de Klerk (ดูความคิดเห็นในตัวอย่างรหัสที่ 1 สำหรับจุดแทรก):

// When the form, thus the control, isn't visible yet, InvokeRequired  returns false,
// resulting still in a cross-thread exception.
while (!control.Visible)
{
    System.Threading.Thread.Sleep(50);
}

ดูความคิดเห็นของ ToolmakerSteve ด้านล่างสำหรับข้อกังวลเกี่ยวกับคำแนะนำนี้

คุณสามารถเขียนวิธีการขยาย:

public static void InvokeIfRequired(this Control c, Action<Control> action)
{
    if(c.InvokeRequired)
    {
        c.Invoke(new Action(() => action(c)));
    }
    else
    {
        action(c);
    }
}

และใช้มันเช่นนี้

object1.InvokeIfRequired(c => { c.Visible = true; });

แก้ไข: Simpzon ชี้ให้เห็นในความคิดเห็นที่คุณสามารถเปลี่ยนลายเซ็นเป็น:

public static void InvokeIfRequired<T>(this T c, Action<T> action) 
    where T : Control

นี่คือแบบฟอร์มที่ฉันใช้ในรหัสทั้งหมดของฉัน

private void DoGUISwitch()
{ 
    Invoke( ( MethodInvoker ) delegate {
        object1.Visible = true;
        object2.Visible = false;
    });
} 

ฉันได้ตามนี้ในรายการบล็อกที่นี่ ฉันไม่ได้มีวิธีนี้ล้มเหลวฉันดังนั้นฉันจึงเห็นว่าไม่มีเหตุผลใดที่จะทำให้โค้ดของฉันซับซ้อนด้วยการตรวจสอบInvokeRequiredคุณสมบัติ

หวังว่านี่จะช่วยได้

สร้างไฟล์ ThreadSafeInvoke.snippet จากนั้นคุณสามารถเลือกคำสั่งการอัพเดทคลิกขวาและเลือก 'Surround With ... ' หรือ Ctrl-K + S:

<?xml version="1.0" encoding="utf-8" ?>
<CodeSnippet Format="1.0.0" xmlns="http://schemas.microsoft.com/VisualStudio/2005/CodeSnippet">
  <Header>
    <Title>ThreadsafeInvoke</Title>
    <Shortcut></Shortcut>
    <Description>Wraps code in an anonymous method passed to Invoke for Thread safety.</Description>
    <SnippetTypes>
      <SnippetType>SurroundsWith</SnippetType>
    </SnippetTypes>
  </Header>
  <Snippet>
    <Code Language="CSharp">
      <![CDATA[
      Invoke( (MethodInvoker) delegate
      {
          $selected$
      });      
      ]]>
    </Code>
  </Snippet>
</CodeSnippet>

นี่คือคำตอบของ Lee's, Oliver's และ Stephan ที่ได้รับการปรับปรุง / รวมกัน

public delegate void InvokeIfRequiredDelegate<T>(T obj)
    where T : ISynchronizeInvoke;

public static void InvokeIfRequired<T>(this T obj, InvokeIfRequiredDelegate<T> action)
    where T : ISynchronizeInvoke
{
    if (obj.InvokeRequired)
    {
        obj.Invoke(action, new object[] { obj });
    }
    else
    {
        action(obj);
    }
} 

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

progressBar1.InvokeIfRequired(o => 
{
    o.Style = ProgressBarStyle.Marquee;
    o.MarqueeAnimationSpeed = 40;
});

ฉันควรใช้วิธีการหนึ่งอย่างแทนการมอบหมายแทนการสร้างอินสแตนซ์ใหม่ทุกครั้ง ในกรณีของฉันฉันใช้เพื่อแสดงความคืบหน้าและข้อความ (ข้อมูล / ข้อผิดพลาด) จาก Backroundworker คัดลอกและส่งข้อมูลขนาดใหญ่จากอินสแตนซ์ sql ทุกครั้งหลังจากที่ความคืบหน้าประมาณ 70000 และข้อความโทรแบบฟอร์มของฉันหยุดทำงานและแสดงข้อความใหม่ สิ่งนี้ไม่ได้เกิดขึ้นเมื่อฉันเริ่มใช้ตัวแทนผู้ใช้อินสแตนซ์สากลเดียว

delegate void ShowMessageCallback(string message);

private void Form1_Load(object sender, EventArgs e)
{
    ShowMessageCallback showMessageDelegate = new ShowMessageCallback(ShowMessage);
}

private void ShowMessage(string message)
{
    if (this.InvokeRequired)
        this.Invoke(showMessageDelegate, message);
    else
        labelMessage.Text = message;           
}

void Message_OnMessage(object sender, Utilities.Message.MessageEventArgs e)
{
    ShowMessage(e.Message);
}

การใช้งาน:

control.InvokeIfRequired(c => c.Visible = false);

return control.InvokeIfRequired(c => {
    c.Visible = value

    return c.Visible;
});

รหัส:

using System;
using System.ComponentModel;

namespace Extensions
{
    public static class SynchronizeInvokeExtensions
    {
        public static void InvokeIfRequired<T>(this T obj, Action<T> action)
            where T : ISynchronizeInvoke
        {
            if (obj.InvokeRequired)
            {
                obj.Invoke(action, new object[] { obj });
            }
            else
            {
                action(obj);
            }
        }

        public static TOut InvokeIfRequired<TIn, TOut>(this TIn obj, Func<TIn, TOut> func) 
            where TIn : ISynchronizeInvoke
        {
            return obj.InvokeRequired
                ? (TOut)obj.Invoke(func, new object[] { obj })
                : func(obj);
        }
    }
}

ฉันชอบที่จะทำมันแตกต่างกันเล็กน้อยฉันชอบที่จะเรียกว่า "ตัวเอง" ถ้าจำเป็นกับการกระทำ

    private void AddRowToListView(ScannerRow row, bool suspend)
    {
        if (IsFormClosing)
            return;

        if (this.InvokeRequired)
        {
            var A = new Action(() => AddRowToListView(row, suspend));
            this.Invoke(A);
            return;
        }
         //as of here the Code is thread-safe

นี่เป็นรูปแบบที่มีประโยชน์ IsFormClosing เป็นเขตข้อมูลที่ฉันตั้งค่าเป็นจริงเมื่อฉันปิดแบบฟอร์มเนื่องจากอาจมีเธรดพื้นหลังบางส่วนที่ยังคงทำงานอยู่ ...

คุณไม่ควรเขียนโค้ดที่มีลักษณะดังนี้:

private void DoGUISwitch() {
    if (object1.InvokeRequired) {
        object1.Invoke(new MethodInvoker(() => { DoGUISwitch(); }));
    } else {
        object1.Visible = true;
        object2.Visible = false;
    }
}

หากคุณมีรหัสที่มีลักษณะเช่นนี้แสดงว่าแอปพลิเคชันของคุณไม่ปลอดภัยสำหรับเธรด หมายความว่าคุณมีรหัสที่เรียก DoGUISwitch () จากเธรดอื่นแล้ว สายเกินไปที่จะตรวจสอบเพื่อดูว่ามันอยู่ในเธรดอื่นหรือไม่ InvokeRequire จะต้องถูกเรียกก่อนที่คุณจะโทรไปที่ DoGUISwitch คุณไม่ควรเข้าถึงวิธีหรือคุณสมบัติใด ๆ จากเธรดอื่น

การอ้างอิง: คุณสมบัติ Control.InvokeRequired ซึ่งคุณสามารถอ่านข้อมูลต่อไปนี้:

นอกเหนือจากคุณสมบัติ InvokeRequired มีสี่วิธีในการควบคุมที่เป็นเธรดที่ปลอดภัยในการโทร: เรียกใช้ BeginInvoke, EndInvoke และ CreateGraphics หากสร้างหมายเลขอ้างอิงสำหรับตัวควบคุมแล้ว

ในสถาปัตยกรรม CPU เดียวไม่มีปัญหา แต่ในสถาปัตยกรรม multi-CPU คุณสามารถทำให้ส่วนหนึ่งของเธรด UI ถูกกำหนดให้กับตัวประมวลผลที่เรียกรหัสกำลังเรียกใช้ ... และถ้าตัวประมวลผลนั้นแตกต่างจากที่เธรด UI กำลังทำงานอยู่เมื่อเธรดการโทรจบลง Windows จะคิดว่าเธรด UI ได้สิ้นสุดลงและจะฆ่ากระบวนการแอปพลิเคชันเช่นแอปพลิเคชันของคุณจะออกโดยไม่มีข้อผิดพลาด