Skip to content
This repository was archived by the owner on Aug 6, 2024. It is now read-only.

Various changes (see description)#147

Draft
silvncr wants to merge 14 commits intomasterfrom
144-replace-input-with-msvcrtgetche
Draft

Various changes (see description)#147
silvncr wants to merge 14 commits intomasterfrom
144-replace-input-with-msvcrtgetche

Conversation

@silvncr
Copy link
Copy Markdown
Member

@silvncr silvncr commented Apr 15, 2023

Ease of use

  • Replace input() with msvcrt.getche(), lowering the key presses per input from 2 to 1

File system

  • Move code into separate files relating to its purpose

Code quality

  • Use more classes for repetitive tasks (in-game locations, etc.)

@silvncr silvncr linked an issue Apr 15, 2023 that may be closed by this pull request
@silvncr silvncr self-assigned this Apr 15, 2023
@silvncr silvncr changed the title Huge rewrite Various changes (see description) Apr 17, 2023
silvncr and others added 4 commits April 17, 2023 12:43
Inspired by #34

Co-Authored-By: Jérémy LARDENOIS <37746231+JeremyLARDENOIS@users.noreply.github.com>
@silvncr silvncr added bug Something isn't working enhancement New feature or request dependencies Pull requests that update a dependency file maintenance GitHub documentation or issues/PRs labels Apr 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working dependencies Pull requests that update a dependency file enhancement New feature or request maintenance GitHub documentation or issues/PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace input() with msvcrt/getch functions

2 participants