ArgumentNullException、索引器和宪兵规则错误

Question

我有索引器，想要检查它是否不为空，如果不为空，则抛出 ArgumentNullException，但 Gendarme 设置警告

InstantiateArgumentExceptionCorrectlyRule：此方法抛出 ArgumentException （或派生）异常而不指定现有的参数名称。这可以隐藏对开发人员有用的信息。修复异常参数以使用正确的参数名称（或确保参数的顺序正确）。

public override LocalizedString this[string key]
{
    get
    {
        if (key == null)
        {
            throw new ArgumentNullException("key");
        }
        return base[key];
    }
    set
    {
        if (key == null || value == null)
        {
            throw new ArgumentNullException("key");
        }
        base[key] = value;
    }
}

如何修复我的索引器？

Answer 1

嗯，目前肯定是不对的。看看这个：

if (key == null || value == null)
{
    throw new ArgumentNullException("key");
}

这意味着它会抛出一个异常，声称“key”为 null，而实际上它应该是“value”。

所以代码应该如下所示：

if (key == null)
{
    throw new ArgumentNullException("key");
}
if (value == null)
{
    throw new ArgumentNullException("value");
}

我不知道这是否会修复警告，但这将是正确的代码。

此错误报告表明这是一个错误宪兵尚未修复。如果您可以明确禁用该索引器的警告，这可能是最好的方法。 （我没有使用过宪兵，所以我不知道这是否可行，但值得研究一下。）