应包装“async”/“await”方法中的参数验证

Question

为什么 SonarQube 在这段代码中抱怨？我阅读了解释，但并不真正理解为什么以及我需要做什么才能让它消失。public async Task Add(SomeModel obj){    if (obj == null)    {        throw new ArgumentNullException(nameof(obj));    }    var obj2 = new OtherObject();    obj2.UpdateWith(obj);    await _localDatabaseService.AddAsync(obj2);}将代码更改为这样并不能解决问题。public Task Add(SomeModel obj){    if (obj == null)    {        throw new ArgumentNullException(nameof(obj));    }    return AddInternal(obj);}private async Task AddInternal(SomeModel obj){    var obj2 = new OtherObject();    obj2.UpdateWith(obj);    await _localDatabaseService.AddAsync(i);}

Answer 1

我只是想你的例子，这个问题不提高你的第二个代码（替换AddAsync(i)用AddAsync(obj2)匹配第一代码逻辑。

关于规则的解释，我认为该网站非常清楚，但让我提供一个代码示例来尝试说明错误的行为。

static async void Main(string[] args)

{

    var x = new Program().Add(null); // Exception is not raised here...

    // do some other things

    await x; // ... but here when awaited

}

正如您所看到的，问题并未在您期望的时候提出。

显然，如果您确定自己永远不会遇到这种情况，并且没有人会以这种方式使用您的方法，那么您可以关闭该规则。