c#中的3X3中值过滤但不起作用？

Question

真的，我正在尝试通过 C# 应用 3X3 中值过滤，根据我对中值过滤概念的理解，我编写了以下代码，但是当我运行它时，表单挂起。我认为在最后一个嵌套的 for 循环中有一些问题，但我不知道应用中位数概念的错误或错误在哪里！

public static Bitmap MedianFiltering(Bitmap bm)
    {
        List<int> termsList = new List<int>();
        Bitmap res, temp;
        Color c;
        int counter = 0;

        //Convert to Grayscale 
        for (int i = 0; i < bm.Width; i++)
        {
            for (int j = 0; j < bm.Height; j++)
            {
                c = bm.GetPixel(i, j);
                byte gray = (byte)(.333 * c.R + .333 * c.G + .333 * c.B);
                bm.SetPixel(i, j, Color.FromArgb(gray, gray, gray));
            }
        }

        temp = bm;

       //applying Median Filtering 
        for (int i = 0; i <= temp.Width - 3; i++)
            for (int j = 0; j <= temp.Height - 3; j++)
            {
                for (int x = i; x <= i + 2; x++)
                    for (int y = j; y <= j + 2; y++)
                    {

                        c = temp.GetPixel(x, y);
                        termsList.Add(c.R);
                        counter++;
                    }
                int[] terms = termsList.ToArray();
                Array.Sort<int>(terms);
                Array.Reverse(terms);
                int color = terms[4];
                temp.SetPixel(i + 1, j + 1, Color.FromArgb(color, color, color));
                counter = 0;
            }
        res = temp;

        return res;
    }

谢谢。

Answer 1

您没有在每次像素处理后清除termsList。这导致列表不断增长。对列表进行排序和反转将花费越来越长的时间。这也会导致不正确的结果，因为您只想获取与当前像素相关的 9 个像素的中位数。

像这样简单地清除列表：

...
int[] terms = termsList.ToArray();
termsList.Clear();
...

更新：

我对代码做了更多优化：

public static void MedianFiltering(Bitmap bm)
{
    List<byte> termsList = new List<byte>();

    byte[,] image = new byte[bm.Width,bm.Height];

    //Convert to Grayscale 
    for (int i = 0; i < bm.Width; i++)
    {
        for (int j = 0; j < bm.Height; j++)
        {
            var c = bm.GetPixel(i, j);
            byte gray = (byte)(.333 * c.R + .333 * c.G + .333 * c.B);
            image[i, j] = gray;
        }
    }

    //applying Median Filtering 
    for (int i = 0; i <= bm.Width - 3; i++)
        for (int j = 0; j <= bm.Height - 3; j++)
        {
            for (int x = i; x <= i + 2; x++)
                for (int y = j; y <= j + 2; y++)
                {
                    termsList.Add(image[x, y]);
                }
            byte[] terms = termsList.ToArray();
            termsList.Clear();
            Array.Sort<byte>(terms);
            Array.Reverse(terms);
            byte color = terms[4];
            bm.SetPixel(i + 1, j + 1, Color.FromArgb(color, color, color));
        }
}

请注意，在您的原始方法中，您返回了 Bitmap。我删除了这个。

请注意temp = bm; 不会创建Bitmap 的副本。它只是将temp 变量指向同一个对象（由bm 指向）。因此，在您的原始方法中，您返回了在方法参数中传递的确切对象。要使用新方法，请传递Bitmap，然后将修改它自己的位图（这也适用于您的方法）。

这在我的机器上提高了 4 倍的性能。

我所做的主要是将位图数据读入字节数组，而不是使用Bitmap它自己多次读/写数据。

如果您需要进一步提升性能，请查看this question。