This code is not yet completed. The purpose of this post is to aid in the development in a larger project. The aim of this code is to design a generic OpenGL Shapes using polygons, quads, spheres, etc. The next posing will probably be dealing with multi-threading & or scripting. Common_objects.h -> namespace _objects{ /* --------------------------------------- ---------- Rendering Object ----------- --------------------------------------- */ class _robject{ public: void SetName( char *Name ){ name = Name; } virtual void Render(); private: char *name; // Object Name }; } Common_objects.cpp -> #include "common_objects.h" void _objects::_robject::Render(){} Common_geom.h -> #include "common_objects.h" #include <iostream> namespace _cGeom{ enum Side{ _top, _bottom, _front, _back, _left, _right }; class _color3f { public: float r, g, b; }; class _vert3f { public: _vert3f(){} // _vert3f( float x1, float y1, float z1 ) : x(x1), y(y1), z(z1) {} ~_vert3f(){}; void Set( float x1, float y1, float z1 ){x = x1; y = y1; z = z1; } float GetX() const { return x; } float GetY() const { return y; } float GetZ() const { return z; } private: float x, y, z; _color3f color; }; // template<class T> class _shape : public _objects::_robject { public: _shape(){} ~_shape(){} virtual void SetPos(){}; virtual void SetSize(){}; }; class _quad : public _shape { public: void SetPos( float x, float y, float z ); void SetSize( float sizef ) { size = sizef; } virtual void Render(); void InitQuad( Side side); // If it is part of a box, then set a side. // void Init( ); private: float size; // Size of the quad; _vert3f vertex[4]; // 4 Sets of vertexes }; class _box { public: _box( float bSize ); ~_box(); virtual void Render(); private: _quad sides[6]; }; } Common_geom.cpp -> #include <windows.h> #include <gl\gl.h> #include "common_geom.h" void _cGeom::_quad::InitQuad(_cGeom::Side side) { switch( side ) { case _top: this->vertex[0].Set( size, size,-size ); this->vertex[1].Set(-size, size,-size ); this->vertex[2].Set(-size, size, size ); this->vertex[3].Set( size, size, size ); break; case _bottom: this->vertex[0].Set( size,-size, size ); this->vertex[1].Set(-size,-size, size ); this->vertex[2].Set(-size,-size,-size ); this->vertex[3].Set( size,-size,-size ); break; case _front: this->vertex[0].Set( size, size, size ); this->vertex[1].Set(-size, size, size ); this->vertex[2].Set(-size,-size, size ); this->vertex[3].Set( size,-size, size ); break; case _back: this->vertex[0].Set( size,-size,-size ); this->vertex[1].Set(-size,-size,-size ); this->vertex[2].Set(-size, size,-size ); this->vertex[3].Set( size, size,-size ); break; case _left: this->vertex[0].Set(-size, size, size ); this->vertex[1].Set(-size, size,-size ); this->vertex[2].Set(-size,-size,-size ); this->vertex[3].Set(-size,-size, size ); break; case _right: this->vertex[0].Set( size, size,-size ); this->vertex[1].Set( size, size, size ); this->vertex[2].Set( size,-size, size ); this->vertex[3].Set( size,-size,-size ); break; } } void _cGeom::_quad::Render() { glBegin(GL_QUADS); for(int i = 0; i < 4; i++ ) { glVertex3f( this->vertex.GetX(), this->vertex.GetY(), this->vertex.GetZ() ); } glEnd(); } _cGeom::_box::_box( float bSize ) { for( int i = 0; i < 6; i++ ) { sides.SetSize( bSize ); if(i == 0)sides.InitQuad(_top); if(i == 1)sides.InitQuad(_bottom); if(i == 2)sides.InitQuad(_front); if(i == 3)sides.InitQuad(_back); if(i == 4)sides.InitQuad(_left); if(i == 5)sides.InitQuad(_right); } } _cGeom::_box::~_box() { } void _cGeom::_box::Render() { for( int i = 0; i < 6; i++ ) this->sides.Render(); } ///////////////////////////////////////// ////////////////// EoF ////////////////// ///////////////////////////////////////// Please post any suggestions or if you would like a copy of the lib.

Why the hell is every identifier name starting with an underscore?

Your "name" member variable has lousy ownership semantics (and to make things worse, it's a pointer). Also, I suspect you wanted to make _robject a structure, and give it a virtual destructor.

Your vector "set" function is useless when you have a constructor and assignment operator. Also, your "color" member is useless.

Your quad needs constructors.

Your box constructor could be written in terms of an array of enumeration values, instead of the dreaded for-switch "idiom".

Quote:Original post by ToohrVyk
Why the hell is every identifier name starting with an underscore?

It's a thing I've been seeing more and more lately. People are finally stopping to use hungarian notation, and for some reason, they're switching to something like this. I've seen a lot of C# / Jiva code where the member variables have an underscore in front of them.

As a note to the original author: why are you doing this? There is no actual reason to do so, because most IDE's will inform you about the scope of a variable. It's also a hell to type and has no added goodness.

Toolmaker

I sometimes use an underscore for member variables when I have properties of the same name, as in

Private _score as Integer

Public Property Score as Integer
Get
return _score
End Get
End Property