0

Помогите решить данную проблему в коде

public void draw(int x,int y)
{
    var map = new Bitmap("Result.png");
    map.SetPixel(x0+x, y0+y, Color.Black);            
    map.Save("Result.png", System.Drawing.Imaging.ImageFormat.Png);
    map.Dispose();
}

Необработанное исключение типа

"System.Runtime.InteropServices.ExternalException" в System.Drawing.dll Дополнительные сведения: В GDI+ возникла ошибка общего вида.

StackTrace:

в System.Drawing.Image.Save(String filename, ImageCodecInfo encoder, EncoderParameters encoderParams)
в System.Drawing.Image.Save(String filename, ImageFormat format)
в Encryption.Form1.draw(Int32 x, Int32 y) в C:\Users\Ivang\documents\visual studio 2015\Projects\Encryption\Encryption\Form1.cs:строка 44
в Encryption.Form1.library(Char symb) в C:\Users\Ivang\documents\visual studio 2015\Projects\Encryption\Encryption\Form1.cs:строка 54
в Encryption.Form1.action() в C:\Users\Ivang\documents\visual studio 2015\Projects\Encryption\Encryption\Form1.cs:строка 191
10
  • 2
    Попробуйте переместить строчку map.Dispose(); на одну ниже. Т.е. map.Save("Result.png", System.Drawing.Imaging.ImageFormat.Png);map.Dispose();
    – user227049
    Commented 8 дек. 2016 в 13:56
  • 1
    @FoggyFinder не решило проблему
    – ZOOM SMASH
    Commented 8 дек. 2016 в 14:35
  • 1
    @ZOOMSMASH, вставь stacktrace и полный текст ошибки, а не картинку
    – Grundy
    Commented 8 дек. 2016 в 14:36
  • 2
    ну и не исключено, что ты просто не можешь сохранить в уже открытый файл. Попробуй сменить имя в Save, например на Result2.png.
    – Grundy
    Commented 8 дек. 2016 в 14:37
  • 1
    не обязательно, но сначала нужно определиться с причиной текущей ошибки, и потом разбираться со следующей ошибкой
    – Grundy
    Commented 8 дек. 2016 в 14:40

3 ответа 3

4
public void draw(int x,int y)
{
    var map = new Bitmap("Result.png");
    map.SetPixel(x0+x, y0+y, Color.Black);            
    // !!! В этом месте файл всё ещё заблокирован!
    // !!! И таким останется до уничтожения битмапа
    // !!! Сейчас в него писать нельзя
    //map.Save("Result.png", System.Drawing.Imaging.ImageFormat.Png);
    map.Dispose();
    // !!! Теперь в файл писать можно
    // !!! Но писать уже нечего, т. к. картинка уничтожена
}

Возможные варианты решения (выбрать любой):

  • Считать файл в память и создать Bitmap из MemoryStream
  • Создать другой bitmap того же размера и нарисовать первый на нём
    Тут надо быть осторожнее с форматом пикселей
  • Записать в другой файл, потом удалить первый и переименовать второй

А ещё вместо явного вызова Dispose следует использовать блок using.

1

Для начала нужно понять, что происходит в вашем коде. Там всего четыре строчки, поэтому это не трудно.

  1. Вы создаете объект. Загружаете в него изображение из файла
  2. Вы изменяете этот объект.
  3. Вы удаляете объект из памяти.
  4. Вы пытаетесь сохранить объект, который уже удалил.

Очевидно, что вам требуется поменять местами строки 3 и 4

1
  • К сожалению, не помогло
    – ZOOM SMASH
    Commented 8 дек. 2016 в 14:35
1
public void draw(int x,int y)
{
  Bitmap map2 = null;
  try
  {
    using (Bitmap map1 = new Bitmap("Result.png"))
    {
      map2 = (Bitmap)map1.Clone();
    }

    map2.SetPixel(x0+x, y0+y, Color.Black);            
    map2.Save("Result.png", System.Drawing.Imaging.ImageFormat.Png);
  }
  finally
  {
    if (map2 != null)
      map2.Dispose();
  }
}
5
  • лучше уж using использовать. И для второго - тоже :)
    – Grundy
    Commented 8 дек. 2016 в 14:55
  • @Grundy не, дла второго - не лучше, эти usingы пересекаются
    – user176262
    Commented 8 дек. 2016 в 14:56
  • ну и что? и что значит пересекаются? один просто внутри другого - это нормально. Нарисовав try...finally - ты просто сразу раскрыл второй using
    – Grundy
    Commented 8 дек. 2016 в 14:59
  • @Grundy не вложены, а пересекаются
    – user176262
    Commented 8 дек. 2016 в 14:59
  • тогда по твоей логике у тебя сейчас using пересекается с try...finally :) и кстати скобку у try забыл закрывающую
    – Grundy
    Commented 8 дек. 2016 в 15:00

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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