The problem is this line in your main.cpp:
You got the syntax wrong. If a class constructor doesn't have parameters, you don't need brackets. If it does have parameters, then the parameters in brackets must be after the instance name, not the class name. (First is class, second is instance.) Also, the variable (instance) name shouldn't be the same as the class, because you won't be able to distinguish them. (The compiler can, apparently, because it doesn't complain if the syntax is otherwise correct, but you will cause problems for yourself if the names are the same.) Declare the instance like this:
player1 myplayer1;
myplayer1.movement(); // etc.
If you plan to re-use this class several times, then it would be better to leave out the number from the class name:
class player .... // etc. class definition
player player1; // instances
player player2;
player lotsofplayers[20];