To co znajduje się w .cpp nie ma całkowicie żadnego sensu. Z tego co rozumiem powinno znaleźć się tam ciało konstruktora a tworzysz coś bardzo dziwnego. Polecam zrobić coś takiego.
opponent::opponent(string name,int life, int damage, int deffence, int money, int exp)
{
// i tutaj piszesz co ma się dziać po stworzeniu obiektu np.
this->name=name;
}