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 ++ требует, чтобы каждый символ мог быть определено только один раз (Одно правило определения ) и, следовательно, компоновщик выдает ошибку.