templatized class inside templatized class

[-EquinoX-]

what is wrong with the implementation of the add function for the vector class here?

namespace project {
template <class _T>
class type
{
 public :
 	type();
 	type(int num);

 	type( const type & typ);
 	~type();

 protected :
 	_T val;

};

template <class type, size_t n>
class vector
{
  public:

  // constructors/destructor
    vector( );                        // default constructor (size==0)
   
    vector( const vector & vec );     // copy constructor
    ~vector( );                       // destructor

  // operator overloads
    const vector & operator = ( const vector<type, n>& vec );

  // add element to a vector
    void add(type value);
 
    vector<type, n>& operator+=(const vector<type, n>& vec);
    vector<type, n>& operator-=(const vector<type, n>& vec);

  // unary
    vector<type, n>& operator-();	
	
  // member functions
    int  length( ) const;                  // capacity of vector

  //read access operator for a polynomial
    const type& operator[]( int index ) const;
	
  //write access operator for a polynomial
    type& operator[]( int index );

 protected:    
    int csize;
    int  size;                             // # elements in array
    type * list;                          // array used for storage
    
};

template <class type, size_t n>
void vector<type, n>::add(type value)
{
 this.list[this.csize] = value;
 this.csize++;
}
}

[m37h0d]

you have to put the method implementation inside the class declaration.

template <class _T>
class type
{
 public :
 	type();
 	type(int num);

 	type( const type & typ);
 	~type();

 protected :
 	_T val;

};

template <class type, size_t n>
class vector
{
  public:

  // constructors/destructor
    vector( );                        // default constructor (size==0)
   
    vector( const vector & vec );     // copy constructor
    ~vector( );                       // destructor

  // operator overloads
    const vector & operator = ( const vector<type, n>& vec );

  // add element to a vector
    void vector::add(type value)
    {
     this.list[this.csize] = value;
     this.csize++;
    }

 
    vector<type, n>& operator+=(const vector<type, n>& vec);
    vector<type, n>& operator-=(const vector<type, n>& vec);

  // unary
    vector<type, n>& operator-();	
	
  // member functions
    int  length( ) const;                  // capacity of vector

  //read access operator for a polynomial
    const type& operator[]( int index ) const;
	
  //write access operator for a polynomial
    type& operator[]( int index );

 protected:    
    int csize;
    int  size;                             // # elements in array
    type * list;                          // array used for storage
    
};

also, what is the purpose of n and the class "type"? you are probably going down the wrong road with those.

[-EquinoX-]

n is the vector size which is implemented as an array

the requirements is:

•Your template class should be called vector. Note that the template parameters for your vector
class will necessarily be different from those of the std::vector class.
• Your vectors should have elements of type, with fixed length n.
• Elements of type must support the full set of operators provided below‡. Elements must be able
to take as their optional constructor argument an int.

I don't really think that the implementation must be inside the class

I got the following error when trying to use the add:
vector_main.cpp:13: error: no matching function for call to âece373::vector<int,u>::add(ece373::type<int>&)â
vector.h:171: note: candidates are: void ece373::vector<type, n>::add(type) [with type = int, unsigned int n = 3u]

[laserlight]

what is wrong with the implementation of the add function for the vector class here?

You forgot that this is a pointer.

[-EquinoX-]

so how can I fix this?

[laserlight]

You can either remove the this. or change them to this->.

[m37h0d]

I don't really think that the implementation must be inside the class

d'oh of course. that is correct. you just can't put it in a separate file.

[-EquinoX-]

You can either remove the this. or change them to this->.

I even removed that and it still gives me the error... I think the problem is in the function signature

[laserlight]

I even removed that and it still gives me the error... I think the problem is in the function signature

Then post the smallest and simplest program that demonstrates the error.

[-EquinoX-]

here's my test:

using namespace project;

int main(){
 type<int> e(1);
 vector<int, 3> a;
 a.add(e);
}

[laserlight]

Notice that vector is a vector<int, 3>, not a vector<type<int>, 3>.

[-EquinoX-]

ok now I got a different error:

vector_main.cpp.text+0xb0): undefined reference to `project::type<int>::type(project::type<int> const&)'
CMakeFiles/vector.dir/vector_main.cpp.o: In function `project::vector<project::type<int>, 3u>::vector()':
vector_main.cpp.text._ZN6project6vectorINS_4typeIiEELj3EEC1Ev[project::vector<project::type<int>, 3u>::vector()]+0x5a): undefined reference to `project::type<int>::type()'


FYI: I encapsulate the whole .h file above into a namespace called project, see my edited code above

[m37h0d]

reading your assignment, it is my interpretation that your professor did not intend for you to implement a separate class called type.

[laserlight]

You declared but did not define the copy constructor for project::type. You did not define the default constructor for project::type, but it is used by project::vector's default constructor.

[-EquinoX-]

I've attached my code for you to see, I have the constructor and everything. I have the constructor and everything, and yes I've confirmed that I need to create a class called type as I need to create a templatized class of vector that stores a type