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

Question

为什么SonarQube在这段代码中抱怨？我读了解释，但并不真正理解为什么以及我需要做什么，让它消失。

链接到声纳Qube规则

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
}

如您所见，问题并未在预期时提出。

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