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 you have read our privacy policy.

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