Блог

У меня есть класс imgmanager, который позволяет мне загружать все мои изображения ровно один раз, это довольно приятно, и во время создания прототипа у меня были все мои файлы в одном месте, поэтому мне не нужно было беспокоиться о циклических определениях. Однако после разделения всех моих классов у меня возникла проблема.

Мой заголовочный файл

 #ifndef IMAGEMANAGER_H   
#define IMAGEMANAGER_H
#include "Img.h"
#include <vector>
#include <map>
#include <string>

class imgmanager{
 protected:
 std::vector<sf::Image*> images;
 std::map<std::string,int> positions;
 public:
 sf::Image* addimg(std::string path); //relative to resources
 sf::Image* getimg(std::string path);
 int size();
 virtual ~imgmanager();
 sf::Image* operator[](int);
}imagemgr;

#endif
  

С экземпляром, созданным после } и перед ; мой компилятор жалуется на меня:

Итак, я спрашиваю: что я должен сделать, чтобы иметь глобальный экземпляр моего класса imagemgr? Должен ли я просто создать глобальный заголовочный файл и создать экземпляр? (в этом конкретном случае я могу просто создать глобальную переменную в моем main.cpp ни для одного из заголовков не требуется экземпляр)

Не создавайте экземпляры объектов в заголовках.

Создайте экземпляр вашего объекта в одном исходном файле.

Если вам нужно получить к нему доступ через несколько единиц перевода, поместите это в свой заголовок:

 extern imgmanager imagemgr; // declaration
  

Это сообщит всему коду, который может «видеть» заголовок, что существует так называемый объект; но он все равно будет фактически определен только в одном исходном файле, где вы написали:

 imgmanager imagemgr; // definition
  

(Это аналогично способу, которым вы объявляете функции в заголовке, но определяете их только в одном исходном файле:

 void f(); // declaration
void f() { ... } // definition
  

)

Приведенный выше общий совет, который я послушно передал, теперь я бы поставил под сомнение целесообразность наличия класса вообще, если вы собираетесь использовать только один его глобальный экземпляр. Либо сделайте его «одноэлементным» классом, либо используйте вместо этого свободные функции в пространстве имен.

Если вам требуется один глобальный экземпляр, я предлагаю вам сделать ImageManager «одноэлементным». Я не уверен, что делать для сложных типов, но для глобального объявления простого типа данных вы должны объявить переменную «extern» в заголовке и создать ее экземпляр ровно в одном модуле (cpp-файле).