0

我正在尝试使用位图在 Android 中实现平均/平均 3x3 过滤器。

每次我尝试应用此过滤器时,应用程序都会继续关闭。当我尝试在图像上应用此过滤器时出现 ArrayIndexOutOfBoundsException 错误。

我的代码如下:

    for (int x = 0; x < width; x++) {
        for (int y = 0; y < height; y++) {

            int index = 0;

            for (int filterX = -filterWidth / 2; filterX < filterWidth / 2; filterX++) {
                for (int filterY = -filterHeight / 2; filterY < filterHeight / 2; filterY++) {

                    A = (pixels[x+filterX+width*(y+filterY)])>>24 & 0xFF;
                    R = (pixels[x+filterX+width*(y+filterY)] >> 16) & 0xFF;
                    G = (pixels[x+filterX+width*(y+filterY)] >> 8) & 0xFF;
                    B = pixels[x+filterX+width*(y+filterY)] & 0xFF;


                        }
                }
       }
 }
4

1 回答 1

3

你没有显示堆栈跟踪(编辑:现在你做了,谢谢),这将表明哪一行正在引发异常。

但是乍一看,您的问题似乎在这里:

for (int x = 0; x < width; x++) {
    for (int y = 0; y < height; y++) {
        int index = 0;
        for (int filterX = -filterWidth / 2; filterX < filterWidth / 2; filterX++) {
            for (int filterY = -filterHeight / 2; filterY < filterHeight / 2; filterY++) {
                A = (pixels[x+filterX+width*(y+filterY)])>>24 & 0xFF;
                R = (pixels[x+filterX+width*(y+filterY)] >> 16) & 0xFF;
                G = (pixels[x+filterX+width*(y+filterY)] >> 8) & 0xFF;
                B = pixels[x+filterX+width*(y+filterY)] & 0xFF;
                ...
            } 
        }
    }
 }

您正在超出像素数组的范围,因为您试图将 3x3 过滤器直接应用于图像的边缘,因此过滤器超出了图像范围。考虑您的第一次迭代:

  • x = 0
  • y = 0
  • filterX = -filterWidth / 2 = -3 / 2 = -1
  • filterY = -filterHeight / 2 = -3 / 2 = -1

然后您尝试访问pixels[x+filterX+width*(y+filterY)]

  • 0 + -1 + 宽度 * (0 + -1)
  • = -1 + -宽度

所以你可以看到,访问pixels[-1 + -width]显然是越界的。如果您的 filterX/filterY 循环正确,上限也会发生同样的情况(请参阅此答案底部的注释——此时您的大小为 3,filterX 和 filterY 停止在 0,而不是 1)。

您将不需要在距边缘的 filterWidth/2(或 filterHeight/2 垂直)像素内应用过滤器,或者在读取图像像素并将边缘以外的区域视为 0 时需要进行一些边界检查。

一种可能的优化是在图像周围创建一个 filterWidth/2(或 filterHeight/2 垂直)黑色边框,并且不处理边缘附近的那些像素。然后你不需要分支来进行边界检查。

顺便说一句,对于奇数大小的过滤器,您需要<= filterWidth / 2在循环中执行(高度相同):

for (int filterX = -filterWidth / 2; filterX <= filterWidth / 2; filterX++) {
    for (int filterY = -filterHeight / 2; filterY <= filterHeight / 2; filterY++) {

这适用于所有奇数大小的过滤器。

于 2013-08-06T08:06:40.733 回答