C# 编译器选择了错误的扩展方法

Question

考虑这段代码:

using System.Linq;

namespace ExtensionMethodIssue
{
    static class Program
    {
        static void Main(string[] args)
        {
            var a = new[] { 1 };
            var b = new[] { 1, 2 }.Where(a.Contains).ToList();
            var c = new[] { 1, 2 }.Where(i => a.Contains(i)).ToList();
        }
    }
}

代码编译成功。然后我添加了 nuget 包“itext7 7.0.4”，现在编译失败，因为:

//Error CS0122: 'KernelExtensions.Contains<TKey, TValue>(IDictionary<TKey, TValue>, TKey)' is inaccessible due to its protection level
var b = new[] { 1, 2, 3 }.Where(a.Contains).ToList();

// This is still ok.
var c = new[] { 1, 2, 3 }.Where(i => a.Contains(i)).ToList();

原因是 itext7 库在 global 命名空间 ( here it is ) 中有一个带有扩展方法的 internal 类。

internal static class KernelExtensions {
    public static bool Contains<TKey, TValue>(this IDictionary<TKey, TValue> dictionary, TKey key) {
        return dictionary.ContainsKey(key);
}
}

出于某种原因，编译器选择了具有全局命名空间中不兼容签名的不可访问扩展方法，而不是具有来自 LINQ 命名空间的兼容签名的可访问扩展方法。

问题是:这种行为是语言规范所期望的还是编译器中的错误？为什么它只在有“方法组”的情况下失败并且仍然可以使用 i => a.Contains(i)？

Answer 1

那是一个编译器错误，一个不可访问的函数不应该影响重载解析。

作为一般解决方法，使用 lambdas 作为参数而不是方法组，因为重载解析似乎可以很好地与它们一起工作。在您的特定场景中哪个更快/更高效并不明显，根据需要使用相关性能指标进行微优化。

在这种特殊情况下，您还可以使用其他扩展方法，例如 Enumerable.Intersect()如果您正在使用集合，并且不担心重复，Enumerable.Join()或一个简单的循环。

更多信息请查看:

• C# compiler chooses inaccessible extension method over accessible one GitHub 上的 Roslyn 问题
• Is there any overhead in the use of anonymous methods?关于 StackOverflow 的讨论
• Is a LINQ statement faster than a 'foreach' loop?关于 StackOverflow 的讨论
• For vs. Linq - Performance vs. Future关于 StackOverflow 的讨论