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#****
暂无签名
中国开发者画像洞察研究报告 2024
分析开发者的行为模式、工作价值、职业发展等内容，帮助整个行业生态更深入地理解开发者，为他们提供更精准...
评论 1 条评论