C++ include guard изглежда не работи?

Изглежда, че искате да създадете единичен glcam обект, който може да се използва на множество места. Бих направил това, като изложа безплатна функция за връщане на static екземпляр. Това е подобно на използването на extern, но намирам, че е малко по-ясно в намерението си.

#ifndef CAMERA_CLASS_HPP
#define CAMERA_CLASS_HPP


class camera_class
{
....
};

camera_class& get_camera();


#endif // CAMERA_CLASS_HPP

// in the CPP

camera_class& get_camera()
{
   static camera_class the_camera;
   return the_camera;
}

Това ви дава възможност да използвате един camera_class екземпляр, без да разчитате на extern, но в същото време не ви принуждава да го използвате като сингълтон, тъй като други области на кода също са свободни да създават свои собствени частни екземпляри.

Това може да се реализира така, както е (безплатна функция) или като static членска функция на camera_class. Избрах първото въз основа на някои отлични съвети от Скот Майерс:

Ако пишете функция, която може да бъде имплементирана или като член, или като не-приятел, не-член, трябва да предпочетете да я имплементирате като функция, която не е член.

Източник: http://www.drdobbs.com/cpp/how-non-member-functions-improve-encapsu/184401197

Предпазителят на заглавката ще предотврати множество включвания в една единица за превод. Заглавката може (и е) да бъде включена в множество единици за превод:

// a.cpp:
#include "camera.hpp"

// b.cpp
#include "camera.hpp"

Това ще създаде a.obj и b.obj, всеки от които съдържа дефиниция за glcam. Когато се свържат заедно, за да произведат крайния двоичен файл, получавате грешката с множество дефиниции.

Трябва да декларирате glcam в заглавката и да го дефинирате точно веднъж във файл .cpp:

// camera.hpp
...

extern camera_class glcam;

// camera.cpp
#include "camera.hpp"

camera_class glcam;

Основна причина:
Защитата на заглавката предотвратява включването на една и съща заглавка многократно в едно и също преводна единица, но не и в различни преводни единици. Когато включите един и същ заглавен файл в множество единици за превод,
копие на glcam наистина се създава във всяка единица за превод, където включите заглавката.
Стандартът на C++ изисква всеки символ да бъде дефинирано само веднъж (Правило за една дефиниция) и следователно линкерът ви издава грешката.