Enum has not been declared

Question

I use the following header Node.h:

/*
    INCLUDE PROTECTION
*/
#ifndef NODE_H_INCLUDED
#define NODE_H_INCLUDED

/*
    Include
*/
#include <vector>
#include <unordered_map>
//#include "NodeConnection.h"
#include "Tile.h"
#include "Network.h"

/*
    Declarations
*/
class Network;
class NodeConnection;

class Node {
    private:

        Tile* tile;

        std :: vector <NodeConnection*> connections;
        std :: unordered_map <Node*, NodeConnection*> connectionmap;

    public:

        /* Enumeration */

        enum NAVIGATION {
            RIGHT = 0, 
            UP = 1, 
            LEFT = 2, 
            DOWN = 3
        };

        Node (Tile* _tile);

        void FindNeighbours(Board* board, Network* network);

        void SetNodeConnection(Node* node, NodeConnection* nodeconnection);

};

struct TileNavigation {
    Tile* tile;
    enum Node :: NAVIGATION navigation;
};

#endif

And the following in the header of NodeConnection.h:

/*
    INCLUDE PROTECTION
*/
#ifndef NODECONNECTION_H_INCLUDED
#define NODECONNECTION_H_INCLUDED

/*
    Include
*/
#include <string>
#include "Node.h"

/*
    Declarations
*/
class Node;
struct Nodes;


enum ConnectionType {
    WALLCONN, PATHCONN, UNKNOWN
};

class NodeConnection {
    private:

        enum ConnectionType contype;

        struct Nodes;

    public:

        //NodeConnection ();
        NodeConnection (Node* a, Node* b,  NAVIGATION initial_nav);
};

struct Nodes {

    Node* left;
    Node* right;
    std :: string steps;

};

#endif

I have tried Node :: NAVIGATION and NAVIGATION, but yet it keeps telling me that

"   'NAVIGATION' has not been declared   "

Does anyone know what I am doing wrong? Thanks in advance for pointers.

Can you provide an SSCCE there is some detail missing in your sample code, using Node::NAVIGATION should work see live example. — Shafik Yaghmour
– Shafik Yaghmour, Commented Oct 11, 2013 at 22:22
NodeConnection (Node* a, Node* b, Node :: NAVIGATION initial_nav); does not work, it gives the same error — bas
– bas, Commented Oct 11, 2013 at 22:22
Yes, so can you provide an SSCCE? There is no way to tell what the problem is from the code given. — Taylor Brandstetter
– Taylor Brandstetter, Commented Oct 11, 2013 at 22:23
Node::NAVIGATION works as is, so you're not telling us some important detail. for example, is Node declared within a namespace? — Ed Swangren
– Ed Swangren, Commented Oct 11, 2013 at 22:23

billz · Accepted Answer · 2013-10-11 22:52:07Z

4

Your have circuit-include issue. Node.h and NodeConnection.h include each other.

To fix:

Forward declare Nodeconnection in Node.h, remove #include "NodeConnection.h" from Node.h to break circuit-including issue.

//#include "NodeConnection.h"  // 
class Network;
class NetworkConnection; // forward declare NetworkConnection
class Node {
//..
};

include NodeConnection.h in Node.cpp file.

#include "NodeConnection.h"

use Node::NAVIGATION in NodeConnection.h

class NodeConnection {
private:

    //enum ConnectionType contype;  don't need to use enum keyword again.
    ConnectionType contype;

public:

    //NodeConnection ();
    NodeConnection (Node* a, Node* b,  Node::NAVIGATION initial_nav);

};

edited Oct 11, 2013 at 22:52

answered Oct 11, 2013 at 22:39

billz

45.6k9 gold badges87 silver badges103 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

bas Over a year ago

It has not fixed the issue :S Thanks for the advice though

bas Over a year ago

Thanks, that does work. But now it indicates an error in invalid use of incomplete type 'class NodeConnection' ><

billz Over a year ago

#include "NodeConnection.h" in Node.cpp file

Collectives™ on Stack Overflow

Enum has not been declared

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related