Мастер Ломастер писал(а):упоминание об улучшении стройности программы за счет иерархии классов - это как раз ваш случай. это я понимаю и могу с этим согласиться без возражений.
Но каждый «листовой» класс при этом имеет единственный экземпляр, а топикстартер не упоминал, его классы самостоятельные, или чьи-то потомки
Мастер Ломастер писал(а):меня смутил единственный экземпляр единственного класса с обработчиком прерываний - по сравнению с отдельным модулем никаких преимуществ не имееющий, как мне кажется, но требующий перехода на заведомо более сложный инструмент С++. т.е. налицо возрастание сложности решения без заметной выгоды...
Если остальная программа
и так на плюсах (а это отдельный разговор, который шире обсуждения одного какого-то модуля-класса), то никакого специального перехода не будет.
Пусть нужен UART (в упомянутом выше приёмнике-анализаторе пакетов) или там I2C -- если не в микроконтроллере, то в программе вполне может быть в единственном экземпляре.
В программе, использующей scmRTOS, оформление «драйвера I2C» в виде модуля С-стиля только сделает программу «неоднородной», проще она точно от этого не станет.
А класс один без иерархии (ОП, а не ООП, если я правильно терминологию понимаю) и с единственным экземпляром -- вполне ляжет в стиль. Всякие там mutex-ы с event-ами будут не отдельно стоящими статическими объектами в отдельном файле, а полями (возможно, тоже статическими) класса «драйвера», конструкторы вызовутся, то-сё. Где тут усложнение -- уже не видно. Да и вообще не видно, после того, как инструмент освоен. А применять удобно.
И по барабану, что этот
uart -- единственный экземпляр единственного класса.
Он имеет
operator FILE* () специально для этого случая, а внутри и экземпляр класса очереди, и разные сервисы scmRTOS использует. Поток просто уснёт (а не будет в цикле поллить флаги UART), если в очереди места нет, будет разбужен, когда появится.
Выписать тот же функционал в чисто С-шном стиле через
Код: Выделить всё
FILE uart_file = FDEV_SETUP_STREAM(uart_putchar, ...);
#define uart (&uart_file)
и что спрятать в модуле в static, что описать как uart_lock() вместо uart.lock() -- проще не будет, пользоваться тоже проще не будет.
Зато inline-доступ к приватным полям будет быстр, а для использования такого же доступа к «приватным» переменным С-шного модуля придётся таки делать их глобальными, а не в файловой области видимости прятать.
Просто в h-файле не описывать на верхнем уровне, а прятать внутрь функций, чтобы скрыть их существование и не дать туда что-то записать минуя опубликованный интерфейс модуля:
Код: Выделить всё
static inline unsigned GetSystemTimer()
{
extern volatile unsigned sys_timer;
return sys_timer;
}