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

Checking the player's initial script changes, numerical inputs and created a function for reseting profiles #95

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

IoannisNik
Copy link

@IoannisNik IoannisNik commented Sep 16, 2016

For the first case I added an if-case for whether the player has changed the template code in his profile's player.rb at all, before running the game, along with 2 seperate functions for compairing the template file with the player's file.

I also thought it to be a good idea to check in on the user's input when trying to access or reset a profile,or choose a difficulty level so, i added a loop on the choose method of the ui.rb file, asking for input again if the numerical input exceeds the number of options.

Additionally, I added a reset function with which the player can reset a profile he wants. I also added a -r option in the parse_options method for the user to choose which profile to reset.

@IoannisNik IoannisNik changed the title Function for reseting profiles Checking the player's initial script changes and a function for reseting profiles Sep 16, 2016
@IoannisNik IoannisNik changed the title Checking the player's initial script changes and a function for reseting profiles Checking the player's initial script changes, numerical inputs and created a function for reseting profiles Sep 16, 2016
Copy link

@0x1eef 0x1eef left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

testing

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.

2 participants