Solved

Having problems with static template function in template class.

Posted on 2003-12-09
4
453 Views
Last Modified: 2008-02-01
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;
}
0
Comment
Question by:mflores88
  • 2
4 Comments
 
LVL 19

Accepted Solution

by:
Dexstar earned 125 total points
ID: 9905759
@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
 
LVL 19

Expert Comment

by:Dexstar
ID: 9905984
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
 

Author Comment

by:mflores88
ID: 9906000
Yeah, your prior comment actually got me to pay more attention to that also.

Thanks!
0
 
LVL 17

Expert Comment

by:rstaveley
ID: 9906037
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

Featured Post

Efficient way to get backups off site to Azure

This user guide provides instructions on how to deploy and configure both a StoneFly Scale Out NAS Enterprise Cloud Drive virtual machine and Veeam Cloud Connect in the Microsoft Azure Cloud.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

When writing generic code, using template meta-programming techniques, it is sometimes useful to know if a type is convertible to another type. A good example of when this might be is if you are writing diagnostic instrumentation for code to generat…
Often, when implementing a feature, you won't know how certain events should be handled at the point where they occur and you'd rather defer to the user of your function or class. For example, a XML parser will extract a tag from the source code, wh…
The viewer will learn how to use the return statement in functions in C++. The video will also teach the user how to pass data to a function and have the function return data back for further processing.
The viewer will be introduced to the technique of using vectors in C++. The video will cover how to define a vector, store values in the vector and retrieve data from the values stored in the vector.

778 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question