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

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

17.03.2016, 00:47. Показов 1834. Ответов 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
14096 / 9313 / 1349
Регистрация: 21.01.2016
Сообщений: 34,988
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
Ответ Создать тему
Новые блоги и статьи
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Ниже машинный перевод статьи The Thinkpad X220 Tablet is the best budget school laptop period . Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы,. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru