事件处理程序/筹集者代码段 - c#

我想听听对以下代码段的意见。有什么可以改善的吗?事件处理程序/引发程序是否遵循最佳做法命名?我知道在同一个类中处理和引发事件不是很有用,但这只是一小段。

public class MyControl
{
    public MyControl()
    {
        this.LogWritten += this.HandleMyControlLogWritten;
    }

    // Event handler
    void HandleMyControlLogWritten(object sender, EventArgs e)
    {
    }

    // Event object
    public event Action<object, EventArgs> LogWritten;

    // Event raiser
    protected virtual void OnLogWritten(EventArgs e)
    {
        if (this.LogWritten != null)
        {
            this.LogWritten(this, e);
        }
    }
}

参考方案

我建议的主要更改是获取事件处理程序的副本:

// Event raiser
protected virtual void OnLogWritten(EventArgs e)
{
    var handler = this.LogWritten;
    if (handler != null)
    {
        handler(this, e);
    }
}

如果您打算(最终)在多线程方案中使用此类,则这一点很重要。因此,我发现养成使用习惯是一个好的“最佳实践”。问题是,在多个线程中使用时,如果不创建副本,则附加的唯一“处理程序”可能会在空检查和调用之间退订,这将导致运行时错误。通过复制到临时变量(var handler = this.LogWritten;)行,可以有效地创建订户列表的“快照”,然后检查它是否为null并在需要时调用。

另一个更改是事件声明本身。代替使用Action<T1,T2>

// Event object
public event Action<object, EventArgs> LogWritten;

我建议使用EventHandler<TEventArgs>(如果要使用自定义的EventArgs子类)或EventHandler(对于标准EventArgs)。这些是“标准做法”,将是其他开发人员期望的:

// Event object
public event EventHandler LogWritten;

LeetCode题解计算机为什么是基于二进制的?

可以是三进制么?二进制有什么好处?题解:为什么叫电子计算机?算盘应该没有二进制

LeetCode题解统计城市的所有灯泡

这个是我刚毕业的时候,一个真实的面试题,这是一个开放题。题目描述:想办法,将一个城市的所有灯泡数量统计出来。题解:费米估算法1、如果某个城市常驻人口有1000万2、假设每5人居住在一套房里,每套房有灯泡5只,那么住宅灯泡共有1000万只3、假设公众场所每10人共享一只灯泡,那么共有100万只4、主要的这两者相加就得出了1100万只当然实际上这是估算的,具体应…

LeetCode题解黑白圆盘

一个圆盘被涂上了黑白二色,两种颜色各占一个半圆。圆盘以一个未知的速度、按一个未知的方向旋转。你有一种特殊的相机可以让你即时观察到圆上的一个点的颜色。你需要多少个相机才能确定圆盘旋转的方向?题解:可以用一个相机即可

LeetCode题解圆上任取三点构成锐角三角形的概率

来自字节跳动的一道几何题题解:1/4

LeetCode题解深度优先遍历和回溯的关系?

深度优先遍历的范围更大还是回溯的范围更大?为什么?题解:我的理解是:dfs是回溯思想的一种体现- 回溯:是在整个搜索空间中搜索出可行解,在搜索过程中不断剪枝回退,这是回溯的思想,这个搜索空间并没有限制于特定的数据结构。- dfs:dfs是指特定的数据结构中如图,树(特殊的图)中搜索答案,范围限制在了特定的数据结构。个人拙见。