0

Собственно, уменьшаю и обрезаю изображения до 200*300 в цикле и выдает ошибку "System.OutOfMemoryException" в System.Drawing.dll Если цикл убрать то все хорошо работает. img_files массив выбранных фото. В каком направлении копать?

if (img_files != null)
        {
            int nWidth, nHeight;
            Image img;
            for (photo_num = 0; photo_num < img_files.Length; photo_num++)
            {

                img = Image.FromFile(img_files[photo_num]);
                if (img.Width / img.Height >= 1.5)
                {
                    nHeight = 200;
                    nWidth = 200 * img.Width / img.Height;
                }
                else
                {
                    nWidth = 300;
                    nHeight = 300 * img.Height / img.Width;
                }
                Bitmap img1 = new Bitmap(img, nWidth, nHeight);
                img1 = img1.Clone(new Rectangle(0, 0, 300, 200), img1.PixelFormat); // тут выдает ошибку
                img1.Save("d:\\pyt\\k\\filu", System.Drawing.Imaging.ImageFormat.Jpeg);
                img1.Dispose();
                img.Dispose();



            }
        }
5
  • 1
    Не диспозится самый первый img1, который создаётся вызовом new Bitmap. Полагаю, можно ввести дополнительную переменную img2. Потом диспозить все три. 11 дек 2015 в 15:26
  • Я так понимаю это и предложил Igor, но не помогло 11 дек 2015 в 18:28
  • 1
    На каком по счёту изображении вылетает исключение? Каков изначальный размер фоток? 11 дек 2015 в 18:40
  • Действительно, с этим и проблема. Ширину выдает 199,997 естественно округляется до 199, и компилятор выдает верную ошибку. Сам код был верный, как я понимаю. Причина в округлении переменных 11 дек 2015 в 19:10
  • А Без цикла работало, потому что использовал другую фотку. 11 дек 2015 в 19:11

2 ответа 2

2

Потеряна первая ссылка img1.

using(Bitmap img1 = new Bitmap(img, nWidth, nHeight))
{
  using (Bitmap img2 = img1.Clone(new Rectangle(0, 0, 300, 200), img1.PixelFormat))
  {
     img2.Save("d:\\pyt\\k\\filu", System.Drawing.Imaging.ImageFormat.Jpeg);
  }
}

PS: Выбирайте для переменных осмысленные имена.

2
  • Спасибо) Думал если переменная используется только в маленьком кусочке, то какая разница? 11 дек 2015 в 18:22
  • Не помогло, выдает ту же ошибку на том же месте (на img2) (( Ребят, помогите, не знаю с чем связано(((( 11 дек 2015 в 18:26
0

Беда в строчке (img.Width / img.Height >= 1.5) Компилятор округлит результат деления до целого значения 1. Условия будет выполнятся не верно и фотография сожмется по неправильному алгоритму. Далее когда фото будет обрезаться не получится построить Rectangle(0, 0, 300, 200) и вылетает ошибка

2
  • (img.Width >= 1.5 * img.Height)
    – user176262
    11 дек 2015 в 19:27
  • Вариант) Но я просто назначил img.Width и img.Height на переменные float 11 дек 2015 в 19:40

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.