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

Answer 1

两者是等价的，编译器会将OnAdd("It Happened");转换为OnAdd.Invoke("It Happened");。

我想这是一个偏好问题，但我个人更喜欢这种格式。

顺便说一下，通常最好在调用它之前获取类级别委托的本地副本，以避免在检查时OnAdd不为空的竞争条件，但是在它被调用的时间：

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

Answer 2

这两种结构完全等效。

OnAdd("It Happened");

只是语法糖。在后台，编译器在生成的MSIL中发出对Action<T>.Invoke的调用。所以使用一个更易读的（对我来说OnAdd("It Happened");足够可读）。

Answer 3

我在最新的C＃6版本中发现了一些内容，因为它可能会鼓励Invoke更多地使用它，并且我认为我会将其添加到这个旧问题中，以防有人帮助：

＆＃34;旧＆＃34;方式：

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")

Answer 4

除非您遇到very strange bug around anonymous functions，否则它们完全等效。

我个人通常使用快捷方式表单，但有时它最终会更明显地调用Invoke。例如，您可能有：

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

这里明确使用Invoke对于对称非常有用。

有关委托调用的更多详细信息，请参阅C＃4规范的第15.4节，尽管它没有在调用Invoke方法时明确指定它。

是否使用Action.Invoke被认为是最佳做法？

4 个答案: