删除二维数组中行的最快方法

Question

我有一个程序需要从byte[][] 数组中删除一组行。我想删除其中没有零的每一行。现在我正在使用以下方法：

public byte[][] checkRows(byte[][] grid) {
    LinkedList<Integer> temp = new LinkedList<Integer>();
    boolean willRemove = false;
    for (int y = 0; y < grid.length; y++) {
        boolean tempCheck = true;
        for (int x = 0; x < grid[0].length; x++) {
            if (grid[y][x] == 0) {
                tempCheck = false;
                break;
            }
        }
        if (tempCheck) {
            temp.add(y);
            willRemove = true;
        }
    }

    if (willRemove) {
        int[] rowArray = convert(temp);
        return removeRows(grid, rowArray);
    }
    return grid;

}

public byte[][] removeRows(byte[][] grid, int[] rows) {
    int total = rows.length;
    int current = 0;
    byte[][] retGrid = new byte[grid.length][grid[0].length];
    for (int i = total; i < grid.length; i++) {
        if (current < total && i-total+current == rows[current]) {
            current++;
        }
        //retGrid[i] = grid[i-total+current].clone();
        System.arraycopy(grid[i-total+current], 0, retGrid[i], 0, xsize);

    }
    return retGrid;
}

public int[] convert(LinkedList<Integer> intList) {
    int[] retArray = new int[intList.size()];
    for (int i = 0; i < retArray.length; i++) {
        retArray[i] = intList.get(i).intValue();
    }
    return retArray;
}

这为我提供了一种从 2D 数组中删除行并用数组顶部的零行替换它们的相当快速的方法。有没有更快的方法来达到同样的效果？

如果不清楚脚本的作用，它是用于删除俄罗斯方块游戏中的完整行。

更新：使用 System.arraycopy() 代替 clone() 可为小型数组提供 5% 的性能提升。

Answer 1

使用链表会产生 O(1) 次删除，请参阅 this answer，因为无论如何都必须迭代列表。

起初我认为 multidim 数组是紧凑的，因为它是一个连续的内存块，但it seems 并非如此。因此，您不会失去任何可能已经生效的缓存优势。

可惜 Java 没有值类型（目前），我会使用一个而不是一个字节来编码信息。好吧，这不是绝对必要的……

从代码审查的角度来看，在方法 checkRows 中使用 bool willRemove 是不必要的，因为在这些情况下，temp 将包含多个元素。如果不需要，我会尝试完全消除 ArrayList 分配 - 推迟它。

Answer 2

恕我直言，这太复杂了。放弃LinkedList，因为无论如何这都是一件可怕的事情。不要费心收集要删除的索引，而是将要保留的每一行复制到另一个数组。然后用arraycopy覆盖原来的。

您正在复制整行，但您不必这样做。只需重新排列它们，使保留的行落下并将完整的行移到顶部并清洁它们。无需分配内存。

由于所有复制操作仅适用于一维，因此没有太多需要优化的地方，因为最耗时的操作可能是确定一行中是否有任何零并（不时）清理一些行。

我想知道它在什么机器上运行，因为我猜它在最慢的手机上也一定非常快。顺便提一句。 CR 会更适合。想想更好的名字，例如，tempCheck -> noZeroFound。

Answer 3

这个小方法针对一维案例执行问题的所需功能。

private static final void onClearFilledRows(final byte[] pTetris, final int pRowLength) {
        int i = 0, j = 0;
        /* Track the last valid position of row data. */
        int lLastValidIndex = 0;
        /* Iterate through each row. */
        for(i = 0; i < pTetris.length; i += pRowLength) {
            /* Iterate through each cell in the row. */
            boolean lContainsZero = false;
            for(j = i; j < (i + pRowLength) & !lContainsZero; j++) {
                lContainsZero |= pTetris[j] == 0;
            }
            /* This row is valid. Copy it to the last valid index. */
            if(lContainsZero) {
                System.arraycopy(pTetris, i, pTetris, (lLastValidIndex++ * pRowLength), pRowLength);
            }
        }
        /* Empty the remaining rows. */
        for(i = lLastValidIndex * pRowLength; i < pTetris.length; i++) {
            /* Set the element to zero. */
            pTetris[i] = 0;
        }
    }

然后可以针对二维情况重新设计此逻辑：

private static final void onClearFilledRows(final byte[][] pTetris) {
        int i = 0, j = 0;
        /* Track the last valid position of row data. */
        int lLastValidIndex = 0;
        /* Iterate through each row. */
        for(i = 0; i < pTetris.length; i ++) {
            /* Iterate through each cell in the row. */
            boolean lContainsZero = false;
            for(j = 0; j < pTetris[i].length & !lContainsZero; j++) {
                lContainsZero |= pTetris[i][j] == 0;
            }
            /* This row is valid. Copy it to the last valid index. */
            if(lContainsZero) {
                System.arraycopy(pTetris[i], 0, pTetris[lLastValidIndex++], 0, pTetris[i].length);
            }
        }
        /* Empty the remaining rows. */
        for(i = lLastValidIndex; i < pTetris.length; i++) {
            for(j = 0; j < pTetris[i].length; j++) {
                /* Set the element to zero. */
                pTetris[i][j] = 0;
            }
        }
    }