Не похоже, чтобы передать EventArgs по ссылке

Question

У меня есть класс, который вызывает событие. Я хочу, чтобы подписчик мог изменять значения, передаваемые в EventArg.

В классе, который поднимает события:

class Factory 
    { 
    public event EventHandler<MessageReceivedEventArgs> MessageReceived; 

    private IServerLib _myObject; 

    public void Connect() 
    { 
     _myObject = new ServerLib(); 
     _myObject.AddMessageReceivedHandler((short terminal, ref string message, ref short functionNo) => 
     { 
      MessageReceivedEventArgs args = new MessageReceivedEventArgs { Terminal = terminal, Message = message, FunctionNo = functionNo }; 
      MessageReceivedEvent(ref args); 
     }); 
    } 

    private void MessageReceivedEvent(ref MessageReceivedEventArgs args) 
    { 
     EventHandler<MessageReceivedEventArgs> handler = MessageReceived; 
     if (handler != null) 
     { 
      handler(this, args); 
     } 
    } 

    public class MessageReceivedEventArgs : EventArgs 
    { 
     public short Terminal { get; set; } 
     public string Message { get; set; } 
     public short FunctionNo { get; set; } 
    } 
} 


interface IServerLib 
    { 
     void AddMessageReceivedHandler(MessageReceivedEventHandler action); 
    } 
    public delegate void MessageReceivedEventHandler(short terminal, ref string message, ref short functionNo); 

Абонент (который бывает VB) выглядит следующим образом:

Dim WithEvents _va As MyAssembly.MyClass 

Private Sub _va_MessageReceived(sender As Object, e As Factory.MessageReceivedEventArgs) Handles _va.MessageReceived 
    Debug.WriteLine($"Message: {e.Message} Terminal: {e.Terminal} Function: {e.FunctionNo}") 
    If e.Message = "1" Then 
     e.Message = "" 
     e.FunctionNo = 0 
     Debug.WriteLine("Cancelled") 
    End If 
End Sub 

Это вызывает событие, но установка e.Message и e.Function, похоже, не устанавливают значения. Я делаю что-то неправильно?

Answer 1

Проблема заключается в использовании этой линии:

var args = new MessageReceivedEventArgs 
      { Terminal = terminal, Message = message, FunctionNo = functionNo }; 

Он копирует все переменные в класс событий args. Изменение его там автоматически не меняет его на другом конце (где вы добавили ref). Не хорошее решение, но доказательство для вас это проблема, добавьте это после handler(this, args):

message = args.Message; 
functionNo = args.FunctionNo; 

Это заставит ref s перезаписать значение.

Answer 2

Кажется, вы считаете, что ваш параметр «ref» должен быть затронут указанным выше кодом, но это неверно. Да, вы передаете свой message ссылкой на функцию MessageReceivedEvent, но тогда вы назначаете его MessageReceivedEventArgs.Message, и это происходит по значению, а не по ссылке.

В результате при изменении MessageReceivedEventArgs.Message в коде VB - message переменная не влияет (но MessageReceivedEventArgs.Messageэто влияет конечно), несмотря на то что вы прошли его по ссылке, как это должно быть.

Что вы должны сделать вместо этого проходит экземпляр MessageReceivedEventArgs вашей функции напрямую (не создавая его внутри этой функции):

private void MessageReceivedEvent(MessageReceivedEventArgs args) 
{ 
    EventHandler<MessageReceivedEventArgs> handler = MessageReceived; 
    if (handler != null) 
    { 
     handler(this, args); 
    } 
} 

Answer 3

Прежде всего, вам не нужно ref, чтобы изменить имущество вашего MessageReceivedEventArgs. Переменная объекта - это просто указатель на реальный объект в памяти, поэтому вы уже меняете его свойства. Если вам нужен ваш абонент для изменения самой ссылки - то есть для сброса указателя на другой объект - тогда вам нужно использовать ref. Во-вторых, этот дизайн ужасен, поэтому я не уверен, что понимаю проблему. Во всяком случае, я собираюсь предоставить правильный способ поднять события и использовать измененные значения после этого.

class Factory 
{ 
    public event EventHandler<MessagereceivedEventArgs> MessageReceived; 

    void ReceiveMessage(string Message) 
    { 
     // Do something with the Message 

     // Then let your subscribers know that the message has been processed: 
     if (MessageReceived != null) 
     { 
      var ea = new MessageReceivedEventArgs(); 
      ea.Message = Message; 
      // Set ea properties as appropriate 
      MessageReceived(this, ea); 
      // Check ea properties for change 
      if (ea.Message != Message) 
      { 
       // A subscriber has changed the message in the MessageReceivedEventArgs 
      } 
     } 
    } 
}