Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 5.00/9: Рейтинг темы: голосов - 9, средняя оценка - 5.00
2549 / 1208 / 358
Регистрация: 30.11.2013
Сообщений: 3,826

Собственная реализация паттерна "Слушатель" - нужна конструктивная критика

17.03.2016, 00:47. Показов 1865. Ответов 3
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Добрый день,

наворотил код по работе с паттерном слушатель - есть класс контейнер данных и при их изменении всех кто подписался должен обновить свои текстовые переменные. Вот что получилось:


C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
#include <iostream>
#include <string>
#include <map>
#include <list>
using namespace std;
//----------------------------------------- DECL ----------------------------------
///////////////////////////////////////////////////////////////////////////////////
class Observable;
class Observer
{
public:
   virtual void processAttributeChange( Observable* sender, int attrID ) = 0;   
};
///////////////////////////////////////////////////////////////////////////////////
class Observable
{
private:
   typedef int                               ID;  
   typedef std::list<Observer*>              ObserversList;
   std::map<ID, ObserversList>               m_map;
protected:
   void addAttribute( ID attrID )
   {
      m_map.insert( {attrID, ObserversList() } );
   }
   void notify( ID attrID )
   {
      const ObserversList& cont = m_map[ attrID ];
      for(auto& it : cont)
      {
         it->processAttributeChange( this, attrID );
      }
   }
public:
   void subcribe( ID attrID, Observer* obj)
   {
      m_map[attrID].push_back( obj );
   }
};
///////////////////////////////////////////////////////////////////////////////////
 
 
 
 
 
 
 
 
 
 
 
 
///////////////////////////////////////////////////////////////////////////////////
//----------------------------------------- USE ----------------------------------
///////////////////////////////////////////////////////////////////////////////////
class Container : public Observable
{
public:
   enum eAttributes
   {
      ATTR_NAME,
      ATTR_X,
   };
private:
   std::string                               m_name;
   int                                       m_x;
public:
   Container()
   {
     
   }
   void setName( const std::string& name )
   {
      if(m_name != name)
      {
         m_name = name;
         this->notify( ATTR_NAME );
      }
   }
   std::string getName() const
   {
      return m_name;
   }
   int getX() const
   {
      return m_x;
   }
};
///////////////////////////////////////////////////////////////////////////////////
class Foo : public Observer
{
private:
   Container*                                m_container;
   std::string                               m_name;
public:
   Foo( Container* container) :
      m_container(container)
   {
      container->subcribe( container->ATTR_NAME, this );
   }
   void processAttributeChange( Observable* sender, int attrID ) override
   {
      if(sender == m_container)
      {
         if(m_container->ATTR_NAME == attrID)
         {
            m_name = m_container->getName();
            cout << "Update attr "<< attrID << " with: " << m_name << endl;
         }         
      }
   }
};
///////////////////////////////////////////////////////////////////////////////////
int main()
{
   Container cont;
   Foo foo( &cont );
   
   cont.setName( "privet" );
 
   return 0;
}
Не очень нравится eAttributes но в другом случаи пришлось бы раскрывать детали реализации: типы и возможно имена переменных в контейнере.

Нужна критикА)
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
17.03.2016, 00:47
Ответы с готовыми решениями:

Нужна конструктивная критика
Добрый день, форумчане. Я только сегодня начал проходить ООП. С указателями еще не подружился. Если можно, вы можете указать на ошибки,...

Собственная реализация стека. Критика
Покритикуйте пожалуйста реализацию. Самому мне не очень нравится момент с завершением работы программы в catch блоке, но не знаю как обойти...

Нужна конструктивная критика по оформлению и читабельности кода (Game Trainer C++)
Относительно недавно начал изучать C++. Для обучения решил написать трейнер попутно изучая язык/читая необходимую литературу разного рода. ...

3
Эксперт .NET
 Аватар для Usaga
14314 / 9398 / 1355
Регистрация: 21.01.2016
Сообщений: 35,431
17.03.2016, 04:46
Применимость данного кода можно было бы расширить, если в метод processAttributeChange передавать не только атрибут, но и какой-нибудь (опциональный) EventArgument с пользовательскими данными.

Так же не помешают методы unsubscribe и unsubscribeAll, так как подписчикам свойственно заканчиваться раньше издателя.

Ещё непонятно назначение метода addAttribute, так как и без него нужная запись будет создана в методе subscribe.
1
 Аватар для lemegeton
4903 / 2696 / 921
Регистрация: 29.11.2010
Сообщений: 5,783
18.03.2016, 21:20
Ну обзерверы на обзерваблы обычно "подписываются" не сами, а явно, ибо избыточная связанность.

Во-вторых, не хотите сделать обзервабл на нейм сразу? Или тогда уж передавать айди аттрибьюта и его значение. А то так вы заставляете обзервера знать тип обзервабла.
0
Модератор
Эксперт CЭксперт С++
 Аватар для sourcerer
5288 / 2376 / 342
Регистрация: 20.02.2013
Сообщений: 5,773
Записей в блоге: 20
19.03.2016, 10:27
Цитата Сообщение от rikimaru2013 Посмотреть сообщение
паттерном слушатель
Наблюдатель же.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
19.03.2016, 10:27
Помогаю со студенческими работами здесь

Нужна конструктивная критика: советы, замечания по заданному фрагменту кода
Есть входные данные, которые я кладу в вектор 5 1 2 3 4 5 Ваша критика #include &lt;iostream&gt; #include &lt;vector&gt;

Нужна конструктивная критика: удалось ли мне уловить хоть частицу ООП в моём проекте?
Добрый день, дорогие форумчане. Меня, как и многих, волнует мое будущее, и поэтому же меня волнует понимание ООП, я написал змейку и хотел...

Нужна конструктивная критика
Недавно залил сайт (удалено топикстартером). Нужна оценка со стороны, в разделе &quot;СТЕНДЫ&quot; категория &quot;ПРОБКОВЫЕ&quot;. С точки...

Нужна конструктивная критика программы
Программа 'Быстрое чтение' (101 КБ) http://websup.narod.ru/prog/Fastread.zip - разработана для развития навыков быстрого чтения и...

Нужна конструктивная критика подбираемому конфигу ПК
День добрый! Набросал предварительный вариант новой конфигурации ПК, прошу конструктивно покритиковать. Буду рад советам и...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Новые блоги и статьи
Вывод данных через динамический список в справочнике
Maks 01.04.2026
Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .
Функция заполнения текстового поля в реквизите формы документа
Maks 01.04.2026
Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .
К слову об оптимизации
kumehtar 01.04.2026
Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .
Идея фильтра интернета (сервер = слой+фильтр).
Hrethgir 31.03.2026
Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .
Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда
anaschu 31.03.2026
В прикрепленном документе раздумья о том, как можно поменять модель в будущем
10 пpимет, которые всегда сбываются
Maks 31.03.2026
1. Чтобы, наконец, пришла маршрутка, надо закурить. Если сигарета последняя, маршрутка придет еще до второй затяжки даже вопреки расписанию. 2. Нaдоели зима и снег? Не надо переезжать. Достаточно. . .
Перемещение выделенных строк ТЧ из одного документа в другой
Maks 31.03.2026
Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .
Functional First Web Framework Suave
DevAlt 30.03.2026
Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru