Error "not declared in the scope" but with interfaces

Question

You see I've been trying to create a std::vector which contains the Entity class inside the IState class. Both classes are interfaces.

The error is

'Entity' was not declared in this scope

and it points to :

protected:
    std::vector <Entity*> ent_map;

inside IState.h

I've been trying for hours now to solve it. Once I made a forward declaration inside IState.h but once I did and tried to use the vector it spews out that it's an incomplete class, so I was back to square one.

Any ideas?

Entity.h

#ifdef __ENTITY__
#define __ENTITY__

#include <iostream>
#include <SDL.h>

class Entity
{
    public:
    virtual ~Entity();
    virtual void load(const char* fileName, std::string id, SDL_Renderer* pRenderer) = 0;
    virtual void draw() = 0;
    virtual void update() = 0 ;
    virtual void clean() = 0;

    /*void int getX() { return m_x;}
    void int getY() { return m_y;}
    void std::string getTexID {return textureID;}
    */


};


#endif // __ENTITY__

IState.h

#ifndef IState_
#define IState_

#include "Entity.h"
#include <vector>

class IState
{
    public :
    virtual ~IState();
    virtual void update() = 0;
    virtual void render(SDL_Renderer* renderTarget) = 0;
    virtual bool onEnter() = 0;
    virtual bool onExit() = 0;
    virtual void handleEvents(bool* gameLoop,SDL_Event event) = 0;
    virtual void resume() = 0;
    virtual std::string getStateID() = 0;
    virtual void setStateID(std::string id) = 0;

    protected:
        std::vector <Entity*> ent_map;

};


#endif // IState_

See [this](http://stackoverflow.com/questions/228783/what-are-the-rules-about-using-an-underscore-in-a-c-identifier). You need to change `__ENTITY__` to something else. — juanchopanza, Jun 09 '16 at 10:59

songyuanyao · Accepted Answer · 2016-06-09T11:16:51.603

11

The content of "Entity.h" won't be included at all.

Change

#ifdef __ENTITY__

to

#ifndef __ENTITY__

BTW: The name contains double underscore or begins with an underscore followed by an uppercase letter is reserved in C++, you need to be careful about it.

edited Jun 09 '16 at 11:16

answered Jun 09 '16 at 10:31

songyuanyao

169,198
16
310
405

1

That is a reserved name. – juanchopanza Jun 09 '16 at 10:33
@juanchopanza - In fairness, OP used the identifier first. – Karl Nicoll Jun 09 '16 at 10:35
@KarlNicoll Yes, but it is something else that needs to be fixed. – juanchopanza Jun 09 '16 at 10:43
@juanchopanza - Absolutely agree :-) – Karl Nicoll Jun 09 '16 at 10:44
@juanchopanza BTW, would it be Okay for macros? – songyuanyao Jun 09 '16 at 11:18
@songyuanyao No, this one is reserved in all scopes. – juanchopanza Jun 09 '16 at 11:56

Error "not declared in the scope" but with interfaces

1 Answers1