Class forward declaration

[Kevin Grigorenko]

Hello,

I have the definitions of classes A and B in a header file. Class A has a
private member of type B. Class B is defined after class A in the header
file. I place a forward declaration of class B above class A but VS.NET
still complains with:

error C2079: 'TextDB::TextDB ::database_vers ion' uses undefined class
'TextDB::TextDB _Version'

namespace TextDB
{
class TextDB_Version;

class TextDB
{
private:
TextDB_Version database_versio n;
};

class TextDB_Version
{
[...]
};

} // namespace TextDB

Is this because this is in a header file? I'm confused.

Thanks a lot,
Kevin Grigorenko

[Cy Edmunds]

"Kevin Grigorenko" <kz****@psu.edu > wrote in message
news:bk******** **@f04n12.cac.p su.edu...

Hello,

I have the definitions of classes A and B in a header file. Class A has a
private member of type B. Class B is defined after class A in the header
file. I place a forward declaration of class B above class A but VS.NET
still complains with:

error C2079: 'TextDB::TextDB ::database_vers ion' uses undefined class
'TextDB::TextDB _Version'

namespace TextDB
{
class TextDB_Version;

class TextDB
{
private:
TextDB_Version database_versio n;
How much space should the compiler allocate for this class? It can't know at
this point. It would work to say

TextDB_Version *pdatabase_vers ion;

because all pointers have the same size.
};

class TextDB_Version
{
[...]
};

} // namespace TextDB

Is this because this is in a header file? I'm confused.

Thanks a lot,
Kevin Grigorenko

--
Cy
http://home.rochester.rr.com/cyhome/

[Phlip]

> I have the definitions of classes A and B in a header file. Class A has a

private member of type B. Class B is defined after class A in the header
file. I place a forward declaration of class B above class A but VS.NET
still complains with:

error C2079: 'TextDB::TextDB ::database_vers ion' uses undefined class
'TextDB::TextDB _Version'

namespace TextDB
{
class TextDB_Version;

class TextDB
{
private:
TextDB_Version database_versio n;
};

class TextDB_Version
{
[...]
};

} // namespace TextDB

Is this because this is in a header file? I'm confused.
C++ obeys parsing rules that behave as-if the compiler were only allowed to
"know" things that have been defined "above" where they are used.

(Also, C++'s core language parses after the preprocessor commands, so the
location of a declaration "in a header file" is irrelevant.)

As a rubric, think of the definition of a class (the {} part) as declaring a
classes "size", and think of the definition only working if the compiler
knows the size of each component object.

So, at this line...
class TextDB
{
private:
TextDB_Version database_versio n;
};

The compiler is trying to determine the size of TextDB. But it does not yet
know the size of TextDB_Version. So it can't define TextDB in terms of
TextDB_Version, which is not defined yet.

The solution is to put TextDB_Version above TextDB. C++'s object
construction rules take precedence - slightly - over any grandiose ideals of
perfectly encapsulated objects.

--
Phlip
http://www.c2.com/cgi/wiki?TestFirstUserInterfaces

[Jonathan Mcdougall]

>I have the definitions of classes A and B in a header file. Class A has a

private member of type B. Class B is defined after class A in the header
file. I place a forward declaration of class B above class A but VS.NET
still complains with:
For the compiler to know the exact size of a class, it must know the
exact size of its members. By putting only a declaration, the
compiler does not know the size of the class, hence the error.
error C2079: 'TextDB::TextDB ::database_vers ion' uses undefined class
'TextDB::TextD B_Version'

namespace TextDB
{
class TextDB_Version;

class TextDB
{
private:
TextDB_Version database_versio n;
};

class TextDB_Version
{
[...]
};

} // namespace TextDB

Is this because this is in a header file? I'm confused.

Take a look :

class A
{
int i;
};

Here the compiler knows 'A' will have the size
sizeof(int) + implementation_ defined_things

In your class :

class TextDB
{
private:
TextDB_Version database_versio n;
};

the compiler tries the same thing :
sizeof(TextDB_V ersion) + implementation_ defined_things

and just cannot find the size of TextDB_Version since you only
declared it.

You have got two choices :

1) Define the class TextDB_Version *before* TextDB

2) Make database_versio n a pointer and allocate the
memory dynamically
Jonathan

[Kevin Grigorenko]

"Jonathan Mcdougall" <DE************ ******@yahoo.ca > wrote in message
news:7m******** *************** *********@4ax.c om...

I have the definitions of classes A and B in a header file. Class A has aprivate member of type B. Class B is defined after class A in the header
file. I place a forward declaration of class B above class A but VS.NET
still complains with:

For the compiler to know the exact size of a class, it must know the
exact size of its members. By putting only a declaration, the
compiler does not know the size of the class, hence the error.

error C2079: 'TextDB::TextDB ::database_vers ion' uses undefined class
'TextDB::TextD B_Version'

namespace TextDB
{
class TextDB_Version;

class TextDB
{
private:
TextDB_Version database_versio n;
};

class TextDB_Version
{
[...]
};

} // namespace TextDB

Is this because this is in a header file? I'm confused.

Take a look :

class A
{
int i;
};

Here the compiler knows 'A' will have the size
sizeof(int) + implementation_ defined_things

In your class :

class TextDB
{
private:
TextDB_Version database_versio n;
};

the compiler tries the same thing :
sizeof(TextDB_V ersion) + implementation_ defined_things

and just cannot find the size of TextDB_Version since you only
declared it.

You have got two choices :

1) Define the class TextDB_Version *before* TextDB

2) Make database_versio n a pointer and allocate the
memory dynamically
Jonathan

Thanks a lot, that cleared things up clearly.

Kevin Grigorenko

[jeffc]

"Kevin Grigorenko" <kz****@psu.edu > wrote in message
news:bk******** ***@f04n12.cac. psu.edu...

Thanks a lot, that cleared things up clearly.

As opposed to clearing unclearly? :-)

[Kevin Grigorenko]

"jeffc" <no****@nowhere .com> wrote in message
news:3f******** @news1.prserv.n et...

"Kevin Grigorenko" <kz****@psu.edu > wrote in message
news:bk******** ***@f04n12.cac. psu.edu...

Thanks a lot, that cleared things up clearly.

As opposed to clearing unclearly? :-)

Haha, touché!