应包装"Async"/"AWait&Quot;方法中的参数验证

Parameter validation in quot;asyncquot;/quot;awaitquot; methods should be wrapped(应包装quot;Asyncquot;/quot;AWaitQuot;方法中的参数验证)
本文介绍了应包装"Async"/"AWait&Quot;方法中的参数验证的处理方法,对大家解决问题具有一定的参考价值,需要的朋友们下面随着小编来一起学习吧!

问题描述

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

推荐答案

我刚刚尝试了您的示例,在您的第二个代码中没有出现问题(将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
}

如您所见,问题并未在您预期的时候提出。

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

这篇关于应包装"Async"/"AWait&Quot;方法中的参数验证的文章就介绍到这了,希望我们推荐的答案对大家有所帮助,也希望大家多多支持编程学习网!

本站部分内容来源互联网,如果有图片或者内容侵犯您的权益请联系我们删除!

相关文档推荐

DispatcherQueue null when trying to update Ui property in ViewModel(尝试更新ViewModel中的Ui属性时DispatcherQueue为空)
Drawing over all windows on multiple monitors(在多个监视器上绘制所有窗口)
Programmatically show the desktop(以编程方式显示桌面)
c# Generic Setlt;Tgt; implementation to access objects by type(按类型访问对象的C#泛型集实现)
InvalidOperationException When using Context Injection in ASP.Net Core(在ASP.NET核心中使用上下文注入时发生InvalidOperationException)
LINQ many-to-many relationship, how to write a correct WHERE clause?(LINQ多对多关系,如何写一个正确的WHERE子句?)