0

Приветствую. Ситуация такая : есть несколько форм, на одну из форм динамически подгружаются картинки с тканями. в процессе работы можно удалить одну из картинок. реализовано это так - если нажать на картинку вылетит новая форма с той же картинкой большего размера и кнопкой удалить. при нажатии на нее имя картинки заносится в отдельный класс а при закрытии формы подгружается в Main_FormClosed. Код :

try
{
    string CompletePatchToEXE = Directory.GetCurrentDirectory();
    DirectoryInfo dir = new DirectoryInfo(CompletePatchToEXE + @"\\materials\\");
    string delNAME = OrderData.deletFiles[0].ToString() + ".jpg"; 

    foreach (FileInfo file in dir.GetFiles())
    {
        if (file.Name == delNAME) 
        {
            file.Delete();
        }
    }
}
catch (Exception ex)
{
    MessageBox.Show(ex.Message, ex.Source);
}

А проблема в том, что указанный файл все еще используется приложением и удалить его оно не может. Как быть, подскажите? Спасибо.

Файлы картинок динамически загружаются из директории @"materials/" и присваиваются как фоновые изображения для пикчеров вот так:

PB.BackgroundImage = Image.FromFile("materials/" + ListName[i] + ".jpg");
8
  • Нужно использовать using при работе с файлом. Не блокировать файл на продолжительный момент. Можно написать две-три попытки удалить файл задействав ILDE и ожидание в 1 сек. Можно процедуры чтения вынести в static-методы и задействовать lock что б избежать "многозадачных" конфликтов.
    – nick_n_a
    Commented 7 апр. 2017 в 8:40
  • можно подробнее про using ? какую строку обернуть? Commented 7 апр. 2017 в 8:43
  • Строку где вы читаете файл. Например using ( File f = File.Open()) {};
    – nick_n_a
    Commented 7 апр. 2017 в 8:52
  • Вы не совсем поняли. файлы картинок динамически загружаются из директории @"materials/" и присваиваются как фоновые изображения для пикчеров вот так : PB.BackgroundImage = Image.FromFile("materials/" + ListName[i] + ".jpg"); Commented 7 апр. 2017 в 8:58
  • 1
    вот так static void DoDeleteFiles() { try { .... file.Delete } } и так static Image DoLoadImage(string filename) { return Image.FromFile(filename); }
    – nick_n_a
    Commented 7 апр. 2017 в 9:40

1 ответ 1

3

В комментарии у вас написано:

PB.BackgroundImage = Image.FromFile("materials/" + ListName[i] + ".jpg");

Изображение загружается и присваивается свойству BackgroundImage. Здесь-то и удерживается его хендл. Чтобы изображение можно было удалить, нужно освободить его хендл.

Например, это можно сделать так:

Image imageCopy;

using (var imageSource = Image.FromFile("materials/" + ListName[i] + ".jpg")
    imageCopy = new Bitmap(imageSource);

PB.BackgroundImage = imageCopy;

В данном коде создаётся копия изображения - его хендл не будет привязан к файлу. А при выходе из блока using будет вызван метод Dispose, который освободит ресурс - файл оригинального изображения. После чего его можно удалять.

P.S. Само удаление выполняется одной строкой:

new FileInfo(delNAME).Delete();

Цикл не нужен.

1
  • Спасибо. все заработало Commented 7 апр. 2017 в 10:50

Ваш ответ

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

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