Granny's Programming Pearls
"inside of every large program is a small program struggling to get out"
JavaRanch.com/granny.jsp
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic

Trouble with recursion in minesweeper

 
Angela Li
Greenhorn
Posts: 12
  • 0
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
This is my code for inspecting a cell in minesweeper, and it is not working as I had expected but I can't quite figure out what is wrong. Some help would be appreciated.
 
Tim Moores
Bartender
Posts: 2502
14
  • 0
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
It would help to know what you expected this method to do, and how its actual behavior deviates from that.
 
Angela Li
Greenhorn
Posts: 12
  • 0
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
So, I expected the method to open up just the cell (x,y) if it is next to a mine so its not zero. But if it is zero or no mines are adjacent to it then I expect it to recursively open the cells around it. But the method is failing to recursively open.
 
Stephan van Hulst
Bartender
Pie
Posts: 4846
34
Chrome Netbeans IDE Windows
  • 0
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
It looks like you are checking your loop variables wrongly. Shouldn't they be allowed to be equal to 0 as well? Anyway, you have a lot of clutter in that snippet of code. Here's a simpler version:
 
Angela Li
Greenhorn
Posts: 12
  • 0
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
thanks for making my code more concise, i was getting confused myself with all the if statements. It works now that I took your suggestion, thanks so much. I spent so long on this.
 
I agree. Here's the link: http://aspose.com/file-tools
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic