Правильно ли реализован паттерн Observer?

Question

[egorggegor]

Привет, помогите понять, как должен выглядеть паттерн Observer, везде пишут по-разному, у всех разные диаграммы, чего в моей реализации не хватает или, может быть, что-то лишнее?


Моя реализация (может быть, если в коде ошибке скажите пожалуйста, где что некорректно):

class Observer;

class Subject {
public:
    virtual ~Subject() {};
    virtual void attach(Observer* follower) {
        followers.push_back(follower);
    }
    virtual void detach(int index) {
        followers.erase(followers.begin() + index);
    }
    virtual void Notify() {
        for (int i = 0; i < followers.size(); ++i) {
            
        }
    }
private:
    std::vector<Observer*> followers;
};

class ConcreteSubject : public Subject {
public:
    int GetState() {
        return SubjectState;
    }
    void SetState(int NewSubjectState) {
        SubjectState = NewSubjectState;
    }
private:
    int SubjectState;
};

class Observer {
public:
    virtual ~Observer() {};
    virtual void Update() = 0;
};

class ConcreteObserver : public Observer {
public:
    ConcreteObserver(ConcreteSubject* subject_to_follow) {
        FollowedSubject = subject_to_follow;
        FollowedSubject->attach(this);
    }
    void Update() {
        ObserverState = FollowedSubject->GetState();
    }
private:
    ConcreteSubject* FollowedSubject;
    int ObserverState;
};

Comments

[Дмитрий]

https://refactoring.guru/ru/design-patterns/observ...

Снизу кнопки для других яп.

https://refactoring.guru/ru/design-patterns/observer

Answer 1

[Сергей Егоров]

Одну строчку осталось добавить...

virtual void Notify() {
  for (int i = 0; i < followers.size(); ++i) {
    followers[i].Update();            
  }
}

Comments

[egorggegor]

Спасибо, и вправду проглядел, а больше ничего в паттерн добавить не нужно?

[Сергей Егоров]

Да вроде всего хватает... суть шаблона в том, что предмет, уведомляет своих подписчиков о каких-то событиях, которые с ним произошли, и проверяется это простым тестом:

/// Где-то выше объявлены Subject и Observer
public class ObserverCounter : Observer 
{
  public int Count { get; set; }

  public void Update()
  {
   this.Count += 1;
  }
}

Subject subject = new Subject();
ObserverCounter observer = new ObserverCounter();
Assert.Equal(0, observer.Count);

subject.attach(observer);
subject.notify(); /// Вот здесь предмет уведомил обозревателя о чем-то

Assert.Equal(1, observer.Count); /// И счетчик оказался больше на единицу