Skip to content

Add paranoid_confirmation:genocide option#1475

Closed
NateChoe1 wants to merge 4 commits intoNetHack:NetHack-3.7from
NateChoe1:NetHack-3.7
Closed

Add paranoid_confirmation:genocide option#1475
NateChoe1 wants to merge 4 commits intoNetHack:NetHack-3.7from
NateChoe1:NetHack-3.7

Conversation

@NateChoe1
Copy link
Copy Markdown

Addresses issue #1474, adds the paranoid_confirmation:genocide config option, which asks to confirm the chosen creature whenever you read a scroll of genocide. Here's a quick excerpt from running with a blessed scroll of genocide. I copy-pasted from ^P so the messages are read bottom-up:

Wiped out all liches.  Wiped out all demiliches.  Wiped out all master liches.  Wiped out all arch-liches.
Are you sure you want to genocide all 'L' monsters? [yes|no] yes
What class of monsters do you want to genocide? [enter the symbol or name representing a class, or '?'] L
Are you sure you want to genocide all '@' monsters? [yes|no] no
What class of monsters do you want to genocide? master mnd flayer # note the typo
As you read the scroll, it disappears.

Here's another with an uncursed scroll of genocide:

Wiped out all master mind flayers.
Are you sure you want to genocide all master mind flayers? [yes|no] yes
What type of monster do you want to genocide? [enter the name of a type of monster, or '?'] master mind flayer
Are you sure you want to genocide all monks? [yes|no] no
What type of monster do you want to genocide? master mnd flayer
As you read the scroll, it disappears.

@copperwater
Copy link
Copy Markdown
Contributor

One interface consideration here is that tiles players are less likely to know the ASCII character representing a monster class, so a prompt like "Are you sure you want to genocide all 'h' monsters?" probably won't save a dwarven adventurer from genociding themselves even with the confirmation.

Personally, I don't think a paranoid option to handle it is the best solution - the way I'd do it is to remove the fuzzy matching and re-prompt until the player enters something unambiguous (or until hitting a "That's enough tries!" case, whatever). For instance, uncursed and cursed genocide should require the entire string to match a monster species, so "master mnd flayer" won't work. Blessed genocide should require the string to match either a monster class symbol or its string description, so entering a single species won't work.

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.

2 participants