CODEWITHSUNDEEP
c++templates
yotamoo
yotamoo

Reputation: 5452

failing to invoke template class, c++

I've defined a template class like so (providing .hpp file):

#ifndef PERSOANLVEC_H_
#define PERSOANLVEC_H_

#include <vector>

using namespace std;

template<class T, class PrnT> class PersoanlVec {
public:
    PersoanlVec();
    ~PersoanlVec();
    void push_back(T t);
    void erase(int index);
    PersoanlVec& operator[](int index);
    const PersoanlVec& operator[](int index) const;
    void print() const;
    size_t size();
private:
    vector<T> _vector;
};

#endif /* PERSOANLVEC_H_ */

Now, everything compiles ok with this class. When I try to use it I get undefined reference to PersoanlVec<Person, Person>::PersoanlVec()'. Here's where I call it:

#include "Person.h"
#include "PersoanlVec.hpp"
#include <cstdlib>

int main(void)
{
    Person p1("yotam");
    Person p2("yaara");
    PersoanlVec<Person,Person> *a = new PersoanlVec<Person,Person>(); //<---ERROR HERE
    return EXIT_SUCCESS;
}

This is my first try with templates, its not very clear for me obviously. I DO have a constructor with no parameters, Any ideas? Thanks!

Upvotes: 1

Views: 75

Answers (3)

John Humphreys
John Humphreys

Reputation: 39314

You need to have all of your template function definitions held in the header file rather than the CPP file - this is basically because the template definition will be used multiple times to create multiple types depending on what parameters you pass in to it as type parameters around your code. The only template related functions that should ever be defined in the CPP file are template specialization functions - those where you want to explicitly say (if user passed in type A and B then do this specifically instead of the default action).

Upvotes: 0

Bogdan Matican
Bogdan Matican

Reputation: 29

I am pretty sure you just forgot to add the file to the compilation process. Be careful with your misspelling, since that can cause generic pain.

Whenever you have different compilation units (classes for example, each with their .h/.cpp), your classes need to know of the interfaces, reason for which you normally include the header files, yet the compiler also needs to know the implementations so that it can bind together your binary file.

As such, you will need to call the compiler passing all the .cpp files in your project to it, otherwise it will fail letting you know you are referencing unimplemented pieces.

Upvotes: 0

Xeo
Xeo

Reputation: 131839

Do you have the content of your constructor and functions in a .cpp file? If yes, there's your problem. Put them in the header file, possible just inline in the class itself:

template<class T, class PrnT> class PersoanlVec {
public:
    PersoanlVec(){
      // code here...
    }

    ~PersoanlVec(){
      // code here...
    }

    void push_back(T t){
      // code here...
    }

    void erase(int index){
      // code here...
    }

    PersoanlVec& operator[](int index){
      // code here...
    }

    const PersoanlVec& operator[](int index) const{
      // code here...
    }

    void print() const{
      // code here...
    }

    size_t size(){
      // code here...
    }

private:
    vector<T> _vector;
};

For the reason, have a look here.

Upvotes: 4

Related Questions