My teacher told me to remove this from the constructor
cout<<endl<<endl;
cout<<"Enter your coordinates player1: ";
cin>>x1>>y1;
get_player1_move();
cout<<"\nEnter your coordinates player2: ";
cin>>x2<<y2;
get_player2_move();
cout<<endl;
He also told me to use just one function for both & use player # as an argument to the function.