Java GImage图像均衡化返回全黑图 newPixelVal恒为0问题排查

问题现象 #

实现Java版本的图像直方图均衡化方法时，经校验方法调用、变量运行逻辑均无语法错误，但计算得到的newPixelVal始终为0，最终输出图像为全黑效果，无法定位根因。原始实现代码如下：

public GImage equalize(GImage source) {
    // TODO
    int[][] pixelArray = source.getPixelArray();
    int[] luminousArray = new int[257]; 
    int[] cumulativeLuminousArray = new int[257]; 
    int height = pixelArray.length;
    int width = pixelArray[0].length;
    int k =-1;
    int luminosity =0;
    int l=0;
    int[][]equalizedArray = new int[height][width];
    int newRed = 0;
    int newGreen = 0; 
    int newBlue = 0;
    int newPixelVal = 0;
    int pixelsLessThan = 0;
    int v = 0;
    int totalPixels = height*width;
    for (int i=0; i<height; i++) {
        for(int j=0;j<width;j++) {
            int pixel=pixelArray[i][j];
            int red = (pixel>>16)& 0xFF;
            int green = (pixel>>8)&0xFF;
            int blue=pixel&0xFF;
            
            luminosity = computeLuminosity(red, green, blue);
            luminousArray[luminosity] = luminousArray[luminosity] +1;
        }
    }
    
    while(k<=256) {
        k++;
        l=k;
        while(l>=0&&k<=256) {
            cumulativeLuminousArray[k] = cumulativeLuminousArray[k]+luminousArray[l];
            l--;
        }
    }
    for (int i=0; i<height; i++) {
        for(int j=0;j<width;j++) {
            int pixel=pixelArray[i][j];
            int red = (pixel>>16)& 0xFF;
            int green = (pixel>>8)&0xFF;
            int blue=pixel&0xFF;
    
            luminosity = computeLuminosity(red, green, blue);
            v = luminosity-1; 
            pixelsLessThan=cumulativeLuminousArray[v];
            newPixelVal = 255*(pixelsLessThan/totalPixels);
            newRed= newPixelVal; 
            newBlue = newPixelVal; 
            newGreen = newPixelVal;
            int newPixel = (0xFF<<24)|(newRed<<16)|(newGreen<<8)|newBlue;
            equalizedArray[i][j]=newPixel;
        }
    }
    return new GImage(equalizedArray);
}

根因分析 #

• 核心问题是Java整数除法的截断特性：代码中newPixelVal = 255*(pixelsLessThan/totalPixels);的两个操作数pixelsLessThan、totalPixels均为int整型，Java中整型除法会直接舍弃小数部分仅保留整数结果。除了亮度最高的像素点外，其余所有像素的pixelsLessThan都小于总像素数totalPixels，因此pixelsLessThan/totalPixels的计算结果始终为0，乘以255后依然为0，直接导致输出全黑。
• 存在隐藏的数组越界bug：当像素亮度luminosity为0时，v = luminosity -1取值为-1，访问cumulativeLuminousArray[-1]会抛出数组下标越界异常，该问题会在整数除法问题修复后暴露。
• 数组长度冗余：像素亮度取值范围为0-255共256个值，声明长度为257的统计数组虽然不会触发运行错误，但属于不必要的内存开销。

修复方案 #

• 调整计算顺序规避整数截断：将乘法操作移到除法之前执行，保证计算过程中数值不会提前被截断为0；如果需要更高精度，也可以将其中一个操作数转为浮点型计算后再四舍五入取整。
• 补全边界判断：处理亮度为0的场景，此时亮度小于当前值的像素数直接为0，避免访问负下标。

修复后的像素映射核心代码如下：

for (int i=0; i<height; i++) {
    for(int j=0;j<width;j++) {
        int pixel=pixelArray[i][j];
        int red = (pixel>>16)& 0xFF;
        int green = (pixel>>8)&0xFF;
        int blue=pixel&0xFF;

        luminosity = computeLuminosity(red, green, blue);
        // 处理亮度为0的边界，避免数组越界
        pixelsLessThan = luminosity == 0 ? 0 : cumulativeLuminousArray[luminosity - 1];
        // 先乘后除，避免整数除法提前截断
        newPixelVal = (255 * pixelsLessThan) / totalPixels;
        // 如需更高精度可替换为下方四舍五入写法
        // newPixelVal = (int)Math.round(255.0 * pixelsLessThan / totalPixels);
        newRed= newPixelVal; 
        newBlue = newPixelVal; 
        newGreen = newPixelVal;
        int newPixel = (0xFF<<24)|(newRed<<16)|(newGreen<<8)|newBlue;
        equalizedArray[i][j]=newPixel;
    }
}

内容的提问来源于stack exchange，提问作者user112167