Как исправить ошибку «invalid use of incomplete type»?

Question

[toster_bodnar]

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

Мой код

#include <iostream>
#include <vector>
#include <memory>

class M;
class A
{
    public:
        A() = default;

        void func()
        {
            auto d = m.getD();
            d->func();
        }

    private:
        M m;
};

class S
{
    public:
        S() = default;
        void func(M &m)
        {
            auto d = m.getD();
            d->func();
        }
};

class D
{
    public:
        D() = default;

        void func()
        {
            std::cout << "d\n";
        }

    private:
        std::vector<std::shared_ptr<S>> s;
};

class M
{
    public:
        M() = default;
        D* getD()
        {
            return &d;
        }

    private:
        D d;
};

int main()
{
    A *a = new A();
    delete a;
    return 0;
}

Answer 1

[Александр Ананьев]

В данном случае можно так

#include <iostream>
#include <vector>
#include <memory>

class S;
class D
{
    public:
        D() = default;

        void func()
        {
            std::cout << "d\n";
        }

    private:
        std::vector<std::shared_ptr<S>> s;
};


class M
{
    public:
        M() = default;
        D* getD()
        {
            return &d;
        }

    private:
        D d;
};

class A
{
    public:
        A() = default;

        void func()
        {
            auto d = m.getD();
            d->func();
        }

    private:
        M m;
};

class S
{
    public:
        S() = default;
        void func(M &m)
        {
            auto d = m.getD();
            d->func();
        }
};


int main()
{
    A *a = new A();
    delete a;
    return 0;
}

Comments

[toster_bodnar]

Да, так отлично сработало! Спасибо! А вот если класс D тоже хочет использовать класс М, так уже не выйдет? По хорошему мне класс М во всем приложении надо таскать, во все классы. Но не хочется его делать статическим, хочется хранить использовать его только как параметр в методах других классов...

[Александр Ананьев]

toster_bodnar, разносить определение и реализацию классов по разным файлам.

[toster_bodnar]

У меня не выходило даже с вынесением, но я некоторые методы оставлял в классах определений, наверное поэтому. Спасибо большое!

[maaGames]

toster_bodnar, Добавлять указатель на объект, вместо объекта. "M* m" вместо "M m". Можно создавать указатели на неопределённый тип. При этом определение и реализация классов должна быть в разных файлах.

Answer 2

[jcmvbkbc]

как правильно выстроить определения данных классов, что бы программа компилировалась?

Перенести определения классов A и S после определения класса M, добавить объявление класса S перед определением класса D.

Comments

[toster_bodnar]

Спасибо за ответ. Вроде получилось, только потом попробовал в классе D в методе заюзать М и я так понимаю это уже никак

[jcmvbkbc]

toster_bodnar, почему никак, как. Нужно оставить в определении класса только объявление метода, а реализацию вынести из класса и поместить после определения класса M. Типа того:

class D
{
...
   void foo();
};
class M
{
...
};

void D::foo()
{
...
}

[toster_bodnar]

jcmvbkbc, Спасибо огромной, вроде бы дошло, буду пробовать