«System.StackOverflowException» произошло в PresentationCore.dll

Question

У меня есть WPF UserControl, в котором я пытаюсь реализовать custom MouseClick (потому что нет события MouseClick в событии WPF (User) Control ).

Я получил следующее:

Код:

/// <summary>
/// Occurs when users left clicks the MyControl.
/// </summary>
public event MouseButtonEventHandler MouseClick { add { AddHandler(MouseClickEvent, value); } remove { RemoveHandler(MouseClickEvent, value); } }

    protected virtual void OnMouseClick(MouseButtonEventArgs e)
    {
        base.RaiseEvent(e);
        //this.RaiseEvent(new RoutedEventArgs(MouseClickEvent, this));
    }

    protected override void OnMouseLeftButtonUp(MouseButtonEventArgs e)
    {
        base.OnMouseLeftButtonUp(e);
        if (!this.movedAfterMouseDown)
        {
            OnMouseClick(e);
        }
        this.gotLeftButtonDown = false;
        this.movedAfterMouseDown = false;
    }

Так в чем же проблема?

ОБНОВЛЕНИЕ 1

protected virtual void OnMouseClick(MouseButtonEventArgs e)
{
    //base.RaiseEvent(e);
    MouseButtonEventArgs args = new MouseButtonEventArgs(e.MouseDevice, e.Timestamp, e.ChangedButton);
    this.RaiseEvent(args);
}

Значение не может быть нулевым. Имя параметра: routedEvent

ОБНОВЛЕНИЕ 2

Другое пользовательское событие, которое я успешно реализовал (работает без проблем) - SelectedChanged:

static void OnIsSelectedChanged(DependencyObject source, DependencyPropertyChangedEventArgs e)
{
    var s = (MyControl)source;

    s.RaiseEvent(new RoutedEventArgs(SelectedChangedEvent, s));
}

ОБНОВЛЕНИЕ 3

Реализация System.Windows.Controls.Control OnPreviewMouseDoubleClick:

protected virtual void OnPreviewMouseDoubleClick(MouseButtonEventArgs e)
{
    base.RaiseEvent(e);
}

Обновление 5 (для людей в танке)

class Foo : FrameworkElement
{
    event EasterCameEvent; // I named it MouseClick

    public DoSomething()
    {
        EasterCameArgs args= ...

        if (Date.Now = EasterDate)
            OnEasterCame(args)
    }

    protected virtual void OnEasterCame(EasterCameArgs e)
    {
        base.RaiseEvent(e);
    }
}

Answer 1

Вам необходимо удалить base.RaiseEvent (e) из ВСЕХ частей вашего пользовательского элемента управления. Это является причиной переполнения стека.

Если вы наследуете от UserControl, события щелчка уже реализованы для вас. Вам НЕ нужно повторно внедрять. Вы МОЖЕТЕ иметь дело с ними, но, скорее всего, нет.

protected override void OnMouseLeftButtonUp(MouseButtonEventArgs e)
{
    // handle the event if you need to do something with the data.
    // this is not over-riding the event, this is attaching a custom handler to the event
    this.gotLeftButtonDown = false;
    this.movedAfterMouseDown = false;
}

Это не перевешивает событие. Это обработчик, когда событие возникает! Пользователи вашего элемента управления будут писать обработчики так же, как этот. Не переопределяйте это. Даже не обращайтесь с этим, если вам не нужно что-то делать с данными. Эти события уже написаны для вас.

Edit:

Хотя мой ответ был неправильным в том смысле, что он не устранил проблему, все же полезно понять, ПОЧЕМУ происходило исключение stackoverflow.

protected virtual void OnMouseClick(MouseButtonEventArgs e)
{
    base.RaiseEvent(e);
    /* this will raise the OnMouseLeftButtonUp event if the MouseButtonEventArgs
       designates that it came from the MouseLeftButtonUp event. 
       That will then call the OnMouseLeftButtonUp because e.RoutedEvent equals
       MouseLeftButtonUpEvent.

       The accepted answer does the following, to stop OnMouseLeftButtonUp being
       called again, and the whole process repeating itself in an infinite loop.

       args.RoutedEvent = MouseClickEvent;
       base.RaiseEvent(e); // No longer fires MouseLeftButtonUp, breaking cycle.

       // Changes the event to be fired, avoiding the cycle of
       // OnMouseLeftButtonUp -> OnMouseClick -> OnMouseLeftButtonUp etc
    */
}

protected override void OnMouseLeftButtonUp(MouseButtonEventArgs e)
{
    base.OnMouseLeftButtonUp(e);
    if (!this.movedAfterMouseDown)
    {
        OnMouseClick(e); // This will call OnMouseClick handler
    }
    this.gotLeftButtonDown = false;
    this.movedAfterMouseDown = false;
}

Поток управления был:

1. Пользователь генерирует MouseLeftButtonUpEvent
2. OnMouseClick вызывается в обработчике OnMouseLeftButtonUp
3. OnMouseClick вызывает событие MouseLeftButtonUp
4. Перейти к 2.

Принятый ответ изменяется на:

1. Пользователь генерирует MouseLeftButtonUpEvent
2. OnMouseClick вызывается в обработчике OnMouseLeftButtonUp
3. OnMouseClick изменяет перенаправленное событие на MouseClickEvent
4. OnMouseClick вызывает MouseClickEvent
5. Управление возобновляется

Это то, что я пытался объяснить в комментариях к другим ответам. Если я не был достаточно ясен, я прошу прощения за это. Я полагаю, что мы с Серхио были не синхронизированы, поскольку пытались объяснить причину стека переполнения стека, когда он искал исправление в коде. Поправь меня, если я ошибаюсь.

Answer 2

protected virtual void OnMouseClick(MouseButtonEventArgs e)
{
    MouseButtonEventArgs args = new MouseButtonEventArgs(e.MouseDevice, e.Timestamp, e.ChangedButton);

    // Don't forget this
    args.RoutedEvent = MouseClickEvent;

    base.RaiseEvent(args);
}

Answer 3

Я думаю, что это в этой строке:

base.RaiseEvent(e);

Если вы обрабатываете щелчок мышью, вы не хотите снова вызывать событие, поскольку это просто вызовет ваш обработчик снова, что вызовет событие ....