Winform ListViewItem Проверка состояния переключения

Question

У меня проблема с условиями гонки с winforms.

  listViewCollections.ItemChecked += foo //Somewhere

  void foo(object sender, ItemCheckedEventArgs e)
    {
        if (e.Item.Checked == false) return;
        bool flag = bar( e.Item.Index);
        if (flag) {
            MessageBox.Show( ... )
            e.Item.Checked = false;
       }
    }

Иногда окно сообщения отображается дважды. Какой правильный способ сделать это? Заранее спасибо!

Answer 1

Не могли бы вы просто поставить вокруг него простой замок?Также я бы предложил переключить способ работы проверенной логики, чтобы немного упростить функцию (убрать возврат средней функции)

private static object fooLock = new object();
void foo(object sender, ItemCheckedEventArgs e)
{
    lock (fooLock)
    {
        if (e.Item.Checked) 
        {
            if (bar(e.Item.Index)) 
            {
                MessageBox.Show( ... )
                e.Item.Checked = false;
            }
        }
    }
}

Многочисленные способы улучшить производительность, но это должно как минимум остановить условия гонки