Just scanning your code,
Charles Christie wrote:
if event.key == K_e or K_i or K_KP8: player.xmove = -3
should probably be:
if event.key == K_e or event.key == K_i or event.key == K_KP8: player.xmove = -3
etc.