Элегантный способ предотвратить циклические события в MVC?

Это сложный вопрос. Если я правильно понимаю, проблема возникает из-за того, что вы предоставили обработчик кликов в своей модели, а событие click модели перехватывается контроллером. Контроллер обновляет представление, которое, в свою очередь, переключает то же событие.

С моей точки зрения, я считаю неуместным для контроллера присоединяться к событию Edge Click, потому что оно раскрывает слишком много деталей о том, как Edge реализован и используется. Контроллер не заботится о том, как используется Edge, или о каких-либо других деталях реализации.

Фактически, канонический стиль MVC вообще не требует, чтобы Контроллер подключался к событиям модели любой, как правило, потому, что состояние модели не изменяется представлением или любыми другими контроллерами. Модель не обязана уведомлять Контроллер о том, что она была изменена.

Чтобы решить вашу проблему, вы должны определить интерфейс View, чтобы иметь один метод, например ToggleEdge:

public interface GraphView
{
    event Action ToggleEdge;
}

Заманчиво создать два метода, EdgeClicked и CheckboxClicked, но настаивание на двух таких независимых методах нарушает принцип инкапсуляции. Он предоставляет слишком много деталей реализации вашему контроллеру или кому-либо еще, кто хочет подключиться к этим событиям. Помните, что Контроллер заботится только об изменении состояния представления, его не волнует, что как изменилось.

Когда вы реализуете интерфейс View в своем пользовательском интерфейсе, вы должны позаботиться о том, чтобы событие ToggleEdge было вызвано из местоположения один. Вы можете сделать это, подключившись к событию Edge.Clicked в вашем представлении и используя его для переключения флажка; это делает ваш флажок ответственным за поднятие клапана Toggle до контроллера:

public class UI : UserControl, GraphView
{
    public event Action ToggleEdge;

    void OnToggleEdge(Edge edge)
    {
        if (ToggleEdge != null)
            ToggleEdge(edge);
    }

    protected void Edge_Clicked(object sender, EventArgs e)
    {
        CheckBox chkbox = FindCheckBoxThatCorrespondsToEdge((Edge)sender);
        chkbox.Checked = !chkbox.Checked;    
    }

    protected void chkEdge_CheckChanged(object sender, EventArgs e)
    {
        Edge edge = FindEdgeThatCorrespondsToCheckbox((CheckBox)sender);
        OnToggleEdge(edge);
    }
}

Вы можете заявить, что View слишком много знает о своей реализации: ему известно, что края и флажки фундаментально связаны. Возможно, это еще один взлом, но его, вероятно, можно отклонить, так как «логика пользовательского интерфейса» должна поддерживать синхронизацию отображения представления.

Просто чтобы повторить модель MVC. Представления обычно должны обновляться. Вот как это работает: контроллер изменяет состояние модели, модель отправляет обновления своим представлениям, представления извлекают новое состояние из модели и обновляются сами. Хотя контроллеры и представления, как правило, объединены (т.е. детализируют данные в графическом представлении), они никогда не должны взаимодействовать напрямую, только через модель. Это в общем конечно.

Таким образом, JS-функции, обновляющие ваши представления, на самом деле не являются контроллерами, что является важным отличием. Их следует рассматривать как часть вашей точки зрения. Возможно, это не поможет решить проблему, но я подумал, что на это стоит обратить внимание.

Вы также не можете удалить свою модель, я полагаю, вы имеете в виду, что вы удаляете что-то из из своей модели, поскольку никакие представления или контроллеры не могут существовать (или находиться в функциональном состоянии), если они не поддерживаются моделью.

Не будучи жокеем кода JS и не использовав gmaps, я действительно не понимаю, в чем проблема. Запускает ли изменение состояния флажка (свойство checked) событие onClick ()? Это действительно не должно ИМХО, но, возможно, они реализовали это таким образом, иначе вы могли бы просто прикрепить свой контроллер к onClick () и добавить некоторую логику к флажку (или, если это JS, где-то в функции), чтобы изменить состояние флажка . Если это невозможно, лучше всего подойдут варианты 1 и 2.

дополнение: пользователь взаимодействует с представлением

Так что же происходит, когда пользователь хочет взаимодействовать с представлением? Часто виджет включает в себя и представление, и контроллер. У флажка есть представление (вы можете увидеть, отмечен он или нет), а также контроллер (вы можете щелкнуть по нему). Когда вы устанавливаете флажок, в принципе должно происходить следующее:

• Контроллер флажка получает событие
• Контроллер флажка изменяет состояние для значения, которое этот флажок представляет в модели
• слушатели обновлений модели (включая флажок)
• флажок обновляет свой вид, чтобы отразить, что это значение изменилось

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