【问题标题】:C# Blur Function Not WorkingC# 模糊功能不起作用
【发布时间】:2016-11-06 14:21:11
【问题描述】:

我正在尝试实现我在网上找到的模糊功能,但由于某种原因,它会将图像完全绘制为白色。我很确定原来的 sn-p 一开始并不完全正常,因此出现了错误。

这是最简单的代码形式:

        public void Blur(int blurSize)
        {
            // look at every pixel in the blur rectangle
            for (int xx = 0; xx < this.Width; xx++)
            {
                for (int yy = 0; yy < this.Height; yy++)
                {
                    float avgR = 0;
                    float avgG = 0;
                    float avgB = 0;
                    float avgA = 0;
                    int blurPixelCount = 0;
                    // average the color of the red, green and blue for each pixel in the
                    // blur size while making sure you don't go outside the image bounds
                    for (int x = xx; (x < xx + blurSize && x < this.Width); x++)
                    {
                        for (int y = yy; (y < yy + blurSize && y < this.Height); y++)
                        {
                            Color pixel = this.GetPixel(x, y);
                            avgR += pixel.R;
                            avgG += pixel.G;
                            avgB += pixel.B;
                            avgA += pixel.A;
                            blurPixelCount++;
                        }
                    }
                    avgR = avgR / blurPixelCount;
                    avgG = avgG / blurPixelCount;
                    avgB = avgB / blurPixelCount;
                    avgA = avgA / blurPixelCount;
                    // now that we know the average for the blur size, set each pixel to that color
                    for (int x = xx; x < xx + blurSize && x < this.Width; x++)
                    {
                        for (int y = yy; y < yy + blurSize && y < this.Height; y++)
                        {
                            SetPixel(x, y, new Color(avgR, avgG, avgB, avgA));
                        }
                    }
                }
            }
        }

例如,这里是模糊之前的图像:

这是在“尝试”模糊之后(我将背景更改为黑色,因为 Stackoverflow 是白色的):

我注意到,如果 blurSize 大于 1,则图像只是一个白色的污点,就像这样(使用 blurSize 2,背景透明度被我设置为黑色。):

您可以想象,任何高于 2 的值都会创建纯白色图像。以防万一不是很明显,这就是我想要的效果:

我的问题很简单,谁能发现这个模糊功能有什么问题?

更新:

根据要求,这是我的自定义位图类:

using System;
using System.Collections.Generic;
using System.Drawing;
using System.Drawing.Imaging;
using System.Linq;
using System.Runtime.InteropServices;
using System.Security.Cryptography.X509Certificates;
using System.Text;
using System.Threading.Tasks;
using GrimoireEngine.Framework.Maths;
using Point = GrimoireEngine.Framework.Maths.Point;
using Rectangle = GrimoireEngine.Framework.Maths.Rectangle;

namespace GrimoireEngine.Framework.Utilities
{
    public class GrimoireBitmap
    {
        public Bitmap Source;
        private IntPtr _iptr = IntPtr.Zero;
        public BitmapData BitmapData;

        public byte[] Pixels { get; set; }
        public int Depth { get; private set; }
        public int Width { get; private set; }
        public int Height { get; private set; }

        public Color this[int x, int y]
        {
            get { return GetPixel(x, y); }
            set { SetPixel(x, y, value); }
        }

        public Color this[Point point]
        {
            get { return GetPixel(point); }
            set { SetPixel(point, value); }
        }

        public bool IsLocked { get; private set; }

        public GrimoireBitmap(int width, int height)
        {
            this.Source = new Bitmap(width, height);
        }

        public GrimoireBitmap(int width, int height, PixelFormat format)
        {
            this.Source = new Bitmap(width, height, format);
        }

        public GrimoireBitmap(Bitmap image)
        {
            this.Source = image;
        }

        public GrimoireBitmap(string file)
        {
            this.Source = new Bitmap(file);
        }

        /// <summary>
        /// Lock bitmap data
        /// </summary>
        public void LockBits()
        {
            Width = Source.Width;
            Height = Source.Height;
            int pixelCount = Width * Height;
            System.Drawing.Rectangle rect = new System.Drawing.Rectangle(0, 0, Width, Height);
            Depth = Bitmap.GetPixelFormatSize(Source.PixelFormat);
            if (Depth != 8 && Depth != 24 && Depth != 32)
            {
                throw new ArgumentException("Only 8, 24 and 32 bpp images are supported.");
            }
            BitmapData = Source.LockBits(rect, ImageLockMode.ReadWrite, Source.PixelFormat);
            int step = Depth / 8;
            Pixels = new byte[pixelCount * step];
            _iptr = BitmapData.Scan0;
            Marshal.Copy(_iptr, Pixels, 0, Pixels.Length);
            this.IsLocked = true;
        }

        /// <summary>
        /// Unlock bitmap data
        /// </summary>
        public void UnlockBits()
        {
            Marshal.Copy(Pixels, 0, _iptr, Pixels.Length);
            Source.UnlockBits(BitmapData);
            this.IsLocked = false;
        }

        /// <summary>
        /// Get the color of the specified pixel
        /// </summary>
        /// <param name="x"></param>
        /// <param name="y"></param>
        /// <returns></returns>
        public Color GetPixel(int x, int y)
        {
            Color clr = Color.Transparent;
            int cCount = Depth / 8;
            int i = ((y * Width) + x) * cCount;

            if (i > Pixels.Length - cCount)
                throw new IndexOutOfRangeException();

            if (Depth == 32) // For 32 bpp get Red, Green, Blue and Alpha
            {
                byte b = Pixels[i];
                byte g = Pixels[i + 1];
                byte r = Pixels[i + 2];
                byte a = Pixels[i + 3]; // a
                clr = Color.FromNonPremultiplied(a, r, g, b);
            }
            if (Depth == 24) // For 24 bpp get Red, Green and Blue
            {
                byte b = Pixels[i];
                byte g = Pixels[i + 1];
                byte r = Pixels[i + 2];
                clr = Color.FromNonPremultiplied(r, g, b);
            }
            if (Depth == 8) // For 8 bpp get color value (Red, Green and Blue values are the same)
            {
                byte c = Pixels[i];
                clr = Color.FromNonPremultiplied(c, c, c);
            }
            return clr;
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="point"></param>
        /// <returns></returns>
        public Color GetPixel(Point point)
        {
            return GetPixel(point.X, point.Y);
        }

        /// <summary>
        /// Set the color of the specified pixel
        /// </summary>
        /// <param name="x"></param>
        /// <param name="y"></param>
        /// <param name="color"></param>
        public void SetPixel(int x, int y, Color color)
        {
            int i = (((y * Width) + x) * (Depth / 8));
            if (Depth == 32) // For 32 bpp set Red, Green, Blue and Alpha
            {
                Pixels[i] = color.B;
                Pixels[i + 1] = color.G;
                Pixels[i + 2] = color.R;
                Pixels[i + 3] = color.A;
            }
            if (Depth == 24) // For 24 bpp set Red, Green and Blue
            {
                Pixels[i] = color.B;
                Pixels[i + 1] = color.G;
                Pixels[i + 2] = color.R;
            }
            if (Depth == 8) // For 8 bpp set color value (Red, Green and Blue values are the same)
            {
                Pixels[i] = color.B;
            }
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="point"></param>
        /// <param name="color"></param>
        public void SetPixel(Point point, Color color)
        {
            SetPixel(point.X, point.Y, color);
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="x"></param>
        /// <param name="y"></param>
        /// <param name="width"></param>
        /// <param name="height"></param>
        /// <param name="color"></param>
        public void FillRectangle(int x, int y, int width, int height, Color color)
        {
            for (int i = 0; i < width; i++)
            {
                for (int j = 0; j < height; j++)
                {
                    SetPixel(i + x, j + y, color);
                }
            }
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="rectangle"></param>
        /// <param name="color"></param>
        public void FillRectangle(Rectangle rectangle, Color color)
        {
            FillRectangle(rectangle.X, rectangle.Y, rectangle.Width, rectangle.Height, color);
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="x"></param>
        /// <param name="y"></param>
        /// <param name="bitmap"></param>
        public void DrawBitmap(int x, int y, GrimoireBitmap bitmap)
        {
            for (int i = 0; i < bitmap.Width; i++)
            {
                for (int j = 0; j < bitmap.Height; j++)
                {
                    SetPixel(x + i, y + j, bitmap.GetPixel(i, j));
                }
            }
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="color"></param>
        public void Fill(Color color)
        {
            FillRectangle(0, 0, this.Width, this.Height, color);
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="x"></param>
        /// <param name="y"></param>
        /// <param name="width"></param>
        /// <param name="height"></param>
        /// <returns></returns>
        public GrimoireBitmap CopyRegion(int x, int y, int width, int height)
        {
            GrimoireBitmap bitmap = new GrimoireBitmap(width, height);
            bitmap.LockBits();
            for (int i = 0; i < width; i++)
            {
                for (int j = 0; j < height; j++)
                {
                    bitmap.SetPixel(i, j, this.GetPixel(x + i, y + j));
                }
            }
            bitmap.UnlockBits();
            return bitmap;
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="rectangle"></param>
        /// <returns></returns>
        public GrimoireBitmap CopyRegion(Rectangle rectangle)
        {
            return CopyRegion(rectangle.X, rectangle.Y, rectangle.Width, rectangle.Height);
        }

        /// <summary>
        /// 
        /// </summary>
        public void Clear()
        {
            for (int i = 0; i < this.Width; i++)
            {
                for (int j = 0; j < this.Height; j++)
                {
                    SetPixel(i, j, Color.Transparent);
                }
            }
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="fileName"></param>
        public void Save(string fileName)
        {
            this.Source.Save(fileName);
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="fileName"></param>
        public void Load(string fileName)
        {
            this.Source = new Bitmap(fileName);
        }

        /// <summary>
        /// 
        /// </summary>
        /// <returns></returns>
        public Bitmap ToBitmap()
        {
            return this.Source;
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="image"></param>
        public void FromBitmap(Bitmap image)
        {
            this.Source = image;
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="x"></param>
        /// <param name="y"></param>
        /// <param name="width"></param>
        /// <param name="height"></param>
        /// <param name="color"></param>
        /// <param name="amount"></param>
        public void Blend(int x, int y, int width, int height, Color color, float amount)
        {
            for (int i = 0; i < width; i++)
            {
                for (int j = 0; j < height; j++)
                {
                    SetPixel(i + x, j + y, Color.Blend(GetPixel(i + x, j + y), color, amount));
                }
            }
        }

        /// <summary>
        /// 
        /// </summary>
        /// <param name="rectangle"></param>
        /// <param name="color"></param>
        /// <param name="amount"></param>
        public void Blend(Rectangle rectangle, Color color, float amount)
        {
            Blend(rectangle.X, rectangle.Y, rectangle.Width, rectangle.Height, color, amount);
        }

        public void Blur(int blurSize)
        {
            // look at every pixel in the blur rectangle
            for (int xx = 0; xx < this.Width; xx++)
            {
                for (int yy = 0; yy < this.Height; yy++)
                {
                    float avgR = 0;
                    float avgG = 0;
                    float avgB = 0;
                    float avgA = 0;
                    int blurPixelCount = 0;
                    // average the color of the red, green and blue for each pixel in the
                    // blur size while making sure you don't go outside the image bounds
                    for (int x = xx; (x < xx + blurSize && x < this.Width); x++)
                    {
                        for (int y = yy; (y < yy + blurSize && y < this.Height); y++)
                        {
                            Color pixel = this.GetPixel(x, y);
                            avgR += pixel.R;
                            avgG += pixel.G;
                            avgB += pixel.B;
                            avgA += pixel.A;
                            blurPixelCount++;
                        }
                    }
                    avgR = avgR / blurPixelCount;
                    avgG = avgG / blurPixelCount;
                    avgB = avgB / blurPixelCount;
                    avgA = avgA / blurPixelCount;
                    // now that we know the average for the blur size, set each pixel to that color
                    for (int x = xx; x < xx + blurSize && x < this.Width; x++)
                    {
                        for (int y = yy; y < yy + blurSize && y < this.Height; y++)
                        {
                            SetPixel(x, y, new Color(avgR, avgG, avgB, avgA));
                        }
                    }
                }
            }
        }
    }
}

【问题讨论】:

  • 您是如何使用此代码的?我的意思是,例如,System.Drawing.Color 没有接受 4 个参数的构造函数,所以您使用的是不同于简单位图的东西?至于算法本身,它工作得很好。
  • 我有一个自定义颜色类。五秒钟。
  • @Evk 我的颜色类,基于 Monogame:pastebin.com/2BuzKjVW
  • 好的,但随后出现了其他问题。您将其传递给 SetPixel,因此它也是一些未显示代码的自定义函数。询问是因为如果将此算法应用于常规位图,没有任何自定义类,那么它可以正常工作。
  • 您遇到的一个(但不仅是)问题是您使用字节值(范围 0-255)计算平均值(avgR 等),但随后您将这些平均值作为浮点数提供给您的 Color 构造函数,这然后将它们乘以 255,认为它们在 0-1 范围内。所以基本上你用 (255,255,255) 颜色填充几乎所有东西。为防止将来发生此类事情,请不要静默调整变量,而是立即抛出异常。因此,如果构造函数期望值在 0-1 范围内,并且我传递值 50 - 不要将其设为 255,而是抛出异常。

标签: c#


【解决方案1】:

编辑: 我修改了代码以使用 System.Drawing.Bitmap。它可以正常工作,这是 Blur=5 的结果:

代码如下:

   public void Blur(int blurSize, Bitmap input)
    {
        // look at every pixel in the blur rectangle
        for (int xx = 0; xx < input.Width; xx++)
        {
            for (int yy = 0; yy < input.Height; yy++)
            {
                float avgR = 0;
                float avgG = 0;
                float avgB = 0;
                float avgA = 0;
                int blurPixelCount = 0;
                // average the color of the red, green and blue for each pixel in the
                // blur size while making sure you don't go outside the image bounds
                for (int x = xx; (x < xx + blurSize && x < input.Width); x++)
                {
                    for (int y = yy; (y < yy + blurSize && y < input.Height); y++)
                    {
                        Color pixel = input.GetPixel(x, y);
                        avgR += pixel.R;
                        avgG += pixel.G;
                        avgB += pixel.B;
                        avgA += pixel.A;
                        blurPixelCount++;
                    }
                }
                avgR = avgR / blurPixelCount;
                avgG = avgG / blurPixelCount;
                avgB = avgB / blurPixelCount;
                avgA = avgA / blurPixelCount;
                // now that we know the average for the blur size, set each pixel to that color
                for (int x = xx; x < xx + blurSize && x < input.Width; x++)
                {
                    for (int y = yy; y < yy + blurSize && y < input.Height; y++)
                    {
                        input.SetPixel(x, y, Color.FromArgb((int)avgA, (int)avgR, (int)avgG, (int)avgB));
                    }
                }
            }
        }
    }

所以 SetPixel 或 GetPixel 或 new Color 等方法之一无法正常工作。

【讨论】:

  • 感谢您的确认;我会回拨并查看我的框架。
  • 我发现问题确实与 Color 构造函数有关。给定浮点数时,它乘以 255。我通过转换为一个字节来解决这个问题。该算法现在可以完美运行。
  • @Krythic 很高兴
猜你喜欢
  • 1970-01-01
  • 2023-03-17
  • 2017-11-27
  • 2017-09-15
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2023-03-14
相关资源
最近更新 更多