Как конкретного наследника связать с конкретным родителем?

Question

[Ilia Zhitenev]

Здравствуйте. Подскажите пожалуйста.
Существует 2 класса. ComPort и производный от него класс Device.
Существует 2 объекта типа ComPort: COM1, COM2, которые содержат собственные настройки (некие параметры), и, 2 объекта типа Device: Dev1 и Dev2.
Объекты Dev1 и Dev2 используют наследуемую функцию write для записи данных в тот или иной ком-порт.

Как сделать так, чтоб Dev1 использовал параметры только объекта COM1, а Dev2 только параметры COM2? Т.е. как указать жёсткую связь - конкретный родитель - конкретный наследник? Ибо если я пытаюсь в наследнике использовать параметры родителя, то получаю их значения по умолчанию, которые задаются через конструктор класса, а мне нужно использовать заданные для родителей значения бод-рейта, номер физического порта и прочее. Спасибо.

Answer 1

[Петр]

Так вы разделите классы ComPort и Device.
И при создание обьекта Device передавайте в него обьект ComPort.
Наследование в вашей задаче не нужно.

Comments

[Ilia Zhitenev]

Спасибо. Создал в открытой части интерфейса функции доступа к настройкам ComPort. При создании Device через конструктор передаю ссылку на "родителя". Далее использую эту ссылку для получения нужных параметров через функции в открытом интерфейсе.

class ComPort {
private:
   int p1, p2;
public: 
   int getP1() {return p1;}
   ...
};

class Device {
private:
    ComPort *port;
public:
   Device(ComPort *p) {
        port = p;
   }
   action(...) {
      write(port->getP1(), ...);
   }
}

// Далее в коде:
ComPort COM1;
...
Device A(&COM1);
A.action(...);

[Adamos]

Илья Житенёв: вот только передаете вы не ссылку, а указатель...

[Ilia Zhitenev]

Adamos: с терминологией мне ещё разбираться и разбираться. Спасибо

[Adamos]

Илья Житенёв: я не про терминологию. Почему, собственно, указатель, а не ссылка?

[Петр]

Adamos: Ссылку лучше использовать когда у вас обьекты с коротким периодом жизни. А в данной задаче скорее (точно не знаю) COM порт будет использоваться не для одной операции. А в общем существенной разницы нет.

[Adamos]

Петр: вопрос, будет ли он использоваться только для одного устройства (и тогда логичнее его сделать членом класса и в нем же инициализировать) или для нескольких (тогда, может быть, стоит превратить его в синглтон).

[Петр]

Adamos: "в синглтон" )) - у меня на машине настроено 18 COM портов с различными параметрами.

[Ilia Zhitenev]

Adamos: Нет, у меня один COM порт может использоваться несколькими устройствами.

[Adamos]

Петр: Ну, ТС, как я понимаю, собирается создавать объект на каждый порт. В чем юмор?
Для разных параметров можно скрестить синглтон с фабрикой...

[Петр]

Adamos: Вы пытаетесь впихнуть модные штучки там, где они совсем не нужны. Почти нет реальных задач, где было бы обоснованно применение синглтонов. Да и обоснованное применение фабрики классов я видел только в COM библиотеках.

[Adamos]

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

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

Answer 2

[Анатолий Медведев]

Ну как то так(сферический код в вакууме):

COM1.write(Dev1.value)

или

Dev1.write(COM1, value)

это если в общем случает. Без заголовков классов ничего конкретного сказать не могу.
Вообще есть предположение о том что вы не отличаете экземпляр объекта от его описания.

Answer 3

[res2001]

А если с наследованием, то СОМ1 и СОМ2 тут явно лишние - все действия надо делать через Devы.

Answer 4

[tsarevfs]

Отношение наследования в реальном мире лучше всего описывается словом "является". Каждый ComPort является девайсом, а обратное не верно (вероятно девайсы бывают разные, иначе не понятно зачем такой класс вовсе). Значит Device должен быть базовым классом.
Далее, что касается связи двух экземпляров. Каждый экземпляр класса ComPort и так является Device благодаря наследованию. Функция write у девайса может быть виртуальной и абстрактной (=0 не имеющей своей реализации). В ComPort она будет оверрайдиться уже с конкретной реализацией для порта.
Таким образом вы можете иметь:

Device * d = new ComPort(setting1, setting2);
d.write(data);

в этом случае write будет вызван с нужными настройками (setting1, setting2).

Более того, такой подход позволит использовать разные девайсы единообразно:

std::vector<Device *> ds;
ds.push_back(new ComPort(setting1, setting2));
ds.push_back(new UsbPort(setting3, setting4));

for (auto d : ds)
   d->write(data);
</code