12

我们目前正在讨论 .NET 中的扩展方法是否不好。或者在什么情况下扩展方法会引入难以发现的错误或以任何其他方式出现意外行为。

我们想出了：

  • 为不受您控制的类型编写扩展方法（例如，使用 GetTotalSize() 扩展 DirectoryInfo 等）是不好的，因为 API 的所有者可能会引入隐藏我们扩展的方法 - 并且可能有不同的边缘情况. 例如，如果由于隐藏而不再使用扩展方法，则在扩展方法中测试 null 将自动转换为 NullReferenceException。

问题：

  • 除了“隐藏”之外，还有其他我们没有想到的危险情况吗？

编辑：

另一个非常危险的情况。假设你有一个扩展方法：

namespace Example.ExtensionMethods
{
    public static class Extension
    {
        public static int Conflict(this TestMe obj)
        {
            return -1;
        }
    }
}

并使用它：

namespace Example.ExtensionMethods.Conflict.Test
{
    [TestFixture]
    public class ConflictExtensionTest
    {
        [Test]
        public void ConflictTest()
        {
            TestMe me = new TestMe();
            int result = me.Conflict();
            Assert.That(result, Is.EqualTo(-1));
        }
    }
}

请注意，您使用它的名称空间更长。

现在你用这个引用一个dll：

namespace Example.ExtensionMethods.Conflict
{
    public static class ConflictExtension
    {
        public static int Conflict(this TestMe obj)
        {
            return 1;
        }
    }
}

你的测试将失败！它将在没有编译器错误的情况下编译。它只会失败。您甚至不必指定“使用 Example.ExtensionMethods.Conflict”。编译器将遍历命名空间名称并在 Example.ExtensionMethods.Extension 之前找到 Example.ExtensionMethods.Conflict.ConflictExtension 并将使用它而不会抱怨不明确的扩展方法。哦，恐怖！

4

8 回答 8

9

一些好奇心：

  • 可以在null实例上调用扩展方法；这可能会令人困惑（但有时很有用）
  • 如果他们有不同的意图，“隐藏”问题是一个大问题
  • 同样，您可能会从 2 个不同的命名空间中获得具有相同名称的不同扩展方法；如果您只有两个命名空间之一，这可能会导致不一致的行为（取决于哪个）...
  • ...但是如果有人在您的代码使用的第二个命名空间中添加类似（相同签名）的扩展方法，它将在编译时中断（不明确）

（编辑）当然，还有“ Nullable<T>/ new()”炸弹（见这里）......

于 2009-03-04T14:46:29.547 回答
5

我不同意，扩展方法的全部意义在于将您的成员添加到黑盒类中。与其他所有事物一样，存在陷阱，您必须注意命名、实施并了解方法的优先顺序。

于 2009-03-04T15:02:58.757 回答
4

我们刚刚在MoreLINQ项目中发现了一个缺陷：如果您编写一个通用扩展方法，则无法确保它适用于所有类型。我们有一个带有这个签名的方法：

public static IEnumerable<T> Concat<T>(this T head, IEnumerable<T> tail)

您不能将其用于：

"foo".Concat(new [] { "tail" });

因为string.Concat方法...

于 2009-03-04T14:52:51.200 回答
2

我使用 Ruby on Rails 的时间几乎与使用 C# 的时间一样长。Ruby 允许您执行类似于新扩展方法的操作。当然，如果有人将方法命名为相同，则存在潜在的问题，但是能够将方法添加到封闭类的优势远远超过潜在的劣势（这可能是由糟糕的设计或糟糕的计划造成的）。

于 2009-03-10T12:36:47.670 回答
2

确保扩展方法不与其他方法（扩展或其他方法）冲突的一件事是将FxCop与诸如防止重复扩展方法签名之类的规则一起使用。

于 2009-08-30T17:31:52.690 回答
1

.Net 调用的扩展方法也是一种有限的MonkeyPatching形式（尝试忽略其中的 php 咆哮）。

那应该为您的讨论提供一些材料。

于 2009-03-04T14:50:46.743 回答
1

首先，我认为您的措辞有点误导。我认为您在谈论“类型”而不是“对象”。

其次，扩展方法的一大优势是你可以为你无法控制的类型添加特性。如果你控制了类型，为什么不直接修改类型而不是依赖扩展方法呢？

于 2009-03-04T14:53:40.213 回答
1

我们在我的团队中采取的态度是扩展方法非常有用，以至于您实际上无法禁止它们，但又非常危险（主要是因为隐藏问题），因此您必须保持谨慎。所以我们决定所有扩展方法名称都必须加上前缀X（例如，我们有一堆XInit...()方法可以以有用的方式初始化控件）。这样 a) 命名冲突的可能性降低了 b) 程序员知道他使用的是扩展方法而不是类方法。

于 2011-05-17T17:33:23.007 回答