Thread: Template sub-classing problem

Forst of all here is my code

Code:

#include <list>
#include <string>

typedef std::string  String;

template<class DT>
class State{
    public:
        class Transition;
    public:
        State();
        State(const DT name);
        
        void setName(const DT name);
        const DT getName() const;
        
        bool addTransition(State<DT>::Transition);
        bool removeTransition(State<DT>::Transition);
    private:
        DT                            name;
//		static Transition __useless__("USELESS");
		std::list< State<DT>::Transition >	  mmap;
/*
	 *
	 *Sub-class transition
	 *
	 */
	public:
	class Transition{
		public:
			Transition(const char *);
        
			void setLabel(const char *label);
			const char *getLabel() const;
        
			void setTarget(const State<DT> *state);
			const State<DT> *getTarget() const;
        
			bool operator==(State<DT>::Transition);
			bool operator<(State<DT>::Transition);
		private:
			String       label;
			State<DT>    *next;
	};
};

#endif

Now when I instanciate a variable of
State<unsigned int> st;

Visual C gives me error

Code:

use of undefined type 'State<unsigned int>::Transition'; 
at line std::list< State<DT>::Transition >	  mmap;

what to do? Do any body knows how to use subclass instance?

you're using Transition before you define it. forward declarations can only be used for pointers or references. move the definition of Transition to where you forward declare it and it should work.

dunno why you're forward declaring it there anyway...

you do realise that you'll need to have the method implementations in the header for a template?

at the point where the list template is instantiated, Transition has not yet been defined. you can move the list declaration below Transition's definition and it should work fine.

>> forward declarations can only be used for pointers or references

are we talking about classes here or variables?

>> you'll need to have the method implementations in the header for a template?

you don't have to, it's just easier that way.

Code:

#include <list>
#include <string>

typedef std::string  String;

template<class DT>
class State{
    public:
/*
 *
 *Sub-class transition
 *
 */
public:
class Transition{
public:
Transition(const char *);
        
void setLabel(const char *label);
const char *getLabel() const;
        
void setTarget(const State<DT> *state);
const State<DT> *getTarget() const;
        
bool operator==(State<DT>::Transition);
bool operator<(State<DT>::Transition);
private:
String       label;
State<DT>    *next;
};  
    public:
        State();
        State(const DT name);
        
        void setName(const DT name);
        const DT getName() const;
        
        bool addTransition(State<DT>::Transition);
        bool removeTransition(State<DT>::Transition);
    private:
        DT                            name;
//static Transition __useless__("USELESS");
std::list< State<DT>::Transition >  mmap;

};

template<class DT>
State<DT>::State(){
	return;
}

template<class DT>
State<DT>::Transition::Transition(const char* nm){
	label=nm;
}

void main(){
	State<int> st();
	State<int>::Transition tr("a");
	return ;
}

here is portion of my code again now it says me

C:\Documents and Settings\1860\Desktop\test.cpp(25) : error C2027: use of undefined type 'State<int>'
C:\Documents and Settings\1860\Desktop\test.cpp(59) : see reference to class template instantiation 'State<int>' being compiled
C:\Documents and Settings\1860\Desktop\test.cpp(26) : error C2027: use of undefined type 'State<int>'
C:\Documents and Settings\1860\Desktop\test.cpp(59) : see reference to class template instantiation 'State<int>' being compiled
C:\Documents and Settings\1860\Desktop\test.cpp(43) : error C2027: use of undefined type 'State<int>'
C:\Documents and Settings\1860\Desktop\test.cpp(59) : see reference to class template instantiation 'State<int>' being compiled
Error executing cl.exe.

it is giving me error on
bool operator==(State<DT>::Transition);
inside Transition

what compiler are you using? - it compiles fine for me (other than the fact that only some of the methods are defined). main should return an int, by the way.