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

为什么 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);

}


慕村9548890
浏览 211回答 1
1回答

慕丝7291255

我只是想你的例子,这个问题不提高你的第二个代码(替换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}正如您所看到的,问题并未在您期望的时候提出。显然,如果您确定自己永远不会遇到这种情况,并且没有人会以这种方式使用您的方法,那么您可以关闭该规则。
打开App,查看更多内容
随时随地看视频慕课网APP