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

Outside as safe cells #13

Open
837951602 opened this issue Dec 27, 2019 · 6 comments
Open

Outside as safe cells #13

837951602 opened this issue Dec 27, 2019 · 6 comments

Comments

@837951602
Copy link

pic
Here outside is safe, but it still allows to guess

Not hard to fix I guess

@pwmarcz
Copy link
Owner

pwmarcz commented Dec 27, 2019 via email

@837951602
Copy link
Author

1 (one that easily created) and
2 (one that may appear in real game) won't need any guess, so allowing guess is against the "cruel but fair" spirit.

In't it just add

if there is outside and outside is safe, then there is safe

?

@837951602
Copy link
Author

837951602 commented Dec 30, 2019

and I actually meet it
a

@kaymes
Copy link

kaymes commented Feb 19, 2020

I actually met this situation in real game play a few times and was just about to report it as a bug.

@meeque
Copy link

meeque commented Mar 20, 2022

Isn't counting remaining mines a standard endgame strategy?

I do not think that fixing this bug "would make the rules more complicated". The rules say:
"If you try to guess, the game will always choose the worst scenario. Except when you are forced to guess"

In the examples given above, you're clearly not forced to guess. Some of the non-adjacent fields are definitely safe. So revealing an adjacent field should be punished;)

Sorry, for smart-assing. Unfortunately, I have no idea how to fix this.

@837951602
Copy link
Author

If someone didn't see: Before, in this case, you're still required to click on border, clicking somewhere else is broken (#10).

837951602 added a commit to 837951602/kaboom that referenced this issue Oct 16, 2022
`hasSafeCells` still only refer to surrounded cells
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

No branches or pull requests

4 participants