Code review exercise: Kevin Thompson - #119

Open

kevint98 wants to merge 18 commits into

MCR-Digital:masterfrom

kevint98:master

Commits on Jan 8, 2024

change add function to addPlayer
Kevin-aps
committed
dee61f5
change howManyPlayers to getPlayerCount
Kevin-aps
committed
115e31a
change roll to rollDice
Kevin-aps
committed
be16a46
change currentCategory to getCurrentCategory
Kevin-aps
committed
725ff5e
change wasWrongAnswer to wasNotCorrectlyAnswered
Kevin-aps
committed
000f47a
create nextPlayer method
Kevin-aps
committed
812f128
extract getAnswerOutcome to print outcome of answering question
Kevin-aps
committed
4d3cf6d
extract movePlaces to calculate new player location and ask question
Kevin-aps
committed
5a3e8b0
set constant to set max number of questions per category
Kevin-aps
committed
4e6dd2c
rename index variable
Kevin-aps
committed
aaeda05

Commits on Jan 9, 2024

make MAX_QUESTIONS_PER_CATEGORY private
Kevin-aps
committed
ba01637
add game board and refactor getCurrentCategory
Kevin-aps
committed
eb86bcb
extract odd roll into function isRollOdd
Kevin-aps
committed
7fd09ff
rename isRollOdd to isRollOddAndDoesOtherThings
Kevin-aps
committed
a1d7e14
create GameBoard class and extract getCurrentCategory
Kevin-aps
committed
02deb70
refactor movePlaces and create currentPlayerPlace
Kevin-aps
committed
de3db5d
refactor askQuestion
Kevin-aps
committed
7a4c647
extract question functionality into GameBoard class
Kevin-aps
committed
c9c98fc