但凡是面试中遇到问设计模式的时候,一定要提想法设法的提设计原则,阐述设计原则在开发中应用的重要性,对代码性能,系统拓展性的重要性,相信按照这篇文章阐述一番,面试官一定能发现你高深设计思想。
开闭原则是SOLID中最难理解、最难掌握,同时也是最有用的一条原则。
之所以说这条原则难理解,那是因为:
- 怎样的代码改动才被定义为‘扩展’?
- 怎样的代码改动才被定义为‘修改’?
- 怎么才算满足或违反‘开闭原则’?
- 修改代码就一定意味着违反‘开闭原则’吗?
等等这些问题,都比较难理解。
之所以说这条原则难掌握,那是因为:
- 如何做到‘对扩展开放、修改关闭’?
- 如何在项目中灵活地应用‘开闭原则’,以避免在追求扩展性的同时影响到代码的可读性?
等等这些问题,都比较难掌握。
之所以说这条原则最有用,那是因为:
扩展性是代码质量最重要的衡量标准之一。在23种经典设计模式中,大部分设计模式都是为了解决代码的扩展性问题而存在的,主要遵从的设计原则就是开闭原则。
如何理解“对扩展开放、修改关闭”?
开闭原则的英文全称是Open Closed Principle,简写为OCP。它的英文描述是:software entities (modules, classes, functions, etc.) should be open for extension , but closed for modification。我们把它翻译成中文就是:软件实体(模块、类、方法等)应该“对扩展开放、对修改关闭”。
这个描述比较简略,如果我们详细表述一下,那就是,添加一个新的功能应该是,在已有代码基础上扩展代码(新增模块、类、方法等),而非修改已有代码(修改模块、类、方法等)。
为了让你更好地理解这个原则,我举一个例子来进一步解释一下。这是一段API接口监控告警的代码。
public class Alert {
private AlertRule rule;
private Notification notification;
public Alert(AlertRule rule, Notification notification) {
this.rule = rule;
this.notification = notification;
}
public void check(String api, long requestCount, long errorCount, long durationOfSeconds) {
long tps = requestCount / durationOfSeconds;
if (tps > rule.getMatchedRule(api).getMaxTps()) {
notification.notify(NotificationEmergencyLevel.URGENCY, "...");
}
if (errorCount > rule.getMatchedRule(api).getMaxErrorCount()) {
notification.notify(NotificationEmergencyLevel.SEVERE, "...");
}
}
}
其中,
AlertRule存储告警规则,可以自由设置。
Notification是告警通知类,支持邮件、短信、微信、手机等多种通知渠道。
NotificationEmergencyLevel表示通知的紧急程度,包括SEVERE(严重)、URGENCY(紧急)、NORMAL(普通)、TRIVIAL(无关紧要),不同的紧急程度对应不同的发送渠道。
上面这段代码非常简单,业务逻辑主要集中在check()函数中。当接口的TPS超过某个预先设置的最大值时,以及当接口请求出错数大于某个最大允许值时,就会触发告警,通知接口的相关负责人或者团队。
现在,如果我们需要添加一个功能,当每秒钟接口超时请求个数,超过某个预先设置的最大阈值时,我们也要触发告警发送通知。这个时候,我们该如何改动代码呢?
主要的改动有两处:
- 第一处是修改check()函数的入参,添加一个新的统计数据timeoutCount,表示超时接口请求数;
- 第二处是在check()函数中添加新的告警逻辑。
具体的代码改动如下所示:
public class Alert {
// ...省略AlertRule/Notification属性和构造函数...
// 改动一:添加参数timeoutCount
public void check(String api, long requestCount, long errorCount, long timeoutCount, long durationOfSeconds) {
long tps = requestCount / durationOfSeconds;
if (tp