refacto move player movement data to player class