C++卷积函数实现问题：程序崩溃或像素偏移

问题分析 #

你的代码存在两个核心问题：

1. 数组越界导致崩溃：固定用y-1和x-1计算邻域偏移，仅适配3x3卷积核。当核尺寸变化，或处理图像边缘像素时，会算出负数或超出图像范围的索引，访问非法内存引发崩溃。
2. 索引计算错误导致像素偏移：(y - 1 - k)和(x - 1 - l)的写法颠倒了卷积核的遍历顺序，相当于对核做反向映射，最终输出图像出现偏移。

修复后的代码 #

// 计算卷积核的半高和半宽，适配任意奇数尺寸的核
int filterHalfHeight = rows / 2;
int filterHalfWidth = cols / 2;

for (int y = filterHalfHeight; y < m_imageHeight - filterHalfHeight; y++)
{
    for (int x = filterHalfWidth; x < m_imageWidth - filterHalfWidth; x++)
    {
        float sumRed = 0, sumGreen = 0, sumBlue = 0;

        // 遍历卷积核的每个元素
        for (int k = 0; k < rows; k++)
        {
            for (int l = 0; l < cols; l++)
            {
                // 计算对应图像像素的索引：当前像素(y,x)加上核偏移(k - filterHalfHeight, l - filterHalfWidth)
                int imgY = y + k - filterHalfHeight;
                int imgX = x + l - filterHalfWidth;
                int posImage = (imgY * m_imageWidth + imgX) * 3;
                
                sumRed += m_copyImage[posImage] * filter[k][l];
                sumGreen += m_copyImage[posImage + 1] * filter[k][l];
                sumBlue += m_copyImage[posImage + 2] * filter[k][l];
            }
        }

        // 结果截断到0-255范围
        sumRed = std::max(std::min(sumRed, 255.0f), 0.0f);
        sumGreen = std::max(std::min(sumGreen, 255.0f), 0.0f);
        sumBlue = std::max(std::min(sumBlue, 255.0f), 0.0f);

        // 更新原图像对应像素
        int pos = (y * m_imageWidth + x) * 3;
        m_myImage[pos] = static_cast<unsigned char>(sumRed);
        m_myImage[pos + 1] = static_cast<unsigned char>(sumGreen);
        m_myImage[pos + 2] = static_cast<unsigned char>(sumBlue);
    }
}

关键修复点 #

• 动态适配核尺寸：通过filterHalfHeight和filterHalfWidth确定图像的有效计算区域，避免边缘越界，同时支持任意奇数尺寸的卷积核（如3x3、5x5）。
• 修正索引映射：用y + k - filterHalfHeight和x + l - filterHalfWidth计算邻域像素位置，确保卷积核与图像邻域的对应关系正确，解决偏移问题。
• 保留原图像副本：继续使用m_copyImage存储原始像素数据，避免计算过程中修改原图像导致的错误叠加。

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