Question

In my current situation I have a class that performs an operation synchronously. During that operation an certain event will the thrown several times, depending on how often the situation where the event is raised occurs.
I know how the event-mechanism works and what the recommended ways of using are. Works well.(Note: My application is single-threaded)

What I want to is, to set a value to a property in my EventArgs. I’ve never had the need to set properties there.

This is a simplified situation of my current situation: (Note, I don’t need answers telling me to use regex, or stringreplace, because that won’t work in this situation)

I have this EventArgs:

public class TestEventArgs : EventArgs
{

    public String OldString { get; private set; }
    public String NewString { get; set; }

    public TestEventArgs(String oldString)
    {
        this.OldString = oldString;
    }
}

I do normally raise events this way:

public event EventHandler<TestEventArgs> ChangeString;

protected virtual void OnChangeString(String oldString)
{
    EventHandler<TestEventArgs> handler = this.ChangeString;
    if (handler != null)
    {
        handler(this, new TestEventArgs(oldString));
    }
}

and by calling the OnChangeString method, because I read that this in the way to raise events a long time ago.

Modified code where I need a value of the EventArgs after it’s been raised:

public event EventHandler<TestEventArgs> ChangeString;

protected virtual void OnChangeString(TestEventArgs args)
{
    EventHandler<TestEventArgs> handler = this.ChangeString;
    if (handler != null)
    {
        handler(this, args);
    }
}
    public void Foo()
    {
        String oldString = "this is the old string";
        // this.OnChangeString(oldString) // this is the way I called before
        // now I need to keep a reference to the EventArgs
        TestEventArgs args = new TestEventArgs(oldString);
        this.OnChangeString(args);
        // here I do have full access to args.NewString
    }

So, is it ok to keep a reference for the EventArgs and to raise the event with a method that accepts my EventArgs as parameter?

Answer 1

I think your question boils down to:

Is it OK to use the the arguments of an event to communicate data back to the source of the event?

First and foremost: at the low level, this will work. You can modify the event object, and you can see those changes in the code that raises the event.

But what if there are multiple event listeners. Who gets to set NewString?

If you expect only one, it is more clear to pass a delegate (of type Func<String, String>) to the class that now raises the event.