C#处理图片内存不足

Question

public static void MakeThumNail(string originalImagePath,string thumNailPath,int width, int height, string model)
        {
                //System.Drawing.Image originalImage = System.Drawing.Image.FromFile(originalImagePath);
                Bitmap originalImage =(Bitmap)Bitmap.FromFile(originalImagePath);
                int thumWidth = width;//缩略图的宽度        
                int thumHeight = height; //缩略图的高度       
                int x = 0; int y = 0;
                int originalWidth = originalImage.Width;//原始图片的宽度       
                int originalHeight = originalImage.Height;//原始图片的高度        
                switch (model)
                {
                    case "HW": //指定高宽缩放,可能变形                
                        break;
                    case "W": //指定宽度,高度按照比例缩放               
                        thumHeight = originalImage.Height * width / originalImage.Width;
                        break;
                    case "H": //指定高度,宽度按照等比例缩放                
                        thumWidth = originalImage.Width * height / originalImage.Height;
                        break;
                    case "Cut":
                        if ((double)originalImage.Width / (double)originalImage.Height > (double)thumWidth / (double)thumHeight)
                        {
                            originalHeight = originalImage.Height;
                            originalWidth = originalImage.Height * thumWidth / thumHeight; y = 0; x = (originalImage.Width - originalWidth) / 2;
                        }
                        else
                        {
                            originalWidth = originalImage.Width;
                            originalHeight = originalWidth * height / thumWidth; x = 0;
                            y = (originalImage.Height - originalHeight) / 2;
                        }
                        break;
                    default:
                        break;
                }  //新建一个bmp图片        
                System.Drawing.Image bitmap = new System.Drawing.Bitmap(thumWidth, thumHeight); //新建一个画板        
                System.Drawing.Graphics graphic = System.Drawing.Graphics.FromImage(bitmap); //设置高质量查值法       
                graphic.InterpolationMode = System.Drawing.Drawing2D.InterpolationMode.High;        //设置高质量，低速度呈现平滑程度   
                graphic.SmoothingMode = System.Drawing.Drawing2D.SmoothingMode.HighQuality;        //清空画布并以透明背景色填充       
                graphic.Clear(System.Drawing.Color.Transparent);        //在指定位置并且按指定大小绘制原图片的指定部分       
                graphic.DrawImage(originalImage, new System.Drawing.Rectangle(0, 0, thumWidth, thumHeight), new System.Drawing.Rectangle(x, y, originalWidth, originalHeight),
               System.Drawing.GraphicsUnit.Pixel);
               originalImage.Dispose();
               graphic.Dispose();
               GC.Collect();
                try
                {
                    bitmap.Save(thumNailPath, System.Drawing.Imaging.ImageFormat.Jpeg);
                }
                catch (Exception ex) { throw ex; }
                finally { originalImage.Dispose(); bitmap.Dispose(); graphic.Dispose(); GC.Collect(); }大家帮我看下吧,什么原因呀纠结死我了...C#图片内存

Answer 1

你用try catch 和 finally要写程序，在finally内写你的清理资源代码，即把 orginimgae.Dispose()这样的代码放在finally内，这样才能确保每次显示一张图片之后都能清除这张图片的资源代码。

Answer 2

不用每次都执行GC的回收
使用using
及时释放内存，MackThumMail这个是自己写的吗？这里边有没有问题，或者注释掉这一行看还有没有问题

Answer 3

不好意思，没看清楚代码就是在MackThumNail里

Answer 4

那  public static void bianliFile()
        {
            DirectoryInfo theFolder = new DirectoryInfo(@"Z:\Marketed");
            DirectoryInfo[] dirInfo = theFolder.GetDirectories();
            //遍历文件夹
            foreach (DirectoryInfo NextFolder in dirInfo)
            {
                FileInfo[] fileInfo = NextFolder.GetFiles();
                foreach (FileInfo NextFile in fileInfo)
                {
                    if (NextFile.Extension.Equals(".jpg") || NextFile.Extension.Equals(".JPG") || NextFile.Extension.Equals(".jpge") || NextFile.Extension.Equals(".jpeg"))
                    {
                        if (NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_AZ.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_BO.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_CH.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_LU.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_NE.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_NI.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_OR.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_RO.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_SO.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_TO.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_VT.jpg" || NextFile.FullName.Substring(NextFile.FullName.Length - 7, 7) == "_XE.jpg")
                        {
                            GC.Collect();
                            Bitmap originalImage =(Bitmap)Bitmap.FromFile(NextFile.FullName);
                            int originalHeight = originalImage.Size.Height;//原始图片的宽度
                            int originalWidth = originalImage.Size.Width;//原始图片的高度
                            originalImage.Dispose();
                            GC.Collect();
                            if (originalWidth == 400 && originalHeight == 400)
                            {
                                int index = NextFile.FullName.LastIndexOf(".");
                                int g = NextFile.Name.LastIndexOf(".");
                                int h = NextFile.Name.LastIndexOf("_");
                                Directory.CreateDirectory("C:\\product\\" + NextFolder.Name + "");
                                MakeThumNail(NextFile.FullName, "C:\\product\\" + NextFolder.Name + "\\" + NextFile.Name.Substring(0, g) + ".jpg", 400, 400, "W");
                            }
                            else
                            { }
                            originalImage.Dispose();
                            GC.Collect();
                        }
                    }
                }
                Thread.Sleep(1000);
            }这个代码要怎么改写呢？？？

Answer 5

MackThumMail这个方法没有什么问题呀，就是下面的这一个有问题呀

Answer 6

GC.Collect只是告诉GC你可以进行回收垃圾了，而不是强制GC进行回收，如果那个时候CPU繁忙，这个指令是无效的。因此你的这个循环过程中，垃圾就没有被回收。

Answer 7

本人建议将originalImage变量声明在foreach循环体外。因为BitMap是一个Class类型，在循环体内声明，则系统可能要为你的1万张图片反复分配堆内存。造成内存来不及回收的情况下，内存不够用。如果在循环体外声明BitMap,则1万张图片反复使用同一块堆内存。
不知这样理解是否有误，有待你的测试。

Answer 8

如果放到外面的话，我这个参数NextFile.FullName就调用不到了阿？？？那么路径怎么给呢

Answer 9

这样也不行呀还是报originalImage=(Bitmap)Bitmap.FromFile(NextFile.FullName);这句话内存不足啊

Answer 10

我Demo了一下，就算按你原来的写法，也是没有问题的。如果没有GC.Collect(),内存是会爆，加了这一句，内存是没有增加，说明GC.Collect()是释放了内存。
你的问题真的很怪异，不如先“清理解决方案”然后“从新生成解决方案”看下如何。