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

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

17.03.2016, 00:47. Показов 1844. Ответов 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
14140 / 9369 / 1350
Регистрация: 21.01.2016
Сообщений: 35,270
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
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image
8Observer8 10.02.2026
Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .
Установка Qt-версии Lazarus IDE в Debian Trixie Xfce
volvo 10.02.2026
В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .
SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer
8Observer8 08.02.2026
Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .
SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств
8Observer8 07.02.2026
Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве
8Observer8 02.02.2026
Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .
Философия технологии
iceja 01.02.2026
На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf
8Observer8 01.02.2026
Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru