Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code Assignment Yannick Bijl #1

Open
wants to merge 30 commits into
base: main
Choose a base branch
from

Conversation

yannickbijl
Copy link

Implementation of the game Paper Rock Scissors

The game can be compiled/played by running the script PaperRockScissors.java
Tests were written using junit5
Written tests pass

Created test for method scoreGame
Updated pom.xml with junit5
Moved tests for scoreGame
Use Player class for user and computer in PaperRockScissors
…es extend from Player class, separating methods needed for user and computer

Move selectRandomMove method and needed support code to NPC class
While repeat asking for input until valid input is given
Valid input is determined as a value within the given string array
Make method asking user to keep playing. If not, loop will be broken.
…y loop without good reason

Rename singleGame to prsGame (paper rock scissors) as the object can now be used for multiple rounds of the game
…ed if user does not want to play any longer.
…ut player moves, so user can see what move the computer has made
… else for "tie" in incrementScoreGame explicit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant