Skip to content

MainBranchUpdated#114

Open
Increase27 wants to merge 6 commits intocfrantzidis:masterfrom
Increase27:Development
Open

MainBranchUpdated#114
Increase27 wants to merge 6 commits intocfrantzidis:masterfrom
Increase27:Development

Conversation

@Increase27
Copy link
Copy Markdown

No description provided.

@LewisG743
Copy link
Copy Markdown

After reviewing program I am able to see that the base requirements have been met. You've used multiple object orientated principles to achieve a working solution and you've used encapsulation and abstraction in such a way that the code is easy to understand and overall simpler. Some errors haven't quite been met, for example in your switch case statements there is not default case where if non of the cases are true then a default case would run. To expand on your solution you could also implement a door so that the key has a purpose.

@BoredHF
Copy link
Copy Markdown

BoredHF commented Mar 13, 2025

Your code is overall very well-documented and easy to read - Good use of encapsulation and methods.

The only suggestions that I could give is to add a feature for the player to name their own character at the start of the game.

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.

3 participants