Завис при обновлении GUI

Question

У меня есть окно с заголовком, ярлыком и кнопкой ОК.Всякий раз, когда он должен появиться, он должен повторно использовать то же самое окно, а не открывать новое окно.

Я получаю диалоговое событие от внешнего потока.Он может получать сообщения за пределами известных процедур.Я хочу звонить Show() каждый раз, когда приходит сообщение.

public partial class AlertBox : Form
{
    private static AlertBox instance;
    public static AlertBox Instance => instance ?? (instance = new AlertBox());

    private AlertBox()
    {
        InitializeComponent();
    }

    private void AlertBox_Load(object sender, EventArgs e)
    {
        MessageLabel.Text = "";
        Text = "";
    }

    public void Show(string text, string title)
    {
        Show();
        BringToFront();
        Text = title;
        MessageLabel.Text = text;
    }

    private void OkButton_Click(object sender, EventArgs e)
    {
        MessageLabel.Text = "";
        Text = "";
        Hide();
    }
}

Однако при звонке Show(text, title) как показано:

internal class Driver
{
    private readonly AlertBox _alertBox = AlertBox.Instance;
    public void Initialize()
    {
        // Receive dialog event.
        _connection.OnDialogReceived += (text, title) =>
        {
            _alertBox.Show(text, title);
        };
    }
}

Появляется окно предупреждения, задается заголовок окна и останавливается.Обратите внимание, что кнопка стала невидимой.

Я пытался использовать Invoke.Он зависает с точно таким же результатом.

    public void ShowWithInvoke(string text, string title)
    {
        if (!Created)
        {
            CreateControl();
        }

        if (!IsHandleCreated)
        {
            CreateHandle();
        }

        Invoke((MethodInvoker)delegate
        {
            Show();
            BringToFront();
            Text = title;
        });

        if (!MessageLabel.Created || !MessageLabel.IsHandleCreated)
        {
            MessageLabel.CreateControl();
        }

        MessageLabel.Invoke((MethodInvoker)delegate
        {
            MessageLabel.Text = text;
        });
    }

Answer 1

Распространенным способом решения этой проблемы является отделение кода, отправляющего предупреждение, от кода, который отображает предупреждение.

Для начала вам следует создать класс, единственная цель которого - передать его.при обновлении любого пользовательского интерфейса.

public class MessageUpdater
{
    public event EventHandler<string> MessageSent;
    public void SendMessage(string message)
    {
        this.MessageSent?.Invoke(this, message);
    }
}

Это очень просто.Он просто принимает сообщение и, если к событию присоединены обработчики, он вызывает событие.

Теперь для вашего AlertBox вы просто принимаете экземпляр MessageUpdater и обновляете Label всякий раз, когда MessageSent событие инициируется.

public partial class AlertBox : Form
{
    public AlertBox()
    {
        InitializeComponent();
    }

    private MessageUpdater _messageUpdater = null;

    public MessageUpdater MessageUpdater
    {
        set
        {
            if (_messageUpdater != null)
            {
                _messageUpdater.MessageSent -= UpdateMessage;
            }
            if (value != null)
            {
                _messageUpdater = value;
                _messageUpdater.MessageSent += UpdateMessage;
            }
        }
    }

    private void UpdateMessage(object sender, string message)
    {
        if (this.InvokeRequired)
        {
            this.Invoke((Action)(() => this.UpdateMessage(sender, message)));
        }
        else
        {
            this.MessageLabel.Text = message;
        }
    }
}

Две хитрые части здесь обрабатывают присоединение нового MessageUpdater (и удаление существующего), а затем маршалинг вызовов в поток пользовательского интерфейса, если это необходимо.

Код, который я создал, чтобы проверить это, был довольно прост.

        var mu = new MessageUpdater();

        var counter = 0;
        var timer = new System.Threading.Timer((System.Threading.TimerCallback)(x =>
        {
            mu.SendMessage((counter++).ToString());
        }), null, TimeSpan.FromSeconds(1.0), TimeSpan.FromSeconds(1.0));

        var ab = new AlertBox();
        ab.MessageUpdater = mu;
        ab.ShowDialog();

Хитрая часть здесь - это System.Threading.Timer, который я использовал для отправки сообщений на MessageUpdater в потоке без пользовательского интерфейса , чтобы .ShowDialog не зависал.