c# - 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

你能不能在它周围加一把简单的锁？此外，我建议切换 Checked 逻辑的工作方式以稍微简化函数(删除中间函数返回)

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;
            }
        }
    }
}

提高性能的方法有很多，但至少应该停止竞争条件？