The following diagram presents a diamond class hierarchy:
As depicted, diamond inheritance denotes when two classes (e.g., CDerived1 and CDerived2), separately extending a common base class (e.g., CBase), are sub classed simultaneously by a fourth class (e.g., CTest).
The following is how one would expect the diagram to be implemented:
#include <stdio.h>
class CBase
{
public:
virtual void test()
{
printf("CBase::test()\r\n");
}
};
class CDerived1 : public CBase
{
};
class CDerived2 : public CBase
{
};
class CTest : public CDerived1, public CDerived2
{
};
int main()
{
CTest test;
test.test();
return 0;
}
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:
Select allOpen in new window
However, the Microsoft VC+ 2005 .NET compiler gives the following message:
1>------ Build started: Project: vft_test, Configuration: Debug Win32 ------
1>Compiling...
1>main_romb.cpp
1>c:\source\vft_test\vft_test\main_romb.cpp(27) : error C2385: ambiguous access of 'test'
1> could be the 'test' in base 'CBase'
1> or could be the 'test' in base 'CBase'
1>c:\source\vft_test\vft_test\main_romb.cpp(27) : error C3861: 'test': identifier not found
1>Build log was saved at "file://c:\Source\vft_test\vft_test\Debug\BuildLog.htm"
1>vft_test - 2 error(s), 0 warning(s)
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
Select allOpen in new window
Since CDerived1 and CDerived2 can have very different implementations of CBase::test(), the compiler is unsure which CTest derives. This is the so called "Diamond problem" or even "Dreaded Diamond".
How to deal with this "dread"?
The following code will give you a clue:
#include <stdio.h>
class CBase
{
public:
CBase()
{
printf("CBase::CBase()");
}
virtual void test()
{
printf("CBase::test()\r\n");
}
};
class CDerived1 : public CBase
{
public:
CDerived1()
{
printf("CDerived1::CDerived1()");
}
};
class CDerived2 : public CBase
{
public:
CDerived2()
{
printf("CDerived1::CDerived1()");
}
};
class CTest : public CDerived1, public CDerived2
{
public:
CTest()
{
printf("CTest::CTest()");
}
};
int main()
{
CTest test;
return 0;
}
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:
35:
36:
37:
38:
39:
40:
41:
42:
43:
44:
45:
46:
47:
48:
49:
50:
51:
52:
Select allOpen in new window
This program is compiled successfully and works:
Of course, you know how CTest object is created. It derives, firstly, from CDerived1 object, that inherits from CBase, and, secondly, CTest derives from CDerived2 that also inherits from CBase. So the compiler (probably any compiler) creates CBase object, then CDerived1, and now it needs to create CBase again, ..., etc., we got 2 CBase objects!
The diagram did not show that the CBase class appears twice in the class hierarchy.
C++ provides a technique for such case. The declaration of the CBase class as virtual solves the situation:
#include <stdio.h>
class CBase
{
public:
CBase()
{
printf("CBase::CBase()\r\n");
}
virtual void test()
{
printf("CBase::test()\r\n");
}
};
class CDerived1 : virtual public CBase
{
public:
CDerived1()
{
printf("CDerived1::CDerived1()\r\n");
}
};
class CDerived2 : virtual public CBase
{
public:
CDerived2()
{
printf("CDerived2::CDerived1()\r\n");
}
};
class CTest : public CDerived1, public CDerived2
{
public:
CTest()
{
printf("CTest::CTest()\r\n");
}
};
int main()
{
CTest test;
test.test();
return 0;
}
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:
35:
36:
37:
38:
39:
40:
41:
42:
43:
44:
45:
46:
47:
48:
49:
50:
Select allOpen in new window
The application output is shown on the following screen shot:
In this example CBase is inherited virtually. CDerived1 and CDerived2 classes share the same implementation of CBase. The compiler creates only one CBase object.
Of course, it is possible to solve this "Diamond problem" in another way. For example, you can explicitly specify which of the super class implementations of test() is desired:
#include <stdio.h>
class CBase
{
public:
virtual void test()
{
printf("CBase::test()\r\n");
}
};
class CDerived1 : public CBase
{
};
class CDerived2 : public CBase
{
};
class CTest : public CDerived1, public CDerived2
{
};
int main()
{
CTest test;
test.CDerived1::test();
return 0;
}
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:
Select allOpen in new window
I think that the best solution is to try to avoid such diamond problems. That will keep the design simple.
The following source code shows how we can use aggregation instead of the diamond inheritance with our same classes:
#include <stdio.h>
class CBase
{
public:
virtual void test()
{
printf("CBase::test()\r\n");
}
};
class CDerived1 : public CBase
{
};
class CDerived2 : public CBase
{
};
class CTest
{
CDerived1 m_One;
CDerived2 m_Two;
public:
void test()
{
m_One.test();
m_Two.test();
//or just
//m_One.test();
}
};
int main()
{
CTest test;
test.test();
return 0;
}
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:
35:
36:
37:
38:
39:
Select allOpen in new window
by: dimadon on 2009-07-28 at 00:23:45ID: 2355
Hello , and thank you,
now I understand. It solves a problem in my current project.