开闭原则、重构

Question

我正在尝试将 OCP 应用于我的代码片段，该代码片段在当前状态下确实很臭，但我觉得我没有完全完成。

当前代码：

public abstract class SomeObject
{}

public class SpecificObject1 : SomeObject
{}

public class SpecificObject2 : SomeObject
{}


// Smelly code
public class Model
{
  public void Store(SomeObject someObject)
  {
    if (someObject is SpecificObject1)
    {}
    else if (someObject is SpecificObject2)
    {}
  }
}

这真的很难看，我的新方法如下所示：

// No so smelly code
public class Model
{
  public void Store(SomeObject someObject)
  {
    throw new Expception("Not allowed!");
  }

  public void Store(SpecificObject1 someObject)
  {}

  public void Store(SpecificObject2 someObject)
  {}

}

当出现新的 SomeObject 类型时，我必须实现该特定对象的方式 已存储，这将破坏 OCP，因为我需要更改模型类。

将存储逻辑移至 SomeObject 也感觉不对，因为那样我会违反 SRP（？），因为在这种情况下，SomeObject 几乎就像 DTO，它的责任不是如何知道存储自身。

如果 SomeObject 出现新的实现，那么谁的存储实现会丢失 由于 Model 类中的 Store 方法出现异常，我会收到一个运行时错误，它也感觉像是代码味道。

的形式

IEnumerable<SomeObject> sequence;

这是因为调用代码将以“我不会知道序列对象的具体类型”

。我似乎无法理解 OCP 的概念。任何人都有任何具体的例子或链接，不仅仅是一些汽车/水果的例子吗？

Answer 1

我展示的模式尝试为特定对象注册处理程序。必须为可能发生的每种类型的对象注册处理程序。如果没有处理程序可以处理它，则会引发异常。

您可以从您的或其他程序集中动态加载处理程序（实现 IHandler 的所有内容），然后实例化并添加注册它们。因此，为任何类型创建处理程序类就足够了实现SomeObject。

public interface IHandler {
    bool TryHandle (SomeObject o); // return true iff handled
}

public class Model
{
    private List<SIandler> _Handlers = new List<IHandlers>();

    // registers a new handler
    public void RegisterHandler (IHandler h) {
        _Handlers.Add(h);
    }

    // this tries to store an object by letting all handlers attempts to store
    public void Store (SomeObject o) {
        foreach (var h in _Handlers) {
            if (h.Store(o)) return;
        }

        // no handler was able to handle the type
        throw new Exception();
    }
}

public class Specific1Handler: IHandler
{
    public bool Handle (SomeObject o) {
        if (o is SpecificType1) {
            /* store... */
            return true; // we handled this object
        } else {
            // we're not qualified
            return false;
        }
    }
}

我相信这会满足您的需求。 （顺便说一句，我不知道这个模式是否有名字，如果有的话我很高兴知道。）