Having problems with static template function in template class.

I'm getting the following errors:

v:\code\vcprograms\include\stackframe.h(90) : error C2244: 'StackFrame<Any>::push' : unable to resolve function overload
v:\code\vcprograms\include\stackframe.h(93) : error C2954: template definitions cannot nest

Here is my class. When I comment out the 2 static methods I don't have a problem. So it has something to do with those 2. What am I doing wrong? Thanks!



//**********************************************************************************************
// Definition of the StackFrame class.
//**********************************************************************************************
#include <iostream>
#include <list>      
using namespace std;

template <typename Any>
typedef list<typename Any> List;            //define list of Any type
typedef List::iterator Iter;               //define itererator type

template <typename Any>
class StackFrame
{

public: //Accessible to the world
   StackFrame();                         //Default constructor
   StackFrame(const StackFrame &pOrig);  //Copy constructor, copy obj passed in
   virtual ~StackFrame();                //Default destructor

   virtual StackFrame &operator=(const StackFrame &pOrig);

   friend std::ostream &operator<<(std::ostream &pout, const StackFrame &pSource);

   bool push(Any &pData);
   static bool push(Any &pData, StackFrame pStack);
   Any pop();
   static Any pop(StackFrame &pStack);

private:
   List mlstStack;
};



//**********************************************************************************************
// Implementation of the StackFrame class.
//**********************************************************************************************
template <typename Any>
StackFrame<Any>::StackFrame()
{
   cout << "\nConstructing StackFrame...\n" << endl;
}

template <typename Any>
StackFrame<Any>::StackFrame(const StackFrame &pOrig)
{
   cout << "\nConstructing StackFrame...\n" << endl;
}

template <typename Any>
StackFrame<Any>::~StackFrame()
{
   cout << "\nDestructing StackFrame...\n" << endl;
}

template <typename Any>
StackFrame<Any> &StackFrame<Any>::operator=(const StackFrame &pOrig)
{
   if (this == &pOrig)      //Self-assignment?
      return *this;         //Just return itself.

   //need to add code to copy List

   return *this;
}

template <typename Any>
std::ostream &operator<<(std::ostream &pout, const StackFrame<Any> &pSource)
{
   //need to change this to loop through List and print each element
   //pout << "name: " << pSource.mstrName << ", age: " << pSource.muintAge << ", sex: " << strSex;

   return pout;
}

//Push data on to this StackFrame
template <typename Any>
bool StackFrame<Any>::push(Any &pData)
{
   mlstStack.push_front(pData);
}

//Push data on to the indicated StackFrame
//static = no instantiation necessary
template <typename Any>
static bool StackFrame<Any>::push(Any &pData, StackFrame &pStack)
{
   pStack.push_front(pData);
}

//Pop data off of this StackFrame
template <typename Any>
Any StackFrame<Any>::pop()
{
   if(mlstStack.size() > 0)
   {
      return mlstStack.pop_front();
   }

   return null;
}
mflores88Asked:
Who is Participating?
 
DexstarCommented:
@mflores88:

>    static bool push(Any &pData, StackFrame pStack);
>    static Any pop(StackFrame &pStack);

Try changing the above to lines to look like this:

   static bool push(Any &pData, StackFrame<Any> pStack);
   Any pop();
   static Any pop(StackFrame<Any> &pStack);

Hope That Helps,
Dex*
0
 
DexstarCommented:
Actually, if "push" is going to modify pStack, then it should be a reference, like this:
     static bool push( Any &pData, StackFrame<Any>& pStack );

If it isn't going to be modifying pStack, then you don't need it...  Actually, in that case, then I would do it like this:
     static bool push( Any &pData, const StackFrame<Any>& pStack );
0
 
mflores88Author Commented:
Yeah, your prior comment actually got me to pay more attention to that also.

Thanks!
0
 
rstaveleyCommented:
typedef templates aren't legal
--------8<--------
template <typename Any>
typedef list<typename Any> List;           //define list of Any type
--------8<--------

You can do the following, if you want to use an uppercase 'L' :-)
--------8<--------
template<typename T,typename Alloc = std::allocator<T> > class List : public std::list<T,Alloc> {};
--------8<--------

0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.