4

我目前正在制作扫雷克隆。我做了一个算法，当点击一个周围有 0 个地雷的图块时，显示所有周围有 0 个地雷的邻居，然后显示它们的所有邻居，周围有 0 个地雷......（递归）。这个结果只需要一键：

它像它应该的那样工作，但它太慢了。最初的扫雷会立即显示这些图块，但就我而言，它们在显示之间有一点延迟。

我写了这段代码：

private void RevealNeighbor(int x, int y) {
    foreach(var neighbor in _neighbors) {
        try {
            Tile tile = _tiles[x + neighbor[0], y + neighbor[1]];
            if(tile.TileType == TileType.Empty && tile.Hidden) {
                tile.Reveal();
                if(tile.Number == 0) {
                    RevealNeighbor(x + neighbor[0], y + neighbor[1]);
                }
            }
        }
        catch(IndexOutOfRangeException) {
        }
    }
}

_neighbors是一个数组数组，它​​有 8 个邻居的位置偏移量：

private readonly int[][] _neighbors = new[] {
                                          new[] {-1, -1},
                                          new[] {0, -1},
                                          new[] {1, -1},
                                          new[] {1, 0},
                                          new[] {1, 1},
                                          new[] {0, 1},
                                          new[] {-1, 1},
                                          new[] {-1, 0}
                                      };

我怎样才能让它更快？

4

2 回答 2

3

使用 SuspendLayout 方法，仅在需要时绘制：

*false 是设计器的默认值，找出它的含义

this.SuspendLayout();
... logic      
this.ResumeLayout(false);

ALSO - 避免滥用 Exceptions 机制，这不是有效且不好的做法

而是使用一堵墙（矩阵末端的额外瓷砖表示......好吧......一堵墙）。

于 2012-12-16T16:22:41.020 回答
1

我猜 RevealNeighbor 实际上会重新绘制董事会。相反，您应该计算将要显示的内容，然后才重新绘制董事会。

于 2012-12-16T15:58:57.503 回答