使用Action.Invoke被认为是最佳实践吗？

Question

如果我有以下代码，我应该只调用Action还是应该调用Action.Invoke？

public class ClassA
{
  public event Action<string> OnAdd;

  private void SomethingHappened()
  {
    if (OnAdd != null)
     OnAdd("It Happened"); //Should it be OnAdd.Invoke("It Happened") ???????
  }
}

public class ClassB
{

  public ClassB()
  {
    var myClass = new ClassA();
    myClass.OnAdd += Add;
  }

  private void Add(string Input)
  {
    //do something
  }  
}

- Jon

---

在 C# 6 中，使用新的 OnAdd?.Invoke("It Happened"); 语法可能会变得更加流行。 - Betty

对于上面的评论，再提供一些细节。该示例使用了 C# 6 的一个功能，即 Null-Conditional 操作符。? 运算符防止在 OnAdd 为空时调用 Invote 方法，从而避免异常。这使得编写代码可以放弃下面答案中显示的显式空检查。请参考 https://msdn.microsoft.com/zh-cn/magazine/dn802602.aspx 获取更多详细信息。 - Mark

这里有一些比较：https://jacksondunstan.com/articles/3283。 - Victor Yarema

Answer 1

两者是等价的，编译器会将OnAdd("It Happened");转换成OnAdd.Invoke("It Happened");。我想这只是个人偏好问题，不过我个人更喜欢更简洁的形式。

另外，通常最好在调用类级委托之前先取一个本地副本，以避免竞争条件，即在检查OnAdd是否为null时它是非空的，但在调用时它为空的情况。

private void SomethingHappened()
{
  Action<string> local = OnAdd;
  if (local != null)
  {
    local("It Happened");
  }
}

- Rich O'Kelly

---

抱歉，我在这个种族的例子中看不出区别。 - Jon

如果有两个线程参与，其中一个线程在另一个线程测试OnAdd是否为空但在调用它之前将其设置为null ... 在您的代码中，最终会出现“NullReferenceException”异常。 - Jon Skeet

Answer 2

最近我注意到在最新的C# 6版本中有一个问题，它可能会更加鼓励使用Invoke，所以我想将其添加到这个旧问题中，以防对某人有所帮助：

“旧”方式：

Action<string> doSomething = null; // or not null
if (doSomething != null)
    doSomething("test");

可能的实用方法（类似于空事件代理模式）：

Action<string> doSomethingPragmatic = s => { }; // empty - might be overwritten later
doSomethingPragmatic("test");

C# 6：

Action<string> doSomethingCs6 = null; // or not null
doSomethingCs6?.Invoke("test");

// Not valid C#:
// doSomethingCs6?("test")
// doSomethingCs6?.("test")

- jamespconnor

---

ReSharper鼓励采用新的C# 6风格。 - William Gross

Answer 3

这两个结构完全等价。

OnAdd("It Happened");

只是语法糖。在幕后，编译器会在结果的 MSIL 中发出对 Action<T>.Invoke 的调用。因此，请使用对您来说更易读的那个（对我而言，OnAdd("It Happened"); 已经足够可读了）。

- Darin Dimitrov

Answer 4

如果你没有遇到有关匿名函数的非常奇怪的错误，那么它们是完全等效的。

个人而言，我通常使用简写形式，但有时显式调用Invoke更易读。例如，你可能会这样写：

if (callAsync)
{
    var result = foo.BeginInvoke(...);
    // ...
}
else
{
    foo.Invoke(...);
    // ...
}

在这里，显式使用Invoke对称性更加有用。

有关委托调用的详细信息，请参见C# 4规范的第15.4节，尽管它没有明确指定以调用Invoke方法的术语。

- Jon Skeet

Answer 5

我更喜欢使用Invoke()，因为这样可以让我使用myAction?.Invoke()来处理可能的空引用。

- Maria Solano