Question

在我的代码中，我使用2D多维数组来表示网格（并不总是大小相等，例如10x15或21x7）。在阅读了锯齿状阵列如何更快并且通常被认为更好之后，我决定将我的2D阵列更改为锯齿状阵列。

这是我声明多维数组的方式：

int[,] array = new int[10, 10];

我正在试图弄清楚如何声明然后初始化相同的东西，但使用锯齿状数组。

编辑此代码位于类中，并且在我已有的构造函数中：

class ProceduralGrid
{
    private int[][] grid;

    private int _columns;
    private int _rows;

    public ProceduralGrid(int rows, int columns)
    {
        _rows = rows;             //For getters
        _columns = columns;

        //Create 2D grid
        int x, y;
        grid = new int[rows][];

        for (x = 0; x < grid.Length; x++)
        {
            grid[x] = new int[10];
        }
    }

    public int GetXY(int rows, int columns)
    {
        if (rows >= grid.GetUpperBound(0) + 1)
        {

            throw new ArgumentException("Passed X value (" + rows.ToString() +
                ") was greater than grid rows (" + grid.GetUpperBound(0).ToString() + ").");
        }
        else
        {
            if (columns >= grid.GetUpperBound(1) + 1)
            {

                throw new ArgumentException("Passed Y value (" + columns.ToString() +
                    ") was greater than grid columns (" + grid.GetUpperBound(1).ToString() + ").");
            }
            else
            {
                return grid[rows][columns];
            }
        }
    }
}

另一种方法我只是在做：

    Console.WriteLine(grid.GetXY(5, 5).ToString());

我收到错误消息：

Unhandled Exception: System.IndexOutOfRangeException: Array does not have that m
any dimensions.
   at System.Array.GetUpperBound(Int32 dimension)
   at ProcGen.ProceduralGrid.GetXY(Int32 rows, Int32 columns) in C:\Users\Lloyd\
documents\visual studio 2010\Projects\ProcGen\ProcGen\ProceduralGrid.cs:line 115
   at ProcGen.Program.Main(String[] args) in C:\Users\Lloyd\documents\visual stu
dio 2010\Projects\ProcGen\ProcGen\Program.cs:line 27

我做错了什么，我应该怎么做？

Answer 1

由于您正在处理一维数组，因此您只需使用Length Property来获取第一维的长度：

int[][] grid = new int[10][];

for (int x = 0; x < grid.Length; x++)
{
    grid[x] = new int[10];
}

（使用GetLength Method也可以：）

int[][] grid = new int[10][];

for (int x = 0; x < grid.GetLength(0); x++)
{
    grid[x] = new int[10];
}

您的代码存在的问题是，您正在调用grid.GetUpperBound(1)，其中grid是一维数组 - 它没有第二个维度（索引1），您可以获得。的上限。

您的GetXY方法应如下所示：

public int GetXY(int x, int y)
{
    if (x < 0 || x >= grid.Length)
    {
        throw ...
    }

    int[] items = grid[x];

    if (y < 0 || y >= items.Length)
    {
        throw ...
    }

    return items[y];
}

请注意，参差不齐的数组并不能让您的代码更快化 - 请测量它们是否确实存在！

将2D数组更改为锯齿状数组

1 个答案: