Наиболее красивый и правильный способ передачи параметров сквозь другой класс?

Question

[Pavel K]

Приветствую!
Частенько сталкиваюсь с тем, что например:
Есть класс, например CamsList, где соответственно идёт работа со списком камер (добавление,изменение, удаление и тд)
есть другой класс, например Cam - сам объект камеры.
Допустим самой камере для работы нужен общий класс для всех камер OutputBuffer (Который нафиг не нужен CamsList).

Т.е. у меня получается либо по указателю пробрасывать класс через CamsList и в дальнейшем при создании в нём новой камеры в её конструкторе указывать этот указатель, либо использовать синглтон.

Ни то, ни то не нравится. Какие есть ещё способы, как бы вы поступили?

Answer 1

[nirvimel]

#include <iostream>

class Cam;

class OutputBuffer {
public:
    OutputBuffer() { std::cout << "OutputBuffer " << this << " initialized\n"; }

    void receive(Cam &cam) { std::cout << "OutputBuffer " << this << " receiving data from camera " << &cam << "\n"; }
};

class Cam {
    static OutputBuffer buffer;
public:
    Cam() { buffer.receive(*this); }
};

OutputBuffer Cam::buffer = OutputBuffer();

int main() {
    Cam cam1, cam2;
};

OutputBuffer 0x6013d1 initialized
OutputBuffer 0x6013d1 receiving data from camera 0x7fff7742970e
OutputBuffer 0x6013d1 receiving data from camera 0x7fff7742970f

Comments

[Pavel K]

Почему-то думал, что статик зло и использовал ну максимум внутри функций... Спасибо!

[Adamos]

Pavel K: В данном случае он и есть зло. Если статик используется только в одном методе - на кой его делать членом класса? Проще объявить статик-переменную внутри самого метода. И не придется где-то как-то инициализировать этот член (частая ошибка - забывают про инициализацию или дублируют ее).

[Pavel K]

Adamos: реализовал и столкнулся с проблемой, что придётся в других классах так же делать, что бы иметь доступ к OutputBuffer. Как всё таки тогда красивее сделать?

[nirvimel]

Pavel K: Тогда остается только синглтон.

Answer 2

[xmoonlight]

Указание любой переменной родительского класса через static. Это используется и в C/C# и Unity3D, и вообще повсеместно.

Comments

[Pavel K]

Почему-то думал, что статик зло и использовал ну максимум внутри функций... Спасибо!

Answer 3

[abcd0x00]

Допустим самой камере для работы нужен общий класс для всех камер OutputBuffer

То, что он общий, ничего не значит. Работать с ним надо так, будто он индивидуальный для каждой камеры.
Передаёшь его каждой камере при её инициализации. А уже готовая камера (которая знает, куда выводить) добавляется в список камер.

Comments

[Pavel K]

Но тогда придётся ссылкой тащить через другой класс. Можно это как-либо красивее сделать?

[abcd0x00]

Pavel K:
Вот так это будет выглядеть

CamsList list;
Output output;

list.add(new Cam(output));
list.add(new Cam(output));
list.add(new Cam(output));

[abcd0x00]

Pavel K:
Вот пример

Код

#include <iostream>

using namespace std;

class Cam {
    int n;
public:
    Cam(int x) { n = x; }
    void print() { cout << n << endl; }
};

void func(Cam *c)
{
    c->print();
    delete c;
}

int main()
{
    func(new Cam(1));
    func(new Cam(2));
    func(new Cam(3));
    return 0;
}

Вывод

[guest@localhost cpp]$ .iso++ t.cpp -o t
[guest@localhost cpp]$ ./t
1
2
3
[guest@localhost cpp]$