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

Interactive Text-Based Adventure Game(Issue no. 24) #79

Merged
merged 3 commits into from
Oct 24, 2024

Conversation

debojit11
Copy link
Contributor

Developer Checklist

  • [✔️ ] Adhered to the guidelines outlined in the README & Contributing file.
  • [ ✔️] Maintained the correct directory structure (e.g., ProjectName/...yourfiles).
  • [ ✔️] Please ensure to include a README file for enhanced project clarity.
  • [ ✔️] Starred ⭐ the repository (optional).

Summary

_This is a simple text-based adventure game where players explore rooms, collect items, and engage in combat with various opponents. Players can choose different paths, fight enemies like goblins and dragons, and manage an inventory of items such as swords and shields. The game features random encounters and decision-making, creating a unique experience each time it is played. This is the PR for the issue #24 _

Screenshot

Attach any relevant screenshots or GIFs here.

Live Project Link

Include a link to the live project here.

Copy link
Contributor

👋 @debojit11 👋

We're delighted to have your pull request! Please take a moment to check our contributing guidelines and ensure you've filled out the PR template for a smooth process. We will review it soon.

Copy link
Owner

@king04aman king04aman left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution, code is well-structured and demonstrates good use of OOPS principles. However, there are opportunities for improvement:

  • Refactor combat() to reduce code duplication for different opponents.
  • Create a helper function for clear inventory updates and displays.
  • Use with statements for all file operations to ensure proper closing.
  • Encapsulate game state in a dedicated class instead of using global variables.

@king04aman king04aman merged commit 85c2ead into king04aman:main Oct 24, 2024
1 check passed
Copy link
Contributor

👋 @king04aman 👋

🎉 You've just merged your pull request! We're excited to have you in our community. Keep up the fantastic contributions to the project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants