CA1062:协同构造函数调用上的 ValidateArgumentsOfPublicMethods

发布于 2024-07-25 13:31:48 字数 606 浏览 7 评论 0原文

我有一个包含两个构造函数的类,如下所示:

public MyClass(SomeOtherClass source) : this(source, source.Name) { }
public MyClass(SomeOtherClass source, string name) { /* ... */ }

当我运行 FxCop 时,它正确报告违反 CA1062: ValidateArgumentsOfPublicMethods,因为如果第一个构造函数中的 sourcenull,它将在 NullReferenceException 上抛出 NullReferenceException代码>源.名称。

有什么办法可以解决这个警告吗?

我可以创建一个扩展方法来检查 null 并返回其参数,但它会很难看。 另外,据我了解,它不会解决警告,因为 FxCop 不会意识到它的作用。

I have a class with two constructors that look like this:

public MyClass(SomeOtherClass source) : this(source, source.Name) { }
public MyClass(SomeOtherClass source, string name) { /* ... */ }

When I run FxCop, it correctly reports a violation of CA1062: ValidateArgumentsOfPublicMethods, because if source is null in the first constructor, it will throw a NullReferenceException on source.Name.

Is there any way to fix this warning?

I could make an extension method that checks for null and returns its argument, but it would be ugly. Also, as I understand, it wouldn't resolve the warning because FxCop wouldn't realize what it does.

如果你对这篇内容有疑问,欢迎到本站社区发帖提问 参与讨论,获取更多帮助,或者扫码二维码加入 Web 技术交流群。

扫码二维码加入Web技术交流群

发布评论

需要 登录 才能够评论, 你可以免费 注册 一个本站的账号。

评论(5

谢绝鈎搭 2024-08-01 13:31:48

像这样?

public MyClass(SomeOtherClass source) : this(source, source == null ? null : source.Name) { }
public MyClass(SomeOtherClass source, string name) { /* ... */ }

Like this?

public MyClass(SomeOtherClass source) : this(source, source == null ? null : source.Name) { }
public MyClass(SomeOtherClass source, string name) { /* ... */ }
围归者 2024-08-01 13:31:48

有合法的时间来关闭 FxCop 警告,这很可能就是一个,但是您可以通过检查 null 并引发异常(或替换默认值)的三元表达式或调用静态来纠正问题检查 null 并抛出适当异常的方法。

There are legitimate times to turn off FxCop warnings and this could very well be one, but you can correct the problem by either a ternary expression that checks for null and throws an exception (or substitutes a default value), or a call to a static method that checks for null and throws the appropriate exception.

伴梦长久 2024-08-01 13:31:48

由于这个问题是前一段时间提出的,我只想注意 C# 中的后续功能,您现在也可以使用它:

public MyClass(SomeOtherClass source) : this(source, source?.Name) { }

Since this question was asked some time ago I just want to note with the later features in C# you can now also use this:

public MyClass(SomeOtherClass source) : this(source, source?.Name) { }
你是我的挚爱i 2024-08-01 13:31:48

从 C# 7.0 开始,您还可以执行以下操作:

public MyClass(SomeOtherClass source) : this(source?.Name ?? throw new ArgumentNullException(nameof(source))) { }

C# 7.0 允许将异常作为表达式抛出。 (请参阅 Microsoft 文档

Since C# 7.0 you can also do this:

public MyClass(SomeOtherClass source) : this(source?.Name ?? throw new ArgumentNullException(nameof(source))) { }

C# 7.0 allows throwing exceptions as expressions. (See Microsoft Docs)

薄荷港 2024-08-01 13:31:48

我想说解决此警告的唯一方法是将其关闭。 FxCop 是一个很棒的工具,但有时您需要记住它只是一个工具,并且可以提出并不总是适合您的代码的建议。

在此示例中,如果您不想看到它,我会说忽略警告或禁用它。

I would say the only way to fix this warning would be to turn it off. FxCop is a great tool but sometimes you need to remember that it is just a tool and can make suggestions that are not always fitting to your code.

In this example I would say ignore the warning or disable it if you don't want to see it.

~没有更多了~
我们使用 Cookies 和其他技术来定制您的体验包括您的登录状态等。通过阅读我们的 隐私政策 了解更多相关信息。 单击 接受 或继续使用网站,即表示您同意使用 Cookies 和您的相关数据。
原文