空接口有代码味道吗？

Question

我有一个函数返回相同类型的对象（查询结果），但没有共同的属性或方法。为了拥有一个通用类型，我使用一个空接口作为返回类型，并在两者上“实现”它。

这听起来当然不对。我只能通过坚持希望有一天这些类会有一些共同点来安慰自己，我会将这些共同逻辑移到我的空界面中。但我并不满足，并思考是否应该有两种不同的方法并有条件地调用 next。这是一个更好的方法吗？

我还被告知 .NET Framework 使用空接口来进行标记。

我的问题是：空接口是设计问题的强烈标志还是它被广泛使用？

编辑：对于那些感兴趣的人，我后来发现函数式语言中的可区分联合是我想要实现的目标的完美解决方案。 C# 似乎对这个概念还不太友好。

编辑：我写了一篇更长的文章关于这个问题，详细解释了问题和解决方案。

Answer 1

尽管该用例似乎存在一种设计模式（现在很多人都提到了“标记接口”），但我相信这种做法的使用表明了代码味道（至少在大多数时候）。

正如 @V4Vendetta 发布的，有一个针对此的静态分析规则：
http://msdn.microsoft.com/en-我们/库/ms182128(v=VS.100).aspx

如果您的设计包含预期类型要实现的空接口，则您可能正在使用接口作为标记或识别一组类型的方法。 如果此识别将在运行时发生，则完成此操作的正确方法是使用自定义属性。使用属性的存在或不存在或属性的属性来识别目标类型。 如果识别必须在编译时进行，那么使用空接口是可以接受的。

这是引用的 MSDN 建议：

删除接口或向其中添加成员。如果使用空接口来标记一组类型，请使用自定义属性替换该接口。

这也反映了已发布的维基百科链接的批评部分。

标记接口的一个主要问题是接口定义了用于实现类的契约，并且该契约由所有子类继承。这意味着您不能“取消实现”标记。在给出的示例中，如果您创建一个不想序列化的子类（可能是因为它依赖于瞬态），则必须显式抛出 NotSerializedException（根据 ObjectOutputStream 文档）。

Answer 2

您声明您的函数“根据某些情况返回完全不同的对象” - 但它们到底有多大不同？一个可以是流编写器，另一个可以是 UI 类，另一个可以是数据对象？不……我怀疑！

您的对象可能没有任何通用的方法或属性，但是，它们的角色或用法可能相似。在这种情况下，标记接口似乎完全合适。

Answer 3

如果不用作标记接口，我会说是的，这是代码味道。

接口定义了实现者遵守的契约 - 如果您有不使用反射的空接口（就像标记接口那样），那么您不妨使用 Object 作为 (已经存在）基本类型。

Answer 4

你回答了你自己的问题...“我有一个函数，根据某些情况返回完全不同的对象。”...为什么你想要有一个返回完全不同对象的相同函数？我看不出它有什么用处，也许你有一个很好的，在这种情况下，请分享。

编辑：考虑到您的澄清，您确实应该使用标记接口。 “完全不同”与“同类”有很大不同。如果它们完全不同（不仅仅是它们没有共享成员），那就会产生代码味道。

Answer 5

正如许多人可能已经说过的那样，空接口确实可以有效地用作“标记接口”。

我能想到的最好的用途可能是将对象表示为属于域的特定子集，由相应的存储库处理。假设您有不同的数据库来检索数据，并且每个数据库都有一个存储库实现。特定存储库只能处理一个子集，并且不应为任何其他子集提供对象的实例。您的域模型可能如下所示：

//Every object in the domain has an identity-sourced Id field
public interface IDomainObject
{
   long Id{get;}
}

//No additional useful information other than this is an object from the user security DB
public interface ISecurityDomainObject:IDomainObject {}

//No additional useful information other than this is an object from the Northwind DB
public interface INorthwindDomainObject:IDomainObject {}


//No additional useful information other than this is an object from the Southwind DB
public interface ISouthwindDomainObject:IDomainObject {}

然后，您的存储库可以通用到 ISecurityDomainObject、INorthwindDomainObject 和 ISouthwindDomainObject，然后您可以在编译时检查代码是否试图将 Security 对象传递给 Northwind DB（或任何其他排列）。在这种情况下，接口提供了有关类性质的有价值的信息，即使它不提供任何实现契约。