use attr_reader calls instead of instance variables
inherit from player
refactoring