为什么我的事件处理程序不更新我的 Windows 窗体文本框？

Question

谁能解释或说明为什么我的事件处理程序不更新我的 Windows 窗体文本框？我已将事件处理程序放在 UI 线程中以更新我的 GUI 窗口中的文本框。我的 UI 线程 #1 SetOperation 类中的 EventLaunch 方法会启动一个事件。 UI 线程 #1 SetOperation 类 OnChDetDisplay 事件处理程序完成，但窗口窗体文本框未更新为分配的值。我缺少什么将事件和处理程序绑定到更新文本框？

感谢任何人都可以分享的任何帮助，

下面是一些代码：

// Class runs in Thread #2:  Prepares message data for Windows Form GUI display and passes to UI Thread #1
    public class Aag_PrepDisplay
    {
        private Aag_PrepDisplay mAagPrep;

        public Aag_PrepDisplay AagPrep
        {
            get { return mAagPrep; }
            set { mAagPrep = value; }
        }

        // Thread #2: prepares message for Windows Form GUI display in UI Thread #1
        public void PrepareDisplay(/*stuff*/)
        {
            mAagPrep = new Aag_PrepDisplay();
            // does message prep stuff  

            SetOperation setOp1 = new SetOperation(); 
            setOp1.FireEvent(mAagPrep);  // call to UI Thread #1 method to fire event to update GUI; passes object with data
        }
    }


    // UI Thread #1 class is the Windows Form. Displays and updates all textboxes. 
    public partial class SetOperation : Form
    {
        public event Action<object> OnChDet;    // declared delegate object event that passes an object

        public SetOperation()
        {
            InitializeComponent();
            OnChDet += chDetDisplayHandler;     // hooks handler to event
        }

        // Thread #1: accepts object w/data from Thread #2; Fires an event to update GUI Textbox(s)
        private void FireEvent(Aag_PrepDisplay aagPrep)
        {
            OnChDet(aagPrep);
        }

        // UI Thread #1 event handler.
        public void chDetDisplayHandler(object name)
        {
            // **** Problem:  event is triggered and value assigned, but doesn't update the GUI window Textbox ********
            actFreqChan1.Text = "402.5";    // this is only a test to see if event handler will update Textbox
            // Next step:  updateAll(name); // pass the object from Aag_PrepDisplay class  
        }   

        //Thread #1: update GUI Textbox values
        public void updateAll(object name)
        {
            // this is where data from the Thread #2 AagPrep object will assign and update Textbox values
        }
    }

Answer 1

在问题行设置断点，告诉我们你看到了什么。

在事件基础架构中，它可能不会被调用并且问题是向上的。 如果它被调用，则问题出在Text 字段的设置器中。

在这两种情况下，缺陷都不是您认为的。

我会简化代码。可能我错过了一些东西，但我正在尝试一下。

public partial class SetOperation : Form
{
    public event Action<object> OnChDet;

    public SetOperation()
    {
        InitializeComponent();
        OnChDet += chDetDisplayHandler;
    }

    private void chDetDisplayHandler(object name)
    {
        ActFreqChan1.Text = "402.5";
    }
}

然后您可以简单地触发事件：

mySetOperationInstance.OnChDet(myNameObject);

问题是谁会触发这个事件？这取决于你自己去发现。 您必须将上述行放在某处。

据我所知，您不需要：

• ChanEventArg;
• ChDetHandler;
• Aag_DisplayEvent（这看起来完全错误）；
• EventLaunch() 方法。

你唯一应该关心的是：

• 拥有event；
• 附加处理程序；
• 在需要时使用参数调用它。

执行此操作：制作代码的备份副本并尝试简化。

如果没有帮助，抱歉，请恢复到您的备份。否则你做的太多了，在某个地方你已经对事件的确切发送方式迷失了方向。

Answer 2

事件处理程序可能会抛出一个Exception，它不会出现在 UI 中并保持潜在状态。以下代码将防止创建控件的线程以外的其他线程抛出异常：

官方参考：MSDN on InvokeRequired

类似问题：Using InvokeRequired vs control.InvokeRequired

更长的解释但真的很好：MSDN tutorial on Thread-Safety in WinForms

将线程安全保护（InvokeRequired 等）包裹在事件处理程序内的此分配周围：

actFreqChan1.Text = "402.5";

我希望这会对你有所帮助。否则你仍然可以回到这里。

Answer 3

谢谢 pid。当事件从线程 1 的 SetOperation() 中触发时，我返回并重新创建了更新文本框的代码。事件处理程序更新了文本框。然后我尝试从线程 2 的 PrepareDisplay() 调用线程 1 方法，并从线程 1 方法触发事件。事件处理程序不会更新文本框。接下来，我将安全线程调用代码添加到 Thread 1 SetOperation 类。文本框不会使用安全线程调用代码进行更新。我把它从 MSDN 教程中拿出来。当我通过它时很难遵循代码流。它在方法之间来回跳跃。看来 InvokeRequired 给出了错误。无论哪种情况，文本框都应更新为 402.5。你有没有看到我放错地方的东西或其他缺失的代码？

下面是我模拟的整个代码。再次感谢您愿意在其中一些方面对我进行辅导。

namespace TstTxtBoxUpdate
{
    static class Program
    {
        /// <summary>
        /// The main entry point for the application.
        /// </summary>
        [STAThread]
        static void Main()
        {
            Aag_PrepDisplay aag_Prep1 = new Aag_PrepDisplay();

            Thread AagPrepDisplayThread = new Thread(new ThreadStart(aag_Prep1.PrepareDisplay));
            AagPrepDisplayThread.Start();

            while(!AagPrepDisplayThread.IsAlive)
                ;
            Thread.Sleep(1000);

            Application.EnableVisualStyles();
            Application.SetCompatibleTextRenderingDefault(false);
            Application.Run(new SetOperation());
        }
    }
}

namespace TstTxtBoxUpdate
{
    // Thread 1: UI
    public partial class SetOperation : Form
    {
        private string text;
        public event Action<object> OnChDet;

        delegate void SetTextCallback(string text);
        private Thread demoThread = null;

        public SetOperation()
        {
            InitializeComponent();
            OnChDet += chDetDisplayHandler;
        }

        public void FireEvent(Aag_PrepDisplay aagPrep)
        {
            OnChDet(mName);
        }

        private void chDetDisplayHandler(object name)
        {
            this.demoThread = new Thread(new ThreadStart(this.ThreadProcSafe));
            this.demoThread.Start();
        }

        private void ThreadProcSafe()
        {
            this.SetText("402.5");
        }

        private void SetText(string text)
        {
            if(this.actFreqChan1.InvokeRequired)
            {
                SetTextCallback d = new SetTextCallback(SetText);
                this.Invoke(d, new object[] { text });
            }
            else
            {
                // TextBox NOT updated when event called from FireEvent() that was called from Thread 2 PrepareDisplay()
                // TextBox IS updated when event called from Thread 1: SetOperation() or FireEvent()
                this.actFreqChan1.Text = text;
            }
        }
    }
}

namespace TstTxtBoxUpdate
{
    // Thread 2: Data prepare
    public class Aag_PrepDisplay
    {
        #region Fields

        private Aag_PrepDisplay mAagPrep;

        #endregion Fields

        #region Properties

        public Aag_PrepDisplay AagPrepDisp;

        public Aag_PrepDisplay AagPrep
        {
            get { return mAagPrep; }
            set { mAagPrep = value; }
        }

        #endregion Properties

        #region Methods

        public void PrepareDisplay()
        {
            mAagPrep = new Aag_PrepDisplay();
            SetOperation setOp1 = new SetOperation();
            setOp1.FireEvent(mAagPrep);     // calls Thread 1 method that will fire the event
        }

        #endregion Methods
    }
}