参数验证" async" /" await"方法应该包装好

时间:2018-06-12 14:19:46

标签: c# sonarqube

为什么SonarQube在此代码中抱怨?我读了解释,但是我真的理解为什么以及我需要做什么,让它消失。

Link to SonarQube Rule

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

1 个答案:

答案 0 :(得分:2)

我刚刚尝试了您的示例,但您的第二个代码未提出此问题(将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
}

正如您所见,问题没有出现。

显然,如果你确定你不会在这种情况下结束,并且没有人会以这种方式使用你的方法,你就可以关闭规则。