1

Имеется 10 сетевых папок. На каждую папку я вешаю свой FileSystemWatcher.

Задача уведомить программы, которые используют эти папки и отключить программы.

Программы могут использовать несколько папок, а так же папки могут пересекаться между программами.

Накидал вот такой код:

 public class Program
    {
        public string ProgramName { get; set; }
        public string ProgramPath { get; set; }
        HashSet<string> ProgramShares = new HashSet<string>();
        public void TernOffProgram()
        {
        //Какой-то код
        }
    }

     public class Programs
    {
        List<Program> ProgramList = new List<Program>();
        void TernOffPrograms(string sharePath)
        {
            foreach (var program in ProgramList)
            {
                var shareIsMatched = program.programShares.SingleOrDefault(share => share == sharePath) == null;
                if (shareIsMatched)
                {
                     program.TernOffProgram();
                }   
            }
        }
        public void ErrorHadler(object sender, ErrorEventArgs args)
        {
            var watcher = (FileSystemWatcher)sender;
            TernOffPrograms(watcher.Path);
        }
    }

Соответственно ErrorHadler подписывается на N событий от FileSystemObject.

Проблема такой реализации в том, что если одновременно упадет несколько сетевых папок, то ввиду того, что каждое событие обрабатывается в отдельном потоке, получится ерунда, например, обращение к одному и тому же объекту Program из разных потоков, что чревато какой-то ерундой.

11
  • А почему бы не перевести все события в один поток?
    – VladD
    2 дек 2016 в 14:13
  • @VladD, lock добавить?
    – iluxa1810
    2 дек 2016 в 14:37
  • Скорее Dispatcher.BeginInvoke если на WPF, или Control.Invoke если на WinForms
    – VladD
    2 дек 2016 в 15:04
  • @VladD я хотел это в службу затолкать. Как будут работать события в 1 поток? Пока одно событие не обработается, другое не наступит? А если во время обработки одного события наступит другое, то оно не потеряется?
    – iluxa1810
    2 дек 2016 в 15:39
  • 1
    @Monk, опечатался малость, там Program. Проблема в том, что будет дергаться одна и та же коллекция из разных потоков.
    – iluxa1810
    3 дек 2016 в 21:29

1 ответ 1

1

Если я правильно понял, при отключении одной любой шары программу надо выключить.

Предлагаю дополнить класс Program вот так:

private int isEnabled = 0;

public void TernOffProgram()
{
  var old = System.Threading.Interlocked.CompareExchange(ref isEnabled, 0, 1);
  if (old == 0)
    return;

  //Какой-то код
}

На запуске наоборот ставить единицу аналогично, т.к. скорее всего по тому же событию будет запуск у вас.

Ваш ответ

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

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