C#的SOLID原则实践

Brannon B. King 是供职于Autonomous Solution Inc.的一名软件工程师，他在MSDN 杂志 2014 年 05 月刊发表了一篇题为《违背 C# 中 SOLID 原则的危险》的文章。作者指出了研发人员在 C# 编码中可能出现的一些常见错误，违背SOLID 原则将导致代码不易扩展、难以维护。

King 提供了计数器的示例代码，并针对 SOLID 每条原则给出了建议，但为了简洁起见我们只节选了开闭原则（OCP）相关的一些内容。开闭原则（OCP）规定“软件实体（类、模块、函数等）应该对扩展开放，对修改关闭”。根据 King 的说法，下面这段代码违背了开闭原则

void DrawNerd(Nerd nerd) {
 if (nerd.IsSelected) DrawEllipseAroundNerd(nerd.Position, nerd.Radius);
 if (nerd.Image != null) DrawImageOfNerd(nerd.Image, nerd.Position, nerd.Heading);
 if (nerd is IHasBelt) // a rare occurrence
DrawBelt(((IHasBelt)nerd).Belt);
 // Etc.
}

因为你需要在客户每次需要显示新增内容时修改此方法，而且，客户始终需要显示新增内容。建议将绘制替换成通用程序：

readonly IList<IRenderer> _renderers = new List<IRenderer>();
void Draw(Nerd nerd) 
{   foreach (var renderer in _renderers)
renderer.DrawIfPossible(_context, nerd); }

思路是这样的：

…编写实现已知接口的绘制类（或有关绘制类的类）。呈现器必须能够决定其是否可以或应该基于输入内容绘制任何内容。例如，带式绘制代码可以移动到其自身的“带式呈现器”，用于检查接口并视需要继续执行。

基类引用继承类是违背开闭原则的另一个例子

class Nerd {
public void DanceTheDisco() {
   if (this is ChildOfNerd)
      throw new CoordinationException("Can't");
   ...
}
}
class ChildOfNerd : Nerd { ... }

作者建议“基类绝不能直接引用其继承类。”。

对等类中也可能存在该问题：

class NerdsInAnArc {
public bool Intersects(NerdsInAnLine line) {
...
 }
 ...
}

King 解释道：

通常情况下，对象层次结构中的弧线和直线是对等的。它们不应该知道彼此之间的非继承的详尽细节，因为这些细节通常是最优交叉算法所需的。随时修改其中一个，而无需更改另一个。这再一次违背了单一责任。存储弧线，还是分析这些弧线？将分析操作置于其自己的实用程序类中。

尽管对于小型项目来说可能不是很必要，但为了避免产生面条式代码，代码规模越大，严格执行 SOLID 原则的重要性就越明显。

原文链接：Becoming SOLID in C#

C# 的 SOLID 原则实践