ошибка: ожидаемый конструктор, деструктор или преобразование типа перед символом '(' - PullRequest
Новая
       21

ошибка: ожидаемый конструктор, деструктор или преобразование типа перед символом '('

3 голосов
/ 12 апреля 2010 
include/TestBullet.h:12: error: expected constructor, destructor, or type conver
sion before '(' token

Я ненавижу сообщения об ошибках C ++ ... lol ^^

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

Вот мойTestBullet.h: 

#pragma once

#include "Bullet.h"
#include "BulletFactory.h"

class TestBullet : public Bullet {
public:
    void init(BulletData& bulletData);
    void update();
};

REGISTER_BULLET(TestBullet);  <-- line 12

И мой BulletFactory.h: 

#pragma once

#include <string>
#include <map>
#include "Bullet.h"

#define REGISTER_BULLET(NAME) BulletFactory::reg<NAME>(#NAME)
#define REGISTER_BULLET_ALT(NAME, CLASS) BulletFactory::reg<CLASS>(NAME)

template<typename T> Bullet * create() { return new T; }

struct BulletFactory {
    typedef std::map<std::string, Bullet*(*)()> bulletMapType;
    static bulletMapType map;

    static Bullet * createInstance(char* s) {
        std::string str(s);
        bulletMapType::iterator it = map.find(str);
        if(it == map.end())
            return 0;
        return it->second();
    }

    template<typename T> 
    static void reg(std::string& s) { 
        map.insert(std::make_pair(s, &create<T>));
    }
};

Заранее спасибо.

И не связано с ошибкой, но есть ли способ позволить Bullet включать BulletFactory без создания множества ошибок (из-за циклического включения)?Таким образом, я смогу удалить #include "BulletFactory.h" с вершины всех подклассов маркеров.

Ответы [ 3 ]

4 голосов
/ 12 апреля 2010

Я не думаю, что вы можете вызывать функции вне функций (если вы не используете результат для инициализации глобального).

2 голосов
/ 12 апреля 2010

Вот как вы получаете то, что хотите. (Не используя ваш код, пропускает, включая заголовки и т. Д. Просто для идеи.): 

// bullet_registry.hpp
class bullet;

struct bullet_registry
{
    typedef bullet* (*bullet_factory)(void); 

    std::map<std::string, bullet_factory> mFactories;
};

bullet_registry& get_global_registry(void);

template <typename T>
struct register_bullet
{
    register_bullet(const std::string& pName)
    {
        get_global_registry().mFactories.insert(std::make_pair(pName, create));
    }

    static bullet* create(void)
    {
        return new T();
    }
};

#define REGISTER_BULLET(x) \
        namespace \
        { \
            register_bullet _bullet_register_##x(#x); \
        }

// bullet_registry.cpp
bullet_registry& get_global_registry(void)
{
    // as long as this function is used to get
    // a global instance of the registry, it's
    // safe to use during static initialization
    static bullet_registry result;

    return result; // simple global variable with lazy initialization
}

// bullet.hpp
struct my_bullet : bullet { };

// bullet.cpp
REGISTER_BULLET(my_bullet)

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

Поскольку порядок статической инициализации не указан, мы помещаем глобальный менеджер в функцию, поэтому при первом вызове этой функции менеджер создается по требованию и используется. Это не позволяет нам использовать неинициализированный менеджер, что может быть в случае простого глобального объекта.

Бесплатно можно попросить разъяснений.

2 голосов
/ 12 апреля 2010

reg() - это функция.Вы не можете вызвать функцию без области видимости.

...