11

Мониторинг файла. При изменениях в файле почему-то дважды запускается Change. Как можно это решить?

Вот код:

public void monitoring(string path)
{
    this.sPath = path;
    string first = Path.GetDirectoryName(path);
    string second = Path.GetFileName(path);
    FileSystemWatcher fsw = new FileSystemWatcher(first, second);
    fsw.Changed += new FileSystemEventHandler(watcher_Changed);
    fsw.EnableRaisingEvents = true;
}

public void watcher_Changed(object sender, FileSystemEventArgs e)
{
    MessageBox.Show("Be happy :3");
    SplitAndQuery(this.sPath);
}
2
  • 2
    А вы не исключаете, что событие таки случилось дважды? Например, файл удалён и снова создан? Попробуйте залогировать е.ChangeType.
    – VladD
    10 авг 2016 в 11:28
  • ассоциация:stackoverflow.com/questions/1764809/…
    – Bald
    10 авг 2018 в 7:13

4 ответа 4

13

Дело в том, что FileSystemWatcher опирается на уведомления от ОС. А запись изменений в файл далеко не всегда может являться атомарным действием: например, notepad.exe пишет на диск в несколько приемов, отдельно файл и отдельно его атрибуты, что повлечет за собой несколько срабатываний события Changed. Или при копировании/перемещении файлов запись происходит в несколько приемов.

Избавиться от этого можно попробовать разными способами. Самый железобетонный, но не всегда подходящий -- отключать генерацию событий. Но пользоваться им надо аккуратно, т.к. можно пропустить другие полезные изменения.

public void watcher_Changed(object sender, FileSystemEventArgs e)
{
    watcher.EnableRaisingEvents = false;
    try
    {
        MessageBox.Show("Be happy :3");
        SplitAndQuery(this.sPath);
    }
    finally
    {
        watcher.EnableRaisingEvents = true;
    }
}

Другой подход -- опираться на дату последней модификации файла (опционально еще и на e.ChangeType):

DateTime lastWriteTime = DateTime.MinValue;

public void watcher_Changed(object sender, FileSystemEventArgs e)
{
    DateTime writeTime = File.GetLastWriteTime(this.sPath);
    if (lastWriteTime != writeTime)
    {
        MessageBox.Show("Be happy :3");
        SplitAndQuery(this.sPath);
        lastWriteTime = writeTime;
    }
}

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

В общем, выбор решения целиков зависит от ваших требований (впрочем, как обычно :)).

2
  • 1
    большое спасибо, помогло
    – YoFi
    10 авг 2016 в 12:24
  • @YoFi не за что!
    – andreycha
    10 авг 2016 в 12:25
1

Никак, просто имейте в виду, что событие срабатывает дважды. Это нормально для FileSystemWatcher.

Конкретные решения придётся принимать по ситуации. Я обычно держу в классах информацию, которая мне помогает понять, надо обрабатывать текущее событие changed или нет - последнее время изменения, текущее местоположение файла, наличие файла и т.д.

1
  • далеко не всегда событий 2, у меня для одного экземпляра прилетает 5 событий для другого 3
    – Bald
    10 авг 2018 в 7:01
1

Решение со счетчиком опасно, т.к. игнорирует возможные другие изменения.

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

private FileSystemWatcher _watcher = new FileSystemWatcher();

public void Start() 
{
    ....
    _watcher.Changed += _watcher_Changed;
    _watcher.EnableRaisingEvents = true;
}

private void _watcher_Changed(object sender, FileSystemEventArgs e)
{
    new Thread(ReloadData).Start(); // запускаем в отдельном потоке, для отсечения очереди событий при многоэтапной записи в файл
}

private void ReloadData()
{ 
    // блокируем выполнение для пропуска нескольких этапов записи файла
    if (Monitor.TryEnter(_watcher))
    {
        try
        {
            Thread.Sleep(3000); // ждём 3 сек пока файл сохраняется
        }
        finally
        {
            Monitor.Exit(_watcher);
        }
    }
    else
    {
        return; // все остальные входы в течении 3 секунд будут игнорироваться
    }
    // через 3 секунды происходит реакция
    Execute();
}
8
  • 1
    Не надо плодить столько потоков. Решение с отложенной обработкой можно сделать гораздо лучше.
    – Qwertiy
    10 фев 2017 в 15:46
  • Так какое именно решение? 11 фев 2017 в 19:52
  • Запустить таймер на 3 секунды и игнорировать события, которые происходят в эти 3 секунды. При желании можно так же сбрасывать таймер, чтобы отсчёт 3 секунд начинался заново.
    – Qwertiy
    11 фев 2017 в 19:56
  • в целом да, будет компактнее, но таймер это тоже поток... 13 фев 2017 в 7:30
  • Во-первых, системный таймер - это не поток. Он при возникновении события просто кинет сообщение в очередь. Во-вторых, если бы он даже был потоком, он был бы одним потоком. А в приведённом варианте на каждое событие создаётся новый поток. Т. е. если в течение 3 секунд произойдёт 10 событий, то будет 10 потоков, а не 1.
    – Qwertiy
    13 фев 2017 в 7:33
-1

Нашел решение которое мне подходит, но не факт что подойдет всем:

public void watcher_Changed(object sender, FileSystemEventArgs e)
        {
            changeCount++;
            if (changeCount == 1)
            {
                MessageBox.Show("Be happy :3");
                SplitAndQuery(this.sPath);
            }
            else
            {
                changeCount = 0;
                MessageBox.Show("Be very happy, its works");
            }
        }
2
  • 2
    Пропускать каждый второй - это не очень хорошо.
    – Qwertiy
    10 фев 2017 в 15:46
  • далеко не всегда событий 2, у меня для одного экземпляра прилетает 5 событий для другого 3
    – Bald
    10 авг 2018 в 7:23

Ваш ответ

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

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