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

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;
}

这为我提供了一种从二维数组中删除行并将其替换为数组顶部零行的相当快的方法。有没有更快的方法达到同样的效果？

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

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

Answer 1

使用链表将进行 O(1) 次删除，参见 this answer，因为无论如何都必须迭代该列表。

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

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

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

Answer 2

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

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;
            }
        }
    }

Answer 3

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

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

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

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

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

Fastest way to remove rows in 2D Array

java

arrays

optimization

clone

arraylist