0

Здравствуйте. Есть файл "project.h", который включает все нужные .h:

#include "driver.h"
#include "animation.h"
#include "eventRec.h"
// etc

#include "a.h"
#include "b.h"

А также файлы "a.h" и "b.h" с одноименным классами внутри. Класс A включает объект класса B. Нужно подключить "a.h" и "b.h" с помощью "project.h" (чтобы у обоих был доступ к driver и пр.)

A.h:

#include "project.h"

class A 
{
private:
    B obj;
};

B.h:

#include "project.h"

class B
{
private:
    //something
};

При таком раскладе не компилируется - понятно - бесконечный цикл включения. Пробовал ставить include guardы в каждом файле:

#ifndef name_h
#define name_h

/* Тело класса */

#endif

При таком раскладе A не видит B.

error C2146: синтаксическая ошибка: отсутствие ";" перед идентификатором "obj"
error C4430: отсутствует спецификатор типа - предполагается int. Примечание. C++ не поддерживает int по умолчанию

Помогите достичь цели.

6
  • Чтобы A видел B, в a.h надо явно прописать #include "b.h" до class A { private: B obj; }; (надеюсь, что B у Вас явно (не по указателю) не ссылается на A :)). Что же касается ошибки с отсутствием ; и int, то, не видя всего кода, сказать что-то определенное не могу.
    – avp
    9 окт 2014 в 21:36
  • Это весь код. В main`е подключается "project.h" и всё. Под "не видит" я и подразумевал эту ошибку. 10 окт 2014 в 6:26
  • @sloow_ua, ну, тогда вместо //something в B надо написать что-то более осмысленное.
    – avp
    10 окт 2014 в 9:54
  • @avp: Упс, проглядел этот момент. Сейчас у меня там "int a" и конструктор в паблике. Но в целом - ошибка заключается в том, что компилятор не может найти нужный класс (B). 10 окт 2014 в 11:03
  • 1
    @sloow_ua, т.е. Вы написали все guards, добавили в a.h #include "b.h", и все равно та же ошибка? Тогда попробуйте все же сначала аккуратно записать весь текст ("заинклюдить вручную") в одном файле, а уже потом разбить его на несколько .h.
    – avp
    10 окт 2014 в 11:17

2 ответа 2

3

Для начала: include guard'ы необходимы, без вопросов.

Затем: ошибка состоит в том, что вы пытаетесь облегчить себе жизнь, сделав «универсальный» header и пытаясь подключить его везде. Так не пойдёт. Включайте только то, что нужно, и только там, где нужно. Например:

A.h:

#ifndef A_H_INCLUDED
#define A_H_INCLUDED

#include "b.h"

class A 
{
private:
    B obj;
};

#endif

A.cpp:

#include "A.h"
#include "project.h"

...

B.h

#ifndef B_H_INCLUDED
#define B_H_INCLUDED

class B
{
private:
    //
};

#endif

B.cpp:

#include "B.h"
#include "project.h"

...
7
  • Почему не работает таким образом? Должно вроде бы. Выходит примерно такой порядок включения, если забыть про driver и пр.: main -> project -> a -> project -> b -> project -> end. И в таком случае всё должно работать. Или я не прав? Ну и интересно, как это реализовано в сторонних библиотеках/игровых движках. В SDL либо Cocos`е один .h файл, который уже включает все остальные. 9 окт 2014 в 18:25
  • @sloow_ua: Ну, смотрите сами. Без include guard'ов понятно, что бесконечный цикл. С include guard'ами, компилируем A.cpp: 1. A.cpp начинается, включает project.h 2. project.h начинается, определяет PROJECT_H_INCLUDED 3. project.h включает A.h 4. A.h определяет A_H_INCLUDED и снова включает project.h 5. project.h не отрабатывает, т. к. определён PROJECT_H_INCLUDED 6. Продолжается компиляция A.h, не видит B _____ @sloow_ua: А в Cocos'е, небось, вместо #include используется #import.
    – VladD
    9 окт 2014 в 18:46
  • @VladD: Если в "project.h" без include guard`ов: 1. Project.h 2. Включает A.h; определяет A_H 3. A.h включает Project.h 4. Project.h включает A.h, но определен A_H, класс не переопределяется (сразу вопрос: если сам файл включается дважды, но внутри не отрабатывает из-за определенного #define, это ок?) 5. Project.h включает B.h; определяет B_H 6. B.h включает Project.h 7. Пункт 4. 8. Тоже самое, но с B.h B.h включен в A.h. 9 окт 2014 в 18:59
  • @sloow_ua: А ошибка та же?
    – VladD
    9 окт 2014 в 19:22
  • @VladD Да, такая же, как и в вопросе. Я в projectе guardы не ставил. 10 окт 2014 в 6:20
0

Есть еще один вариант - вообще не включать ничего в файл-хедер и сделать предопределение класса:

class B;

class A 
{
private:
    B obj;
};

А уже в cpp, где все будет реализовываться, подключить B. Это избавит вас от проблем с дублирующимися определениями, поскольку cpp, по идее, никуда подключаться не должен (по идее, т.к. как минимум теоретически это возможно). Но гарды все равно нужны.

1
  • @aknew: Это не откомпилируется: компилятору нужно знать, сколько памяти занимает экземпляр B, так что одной только forward-декларации недостаточно. Сработало бы, если бы вместо B obj было бы B* pObj.
    – VladD
    10 окт 2014 в 20:33

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.