Enum type not declared

May 16, 2015 at 8:08am
hyperfine (155)
Hi, could someone help me figure out where the bug is in this code snippet? Compiler error is: 'A::type has not been declared". Below are the four files that should be relevant; the weird thing is that I have another example in my code that does almost the exact same thing but compiles just fine. The only difference between that example and this one is that the file dependencies are different; A does not need to call one of B's methods in the other example but it does in this one. Not sure if that's the reason.

Anyways, the code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
#ifndef A_H
#define A_H


//classA.h
#include "classB.h"
class A
{
public:
   enum type{type1, type2};
   type TYPE = type1;

   B* b;
   
   void callBfunc();
}

#endif //A_H



1
2
3
4
5
6
7
8
 

//classA.cpp
#include "classA.h"
void A::callBfunc()
{
   b->func(TYPE);
   return;
}


1
2
3
4
5
6
7
8
9
10
11
12
13
 
#ifndef B_H
#define B_H

//classB.h
#include "classA.h"

class B
{
public:
   void func(A::type TYPE);
}

#endif //B_H 


1
2
3
4
5
6
7
8
 
//classB.cpp
#include "classB.h"
void B::func(A::type TYPE)
{
   //do something
   return;
}




Last edited on May 16, 2015 at 6:47pm
May 16, 2015 at 8:40am
Codermik (835)

By default if you do not specify Public, Private or Protected in a class it will default to Private.
May 16, 2015 at 8:44am
hyperfine (155)
Thank you for your reply. It still does not compile as public.
May 16, 2015 at 10:15am
Codermik (835)

Try this...

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
//classA.h
#include "classB.h"
enum type{ type1, type2 };
class A
{
private:
	B* b;
	int TYPE = type1;
public:	
	void callBfunc();
};

//classA.cpp
#include "classA.h"
void A::callBfunc()
{
	b->func(TYPE);
	return;
}

//classB.h
class B
{
public:
	void func(int);
}

//classB.cpp
#include "classB.h"
void B::func(int type)
{
	//do something
	return;
}
May 16, 2015 at 4:15pm
hyperfine (155)
I'll give that a shot. I'm still really curious why my way doesn't work, though.
May 16, 2015 at 5:18pm
AbstractionAnon (6954)
You have recursive headers and no include guards.
a.h includes b.h. b.h includes a.h.

May 16, 2015 at 6:39pm
hyperfine (155)
You have recursive headers and no include guards.
a.h includes b.h. b.h includes a.h.


Sorry, I actually do have include guards but the program still doesn't compile.
Last edited on May 16, 2015 at 6:40pm
May 16, 2015 at 7:23pm
S G H (2638)
Actually, include guards are the issue.
In classA.h, remove your include line, swap it with a "class B;"
Include classB.h from classA.cpp.
May 16, 2015 at 7:30pm
AbstractionAnon (6954)
Include guards by themselves don't solve the problem of recursive includes.
They simply prevent the compiler from looping forever reading header files

In a.h, remove line 6 (the include of b.h). In its place, put the following: 

 
 
class B;  // forward declaration


Now, b.h can include a.h without concern for recursive includes.

Topic archived. No new replies allowed.