Как и какие идиомы и паттерны можно (и лучше) применять?

@DVyacheslavN · Регистрация: 28.11.2015

Author24 — интернет-сервис помощи студентам

Здравствуйте. Писал я проги для себя ну и так по мелочи, для абы кого, да и проги абы как, не задумываясь о том как какие идиомы (паттерны) можно применять, так получилось что узнал я значит про все и теперь сижу и ломаю голову как все это лучше сделать.
Вот к примеру решил я написать класс Service(создание объектов и предоставление доступа к методам get send) хотел сделать Паттерн Factory Method , для управления сетевыми соединениями (Tcp, serial, ну и тд.) после установки соединения от них нужны методы get и send, с этим понятно, а вот контроль создания Tcp можно отдать Service но так получилось что Tcp у меня singleton и он уже контролирует сам себя.

Добавлено через 21 час 4 минуты
решил я сильно голову не греть и потихоньку приводить проект в нормальный вид, а там уж что получится ))

MrGluck · 19.04.2018, 13:17

Молодец.

@COKPOWEHEU · 19.04.2018, 13:59

Сообщение от DVyacheslavN

так получилось что Tcp у меня singleton и он уже контролирует сам себя.

Вы уверены что это хорошая идея? Обычно программа может открыть несколько соединений с разными адресами, единственное соединение это все же скорее исключение.
Контроль создания (наверное, подключения?) возможно стоит отдать самому объекту соединения и уже в его "статусе" смотреть подключилось или нет.
Впрочем, саму задачу вы не раскрыли, так что все это только общие соображения, которые в вашем случае могут оказаться совсем неверными.

@DVyacheslavN · 19.04.2018, 14:30 **[ТС]**

COKPOWEHEU, вот Tcp может работать с несколькими соединениями, при новом соединении на стороне сервера Мы получаем новый соккет, а все остальное остается неизменным. Но объект TCP один поэтому и singleton, а вот Serial можно много открывать, Думаю можно создать один SuperSerial для много serial и а при создании объекта получать дескриптор (ключ) по которому отправлять данные в нужный serial. Хотя я пытался что то похожее сделать но запутался и откатился назад.

Сообщение от COKPOWEHEU

Контроль создания (наверное, подключения?) возможно стоит отдать самому объекту соединения

RAII идиому хочу по отношению к serial применить. ? Разве правильно что объект будет сам себя контролировать. Если при инициализации будет ошибка вызову деструктор. Зачем объект нерабочий нужен.

Добавлено через 3 минуты
в общем раньше я не парился ))) а сейчас все же охота порядок навести, да и читаемость кода улучшится RAII мне за это и нравится постараюсь полностью избавится от лишних seting ов

@COKPOWEHEU · 19.04.2018, 16:09

Я бы скорее делал на каждое соединение отдельный объект, но это уже вкусовщина.

Сообщение от DVyacheslavN

а вот Serial можно много открывать

Что такое Serial? COM-порт, UDP-соединение?

Сообщение от DVyacheslavN

Разве правильно что объект будет сам себя контролировать

А с другой стороны меньше связей с другими объектами, меньше разделения ответственности. В целом это опять вкусовщина.
Любую задачу можно решить несколькими способами. А еще несколькими правильными способами.
Все эти паттерны и идеологии нужны для того чтобы было проще использовать код повторно или выдрать отдельный объект (не обязательно в терминах ООП, это может быть пара cpp-h файлов) и отладить его максимально изолировано. Или просто навести порядок с уровнями абстракции, чтобы низкий уровень железа и битов не смешивался с высоким уровнем конкретных задач и объектов.

@DVyacheslavN · 19.04.2018, 16:17 **[ТС]**

Serial это com , про паттерны я и не задумывался "было проще использовать код повторно или выдрать "

@COKPOWEHEU · 23.04.2018, 10:29

Сообщение от DVyacheslavN

Serial это com

А в чем разница? Что COM-портов может быть несколько, что TCP-соединений. А в некоторых случаях и по нескольку слушателей на "том конце" интерфейса.

Сообщение от DVyacheslavN

про паттерны я и не задумывался

Ну вы же хотите их использовать не только "чтобы было", а чтобы упростить разработку.

@DVyacheslavN · 23.04.2018, 11:38 **[ТС]**

Сообщение от COKPOWEHEU

А в чем разница? Что COM-портов может быть несколько, что TCP-соединений.

http://rsdn.org/article/unix/sockets.xml
Функция accept создаёт для общения с клиентом новый сокет и возвращает его дескриптор. Параметр sockfd задаёт слушающий сокет. После вызова он остаётся в слушающем состоянии и может принимать другие соединения.. Поэтому и singleton, да и нет необходимости, Вопрос есть про отладку

C++ (Qt)

1
2
3

#ifdef QT_DEBUG
         qDebug() << "TcpSocket is null";
#endif

стоит ли с макросами колдовать ?? может другие варианты есть;
отладка нужна ведь только мне и когда все будет отлажено нужно ее убрать из кода.

@COKPOWEHEU · 23.04.2018, 13:29

Сообщение от DVyacheslavN

Параметр sockfd задаёт слушающий сокет. После вызова он остаётся в слушающем состоянии и может принимать другие соединения.

TCP предназначен для связи двух собеседников. Вы открываете сокет, ждете пока постучится клиент и создаете с ним связь. Если постучится второй клиент, для него можно открыть другой сокет и так далее. И даже разнести эти сокеты по разным потокам. Именно по TCP, правда, только теорию читал, а вот с UDP хочу повозиться на практике, и там предполагается либо один общий сокет для всех клиентов и фильтрация по IP, либо на каждого свой. Пока не знаю как лучше, там видно будет.

Сообщение от DVyacheslavN

стоит ли с макросами колдовать ?? может другие варианты есть;

assert()'а не хватит?

Сообщение от DVyacheslavN

отладка нужна ведь только мне и когда все будет отлажено нужно ее убрать из кода.

"когда все будет отлажено"! Какой оптимизм. Лучше настроить уровни предупреждений, чтобы собранная с максимальным уровнем писала в логи вообще все и как можно подробнее, а с минимальным (когда ее уже можно отдавать пользователям) - только о самых важных ошибках, которые они могут сами исправить. Никто ведь не застрахован от неправильного ввода или опечаток в коде.
Собственно, я бы предложил логи вообще не отключать.

@DVyacheslavN · 23.04.2018, 14:37 **[ТС]**

Сообщение от DVyacheslavN

qDebug() << "TcpSocket is null"

ну это у меня не совсем даже и ошибка просто исключительная ситуация, можно конечно сообщения для пользователя написать "! Alarm TcpSocket is null", пытаюсь минимальный функционал сделать, наверное так и оставлю потом просто signal message("blabla") сделаю вместо qDebug();

@COKPOWEHEU · 23.04.2018, 15:29

Как вариант - обмазать все макросами:

C

#define DEBUG
 
#ifdef DEBUG
  #define DBGIF(cond, func...) do{if(cond){func}}while(0)
#else
  #define DBGIF(cond, cunc...)
#endif
 
int main(){
  DBGIF(1>0, printf("AA\n"); printf("BB\n"););
}

В варианте дебага в исходник добавится весь код отладочного вывода, в варианте бага - не добавится. Хотя такая запись макроса, конечно, немного мозговыносящая

Добавлено через 34 минуты
Да, кстати. Вы можете поменять название на более осмысленное? Ну хотя бы "подгонка архитектуры под идеологию RAII"

@DVyacheslavN · 27.04.2018, 04:31 **[ТС]**

Добавлено через 1 минуту

Сообщение от COKPOWEHEU

Как вариант - обмазать все макросами:

решил не обмазывать ))) сделал вот так

C++ (Qt)

#ifdef SERIAL_MODBUS_DEBUG
   connect(this, &SeriaModbus::message, this, [] (QString str)
   {
      qDebug() << str << "<-- SerialModbus";
   });
#endif

а дальше по коду

C++ (Qt)

1	emit message("bla bla");

@DVyacheslavN · 28.04.2018, 22:43 **[ТС]**

Есть вопрос, в общем есть 2 класса (A, B)с похожим функционалом унаследованные от 1 Абстрактного родителя, классы собирают инфу с сетевых портов (COM, и TCP), что я сделал создал класс SuperService создаю там объекты классов (A, B) соединяю сигналы этих классов с методами SuperService, правильный подход или можно было как то лучше решить, в конечном итоге задача сводится к аккумулированию информации из разных объектов в одном месте.

Добавлено через 8 минут
и 2 вопрос, принципы ООП заставляют меня создавать больше классов чем обычно (раньше все было в куче), обзор проекта в IDE уже глаз начинает резать, как то можно от этого избавится ?

Добавлено через 37 минут

Сообщение от DVyacheslavN

обзор проекта в IDE

пробовал includepath, я так понял но просит чтобы реализация методов была в header файле который подключаю ((

@vlisp · 29.04.2018, 06:12

Сообщение от DVyacheslavN

принципы ООП заставляют меня создавать больше классов чем обычно

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

@DVyacheslavN · 05.05.2018, 10:29 **[ТС]**

Сообщение от vlisp

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

только мне знать сколько их нужно и сколько обычно я создаю в скобках я пояснил почему так

Сообщение от DVyacheslavN

больше классов чем обычно (раньше все было в куче),

Вообще может спрятать часть классов в статическую библиотеку ) пока других идей нет. Список классов большой а работаю с 3-4 взаимосвязанными классам, жаль в Qt Creator нет фильтра по избранному.

@DVyacheslavN 3 / 3 / 1 Регистрация: 28.11.2015 Сообщений: 108
		1
	Как и какие идиомы и паттерны можно (и лучше) применять? 19.04.2018, 13:13. Показов 1793. Ответов 14 Метки нет (Все метки) Здравствуйте. Писал я проги для себя ну и так по мелочи, для абы кого, да и проги абы как, не задумываясь о том как какие идиомы (паттерны) можно применять, так получилось что узнал я значит про все и теперь сижу и ломаю голову как все это лучше сделать. Вот к примеру решил я написать класс Service(создание объектов и предоставление доступа к методам get send) хотел сделать Паттерн Factory Method , для управления сетевыми соединениями (Tcp, serial, ну и тд.) после установки соединения от них нужны методы get и send, с этим понятно, а вот контроль создания Tcp можно отдать Service но так получилось что Tcp у меня singleton и он уже контролирует сам себя. Добавлено через 21 час 4 минуты решил я сильно голову не греть и потихоньку приводить проект в нормальный вид, а там уж что получится )) 0

MrGluck Форумчанин 8215 / 5045 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	19.04.2018, 13:17	2
	Сообщение было отмечено IGPIGP как решение Решение Молодец. 1

@COKPOWEHEU 3881 / 2479 / 418 Регистрация: 09.09.2017 Сообщений: 10,888
	19.04.2018, 13:59	3
	Сообщение от DVyacheslavN так получилось что Tcp у меня singleton и он уже контролирует сам себя. Вы уверены что это хорошая идея? Обычно программа может открыть несколько соединений с разными адресами, единственное соединение это все же скорее исключение. Контроль создания (наверное, подключения?) возможно стоит отдать самому объекту соединения и уже в его "статусе" смотреть подключилось или нет. Впрочем, саму задачу вы не раскрыли, так что все это только общие соображения, которые в вашем случае могут оказаться совсем неверными. 0

@DVyacheslavN 3 / 3 / 1 Регистрация: 28.11.2015 Сообщений: 108
	19.04.2018, 14:30 [ТС]	4
	COKPOWEHEU, вот Tcp может работать с несколькими соединениями, при новом соединении на стороне сервера Мы получаем новый соккет, а все остальное остается неизменным. Но объект TCP один поэтому и singleton, а вот Serial можно много открывать, Думаю можно создать один SuperSerial для много serial и а при создании объекта получать дескриптор (ключ) по которому отправлять данные в нужный serial. Хотя я пытался что то похожее сделать но запутался и откатился назад. Сообщение от COKPOWEHEU Контроль создания (наверное, подключения?) возможно стоит отдать самому объекту соединения RAII идиому хочу по отношению к serial применить. ? Разве правильно что объект будет сам себя контролировать. Если при инициализации будет ошибка вызову деструктор. Зачем объект нерабочий нужен. Добавлено через 3 минуты в общем раньше я не парился ))) а сейчас все же охота порядок навести, да и читаемость кода улучшится RAII мне за это и нравится постараюсь полностью избавится от лишних seting ов 0

@COKPOWEHEU 3881 / 2479 / 418 Регистрация: 09.09.2017 Сообщений: 10,888
	19.04.2018, 16:09	5
	Я бы скорее делал на каждое соединение отдельный объект, но это уже вкусовщина. Сообщение от DVyacheslavN а вот Serial можно много открывать Что такое Serial? COM-порт, UDP-соединение? Сообщение от DVyacheslavN Разве правильно что объект будет сам себя контролировать А с другой стороны меньше связей с другими объектами, меньше разделения ответственности. В целом это опять вкусовщина. Любую задачу можно решить несколькими способами. А еще несколькими правильными способами. Все эти паттерны и идеологии нужны для того чтобы было проще использовать код повторно или выдрать отдельный объект (не обязательно в терминах ООП, это может быть пара cpp-h файлов) и отладить его максимально изолировано. Или просто навести порядок с уровнями абстракции, чтобы низкий уровень железа и битов не смешивался с высоким уровнем конкретных задач и объектов. 1

@DVyacheslavN 3 / 3 / 1 Регистрация: 28.11.2015 Сообщений: 108
	19.04.2018, 16:17 [ТС]	6
	Serial это com , про паттерны я и не задумывался "было проще использовать код повторно или выдрать " 0

@COKPOWEHEU 3881 / 2479 / 418 Регистрация: 09.09.2017 Сообщений: 10,888
	23.04.2018, 10:29	7
	Сообщение от DVyacheslavN Serial это com А в чем разница? Что COM-портов может быть несколько, что TCP-соединений. А в некоторых случаях и по нескольку слушателей на "том конце" интерфейса. Сообщение от DVyacheslavN про паттерны я и не задумывался Ну вы же хотите их использовать не только "чтобы было", а чтобы упростить разработку. 0

@COKPOWEHEU 3881 / 2479 / 418 Регистрация: 09.09.2017 Сообщений: 10,888
	23.04.2018, 13:29	9
	Сообщение от DVyacheslavN Параметр sockfd задаёт слушающий сокет. После вызова он остаётся в слушающем состоянии и может принимать другие соединения. TCP предназначен для связи двух собеседников. Вы открываете сокет, ждете пока постучится клиент и создаете с ним связь. Если постучится второй клиент, для него можно открыть другой сокет и так далее. И даже разнести эти сокеты по разным потокам. Именно по TCP, правда, только теорию читал, а вот с UDP хочу повозиться на практике, и там предполагается либо один общий сокет для всех клиентов и фильтрация по IP, либо на каждого свой. Пока не знаю как лучше, там видно будет. Сообщение от DVyacheslavN стоит ли с макросами колдовать ?? может другие варианты есть; assert()'а не хватит? Сообщение от DVyacheslavN отладка нужна ведь только мне и когда все будет отлажено нужно ее убрать из кода. "когда все будет отлажено"! Какой оптимизм. Лучше настроить уровни предупреждений, чтобы собранная с максимальным уровнем писала в логи вообще все и как можно подробнее, а с минимальным (когда ее уже можно отдавать пользователям) - только о самых важных ошибках, которые они могут сами исправить. Никто ведь не застрахован от неправильного ввода или опечаток в коде. Собственно, я бы предложил логи вообще не отключать. 0

@DVyacheslavN 3 / 3 / 1 Регистрация: 28.11.2015 Сообщений: 108
	23.04.2018, 14:37 [ТС]	10
	Сообщение от DVyacheslavN qDebug() << "TcpSocket is null" ну это у меня не совсем даже и ошибка просто исключительная ситуация, можно конечно сообщения для пользователя написать "! Alarm TcpSocket is null", пытаюсь минимальный функционал сделать, наверное так и оставлю потом просто signal message("blabla") сделаю вместо qDebug(); 0

@DVyacheslavN 3 / 3 / 1 Регистрация: 28.11.2015 Сообщений: 108
	28.04.2018, 22:43 [ТС]	13
	Есть вопрос, в общем есть 2 класса (A, B)с похожим функционалом унаследованные от 1 Абстрактного родителя, классы собирают инфу с сетевых портов (COM, и TCP), что я сделал создал класс SuperService создаю там объекты классов (A, B) соединяю сигналы этих классов с методами SuperService, правильный подход или можно было как то лучше решить, в конечном итоге задача сводится к аккумулированию информации из разных объектов в одном месте. Добавлено через 8 минут и 2 вопрос, принципы ООП заставляют меня создавать больше классов чем обычно (раньше все было в куче), обзор проекта в IDE уже глаз начинает резать, как то можно от этого избавится ? Добавлено через 37 минут Сообщение от DVyacheslavN обзор проекта в IDE пробовал includepath, я так понял но просит чтобы реализация методов была в header файле который подключаю (( 0

	05.05.2018, 10:29

@vlisp 621 / 941 / 150 Регистрация: 10.08.2015 Сообщений: 5,018
	29.04.2018, 06:12	14
	Сообщение от DVyacheslavN принципы ООП заставляют меня создавать больше классов чем обычно классы нужно создавать исходя из принятой объектной модели. их не может быть больше, чем нужно и тем более больше, чем обычно 0

@DVyacheslavN 3 / 3 / 1 Регистрация: 28.11.2015 Сообщений: 108
	05.05.2018, 10:29 [ТС]	15
	Сообщение от vlisp классы нужно создавать исходя из принятой объектной модели. их не может быть больше, чем нужно и тем более больше, чем обычно только мне знать сколько их нужно и сколько обычно я создаю в скобках я пояснил почему так Сообщение от DVyacheslavN больше классов чем обычно (раньше все было в куче), Вообще может спрятать часть классов в статическую библиотеку ) пока других идей нет. Список классов большой а работаю с 3-4 взаимосвязанными классам, жаль в Qt Creator нет фильтра по избранному. 0

Как и какие идиомы и паттерны можно (и лучше) применять?

Решение