Separated penalty box related tests to its o home
put some todos out of scope of refactoring, these are changing the behaviour of the original code
ask players for name of current player instead of retrieving current player (hiding the current player)
Reordered methods
Player in penalty box should win coin on correct answer
hiding accessor of current player
Renamed players to currentPlayer and duplication in method names
Removed boolean flag and created two methods to stay or get out of penalty box