#refact player constructor