Замечания по паттерну наблюдатель

vstdio / ood

Репозиторий для лабораторных работ по дисциплине "Объектно-ориентированное проектирование"

MIT License

0 stars 0 forks source link

Замечания по паттерну наблюдатель #2

Closed alexey-malov closed 6 years ago

alexey-malov commented 6 years ago

https://github.com/chosti34/ood/blob/595d49a7b1688a4cec92071c7c03ccb69894846f/Observer/ObserversPriorities/Observer.h#L93-L99

[x] Странные имена для map-ов. В коде получается, что наблюдателя мы ищем среди приоритетов.

alexey-malov commented 6 years ago

https://github.com/chosti34/ood/blob/595d49a7b1688a4cec92071c7c03ccb69894846f/Observer/ObserversPriorities/Observer.h#L42-L50

[x] Вместо insert(make_pair()) лучше использовать emplace
[x] Если второй insert выбросит исключение, то класс перейдёт в невалидное состояние. Можно использовать BOOST_SCOPE_EXIT, gsl::finally либо ловить и обрабатывать исключения вручную.

alexey-malov commented 6 years ago

https://github.com/chosti34/ood/blob/595d49a7b1688a4cec92071c7c03ccb69894846f/Observer/ObserversPriorities/Observer.h#L85-L88

[x] Убрать ведение счётчика. Это лишняя деталь реализации. Узнать, кто подписан, а кто нет, можно отправив уведомление. Подписчики должны его получить, а отписанные объекты - нет

alexey-malov commented 6 years ago

https://github.com/chosti34/ood/blob/595d49a7b1688a4cec92071c7c03ccb69894846f/Observer/MultipleSubjects/Observer.h#L28-L37

[x] Не надо добавлять description в Observable. Вместо этого надо сравнивать указатель, передаваемый в Update с адресами известных weather info

alexey-malov commented 6 years ago

[x] ValueStatistics::OnValueChange переименовать в ValueStatistics::Accumulate

alexey-malov commented 6 years ago

[x] Для реализации дисплея, наблюдающего за погодой внутри (без ветра) и вне помещения (с ветром) надо, чтобы наблюдатель реализовывал 2 интерфейса: IObserver<WeatherInfo> и IObserver<WeatherInfoPro>

alexey-malov commented 6 years ago

template <typename T>
class IObservable
{
public:
    virtual ~IObservable() = default;
    virtual void RegisterObserver(EventType event, IObserver<T>& observer) = 0;
    virtual void RemoveObserver(EventType event, IObserver<T>& observer) = 0;
    virtual void NotifyObservers(EventType event) = 0;
};

[x] IObservable - универсальный интерфейс, а EventType - application-specific. Сделать тип события параметром шаблона

alexey-malov commented 6 years ago

[x] В реализации с сигналами не нужно вводить IObservable и IObserver. Посмотрите пример с валютой на сигналах