Неправильное использование структуры неполного типа, даже с предварительным объявлением - PullRequest
11 голосов
/ 04 апреля 2011

Мне известны циклические зависимости, но даже с предварительными декларациями я получаю эту область. Что я делаю не так?

// facility.h
class Area;

class Facility {
public:
    Facility();
    Area* getAreaThisIn();
    void setAreaThisIsIn(Area* area);
private:
    Area* __area;
};

// facility.cpp
#include "facility.h"
#include "area.h"
{ ... }

// area.h
class Facility;
class Area {
public:
    Area(int ID);
    int getId();

private:
    std::list<Facility*> _facilities;
};

// area.cpp
#include "area.h"
#include "facility.h"

Так что это прекрасно компилируется, но если я сделаю

// foo.h
#include "facility.h"
class Foo { .. };

// foo.cpp
#include "foo.h"
void Foo::function() {
    Facility* f = new Facility();
    int id = f->getAreaThisIsIn()->getId();

Когда я получу invalid use of incomplete type struct Area

Ответы [ 3 ]

21 голосов
/ 05 апреля 2011

Чтобы уточнить: предварительное объявление позволяет вам работать с объектом, если оно очень ограничено:

struct Foo; // forward declaration

int bar(Foo* f); // allowed, makes sense in a header file

Foo* baz(); // allowed

Foo* f = new Foo(); // not allowed, as the compiler doesn't
                    // know how big a Foo object is
                    // and therefore can't allocate that much
                    // memory and return a pointer to it

f->quux(); // also not allowed, as the compiler doesn't know
           // what members Foo has

В некоторых случаях могут помочь предварительные объявления.Например, если функции в заголовке только когда-либо принимают указатели на объекты вместо объектов, тогда вам не нужно #include полное определение класса для этого заголовка.Это может улучшить время компиляции.Но реализация этого заголовка почти гарантированно потребует #include соответствующего определения, потому что вы, вероятно, захотите разместить эти объекты, вызвать методы для этих объектов и т. Д., И вам нужно больше, чем предварительное объявление для этого.

8 голосов
/ 04 апреля 2011

Для Facility* f = new Facility(); вам нужна полная декларация, а не просто прямая декларация.

4 голосов
/ 04 апреля 2011

Вы #include и area.h и Facility.h в foo.cpp (при условии, что это файл, где вы получаете ошибку)?

...