Question

Using a separate class which is being made reusable, need to pass it a function which would be called on the click event from the context menu…

Problem : Functions aren’t type EventHandlers…. also Different EventHandlers require different paramters inside… e.g. the OnClose for the exit button….

Edit:

In the Class X

public void AddMenuItem(String name, EventHandler target )
    {
        MenuItem newItem = new MenuItem();
        newItem.Index = _menuItemIndex++;
        newItem.Text = name;
        newItem.Click += target;

        _contextMenu.MenuItems.Add(newItem);
    }

In the Wpf:

addToTray.AddMenuItem("&Exit", Exit); 

I would love it to link to the following method but at this point any method would do fine.

private void ShouldIExit(object sender, System.ComponentModel.CancelEventArgs e)
    {
        // checks if the Job is running and if so prompts to continue
        if (_screenCaptureSession.Running())
        {
            MessageBoxResult result = System.Windows.MessageBox.Show("Capturing in Progress. Are You Sure You Want To Quit?", "Capturing", MessageBoxButton.YesNo);
            if (result == MessageBoxResult.No)
            {
                e.Cancel = true;
                return;
            }
        }
        _screenCaptureSession.Stop();
        _screenCaptureSession.Dispose();
    }

Answer 1

I’m guessing that the problem is that your ShouldIExit method does not match the EventHandler delegate. Try changing it to take a regular EventArgs parameter, and see if that works. It’s best to avoid reusing the same event handler for different event types. You should encapsulate common code into separate methods, and then have different handlers call that code.

private bool CheckExit()
{
    // checks if the Job is running and if so prompts to continue
    if (_screenCaptureSession.Running())
    {
        MessageBoxResult result = System.Windows.MessageBox.Show("Capturing in Progress. Are You Sure You Want To Quit?", "Capturing", MessageBoxButton.YesNo);
        if (result == MessageBoxResult.No)
        {
            return false;
        }
    }
    _screenCaptureSession.Stop();
    _screenCaptureSession.Dispose();
    return true;
}

private void ExitButtonClicked(object sender, System.ComponentModel.CancelEventArgs e)
{
    if (!CheckExit())
    {
        e.Cancel = true;
    }
}

private void ExitMenuItemClicked(object sender, EventArgs e)
{
    CheckExit();
}