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

Room name randomizer doesn't increment correctly #43

Open
tmcclintock opened this issue Jan 2, 2021 · 0 comments
Open

Room name randomizer doesn't increment correctly #43

tmcclintock opened this issue Jan 2, 2021 · 0 comments
Labels
bug Something isn't working

Comments

@tmcclintock
Copy link
Owner

The AlphaNumRoomName is used to name Rooms being put in the dungeon. It is supposed to name room A0, B0, ... Z0, A1, B1, etc. in that order. However it is bugged because it generates names for rooms without knowing if they get put in the dungeon (e.g. they may overlap and get rejected).

This is a bug.

The fix is the following: the AlphaNumRoomName.randomize_room_name should take the dungeon as an argument. It should store a name and keep trying to use it unless it is in the dungeon.rooms dictionary, in which case it generates a new name.

@tmcclintock tmcclintock added the bug Something isn't working label Jan 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant