In C++, I have some header files such as: Base.h
, and some classes using Base.h
:
//OtherBase1.h
#include "Base.h"
class OtherBase1{
// something goes here
};
//OtherBase2.h
#include "Base.h"
class OtherBase2{
// something goes here
};
And in main.cpp
, I can only use one of those two OtherBase
classes because of duplicate header. If I want to use both classes, in OtherBase2.h
I have to #include "OtherBase1.h"
instead of #include "Base.h"
. Sometimes, I just want to use OtherBase2.h
and not OtherBase1.h
, so I think it's really weird to include OtherBase1.h
in OtherBase2.h
. What do I do to avoid this situation and what's the best practice for including header file?
You have to use header guards to avoid duplication.
http://en.wikipedia.org/wiki/Include_guard
For example in your
Base.h
add following:See this SO question for heard guard formats
#include header guard format?
To avoid having problems related to importing the same header file more than once, you can use the preprocessor to guard against that. The usual way of doing this is by adding these bits to
Base.h
:You should be using include guards in
Base.h
.An example:
This will prevent multiple-inclusion of
Base.h
, and you can use bothOtherBase
headers. TheOtherBase
headers could also use include guards.The constants themselves can also be useful for the conditional compilation of code based on the availability of the API defined in a certain header.
Alternative:
#pragma once
Note that
#pragma once
can be used to accomplish the same thing, without some of the problems associated with user-created#define
constants, e.g. name-collisions, and the minor annoyances of occasionally typing#ifdef
instead of#ifndef
, or neglecting to close the condition.#pragma once
is usually available but include guards are always available. In fact you'll often see code of the form: