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

Fix Logic and Data Type Errors in Rock-Paper-Scissors Game #767

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Yashjain0099
Copy link

This pull request addresses and resolves the issues identified in the Rock-Paper-Scissors game code, improving gameplay flow and accuracy.

Changes Made:

Corrected Data Type Mismatch: Replaced random.randint() with random.choice() for the computer's choice, ensuring direct selection of "rock," "paper," or "scissors" and preventing integer-string comparison errors.

Improved Game Logic: Modified the game loop to allow continuous play until a clear win or loss is determined, with proper handling of ties.

Added Input Validation: Ensured that user input is validated, prompting users to enter only valid choices ("rock," "paper," or "scissors") to prevent unintended behavior.

Refactored Code Structure: Enhanced readability with consistent indentation and streamlined conditions to make the game logic more intuitive and maintainable.

Outcome:
These changes result in smoother, error-free gameplay and provide users with a clearer and more reliable Rock-Paper-Scissors experience.

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