c# - 如何考虑不同的实现需求？

Question

假设我有一个包含两个具体类的接口(interface)。一个具体需要实现IDisposable。是否应该为了一个类的利益修改接口(interface)以实现 IDisposable，或者接口(interface)的使用者是否必须执行运行时检查以确保可处置性？

我认为应该修改接口(interface)，因为它是一个简单的更改(特别是如果它是一个新接口(interface))但我也可以看到在更改设计以适应特定实现时可能违反 liskov(特别是如果其他类或类必须抛出不受支持的异常)

Answer 1

如果框架本身有任何迹象，让接口(interface)实现 IDisposable 的适当性取决于可处置性是否是实现接口(interface)定义的契约的必要属性。少数框架接口(interface)确实实现了 IDisposable，包括:

System.Collections.Generic.IEnumerator<T>
System.Deployment.Internal.Isolation.Store
System.Resources.IResourceReader
System.Resources.IResourceWriter
System.Security.Cryptography.ICryptoTransform
System.ComponentModel.IComponent
System.ComponentModel.IContainer

就其性质而言，这些接口(interface)通常定义将消耗并因此需要释放资源的结构。从这个意义上讲，资源处置可以被视为实现契约的组成部分，而不是实现接口(interface)的具体类的实现细节。例如，IResourceReader 将从资源中读取，关闭资源是实现契约的必要部分。

相比之下，在框架中，具体类直接实现IDisposable(而不是通过另一个接口(interface))是很常见的。对于框架类，可以通过反射查询:

foreach (var v in typeof(/*any type*/)
                      .Assembly.GetTypes()
                      .Where(a => a.IsClass 
                              && typeof(IDisposable).IsAssignableFrom(a)
                              && a.GetInterfaces().Where(
                               i=>i!=typeof(IDisposable)
                       ).All(i=>!typeof(IDisposable).IsAssignableFrom(i))))
{
   foreach (var s in v.GetInterfaces())
       Console.WriteLine(v.FullName + ":" + s.Name);
}

通常，这些类的实现需要消耗资源，附带于实现接口(interface)契约。例如，System.Data.SqlClient.SqlDataAdapter分别实现了IDbDataAdapter和IDisposable； IDbDataAdapter完全有可能不需要配置，但是SqlDataAdapter的实现需要消耗和释放资源。

在您的情况下，您指出有两个类实现了您的接口(interface)，一个需要实现 IDisposable，一个不需要。鉴于没有，根据定义，处理资源的能力并不是满足接口(interface)要求不可或缺的；因此接口(interface)本身不应实现 IDisposable。

顺便说一句，Dispose() 不应抛出异常(参见 CA1065: Do not raise exceptions in unexpected locations。)如果实现了 IDisposable 的类实例没有可处置的资源，它可以简单地返回;满足释放所有资源的后置条件。不必抛出 NotSupportedException。

附录

第二个可能的考虑因素是界面的预期用途。例如，在数据库场景中通常使用以下模式:

 System.Data.IDbCommand cmd = ...;
 using (var rdr = cmd.ExecuteReader()) // returns IDataReader (IDisposable)
 {
     while (rdr.Read()) {...}
 } // dispose

如果 IDataReader 不实现 IDisposable，等效代码将需要复杂得多:

 System.Data.IDbCommand cmd = ...;
 System.Data.IDataReader rdr;
 try
 {
     rdr = cmd.ExecuteReader();
     while (rdr.Read()) {...};
 } finally {
     if (rdr is IDisposable) ((IDisposable)rdr).Dispose();
 }

如果预期这种类型的用法很常见，则可以证明将接口(interface) IDisposable 作为一种特殊情况，即使并非所有实现都有望实现 IDisposable.