我想听听对以下代码段的意见。有什么可以改善的吗?事件处理程序/引发程序是否遵循最佳做法命名?我知道在同一个类中处理和引发事件不是很有用,但这只是一小段。
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是指特定的数据结构中如图,树(特殊的图)中搜索答案,范围限制在了特定的数据结构。个人拙见。