我们该重用代码还是复制粘贴?——对单一职责的想法

我们该重用代码还是复制粘贴?答案是“分情况”,那么这个边界在哪呢?这也是我这篇博客所思考的问题。

今天在老德德的推荐下开始看了《架构之道》,这本书并不厚,但是干货挺多。我用了一些琐碎的时间看了几章,每章长度也不长,每章都像一篇高手的博客,很有意思。
这本书的四分之一处开始介绍一些设计原则,首当其冲的就是单一职责原则,这个职责的学术范儿定义我不想赘述,因为并不好理解,我打算从书中的一个例子开始说起。

假设有这样一种情况

假设有这么一个类Employee雇员类,这个类里面有3个方法:

class Employee {
  caculatePay();  //这个方法用于计算当月工资
  reportHours(); // 这个方法用于计算当月工时
  save(); //这个方法用于将工资和工时存入数据库
}

你看这三个方法:

  1. caculatePay(); 这个方法用于计算当月工资,财务部的软件会调用这个方法来显示工资。
  2. reportHours(); 这个方法用于计算工时,人事部的软件会调用这个方法来显示工时。
  3. save();这个方法用于将工资和工时存入数据库

那么此时,程序员在实现这两个方法的时候,都使用了一个公共函数,regularHours(),如下图:

image.png

有一个程序员不小心踩了雷

财务部的程序员需要对caculatePay这个方法进行更新,因为财务部的计算模型变了。于是这位程序员发现caculatePay这个方法其实是调用了一个regularHours方法,于是他就把regularHours给改动了一下。拜托,他可不知道这个方法正在被别人使用呢!理所当然的,人事部的数据就出现了问题。

单一职责的解决之道

单一职责的思路是:我们需要将Employee类拆分成两个类。同样的,这两个类所共同依赖的regularHours方法当然也要被实现2次。没错,目前看来,我确实是在复制粘贴regularHours方法中的代码,可是这样一来,财务部的程序员就不会影响到其他部门的同事了。

内心OS:如果这本书的作者不是大牛的话,可能这个复制粘贴的行为会被喷。但这个例子是可以说服我的。

缺点

在《设计模式之禅》这本书里也提到了,单一职责这种设计方法很容易导致代码超级膨胀,本来一个类可以做的事情被拆成了多个类。那么什么时候该拆分,什么时候不该拆分呢?我需要讲述另一个例子。

一个现实中的例子

我们当前的项目中,在订票的流程页面写了一个卡片,类似于这样:

image.png

过了很久后来了一个新需求,我们需要在管理页面也实现这样一个卡片的样式。我有两个选择:

  1. import scss from 'normalBookingFlow';这句话能看明白吧?哈哈,就是我在管理页面直接导入之前的scss文件。
  2. 创建一个新的scss文件,并且将之前的scss样式复制粘贴过来。

显然,这是一个分水岭,而我该怎么做呢?
我的个人见解是:根据team来决定是否选择复制粘贴。说的直白点:
对于复制粘贴还是直接复用的边界在于,两段代码当前是否处于同一个小团队在开发,或者是否处于同一个业务流程。

因为人事部和财务部不是同一批开发团队,所以对于这两个部门共用的代码应该拆分成两个部分。
因为普通的订票流程和管理页面的流程不是一个业务流程,所以也应该被拆分。

说来说去,其实结论一句话就说明白了。但是如果我在题头直接写这句话:对于复制粘贴还是直接复用的边界在于,两段代码当前是否处于同一个小团队在开发,或者是否处于同一个业务流程。 不着头不着尾的谁他妈看得懂啊?

结论:

1. 单一职责其实解决的是代码复用与代码复制的边界问题。
2. 对于复制粘贴还是直接复用的边界在于,两段代码当前是否处于同一个小团队在开发,或者是否处于同一个业务流程。

以上只是我结合自己的开发经验和书中所写产生的联想和理解,毕竟代码开发因人而异,如果编程可以做到完全客观呈现没有个人风格,那么要我们还有什么用呢?找一帮编程的机器人就完事儿了。编程的艺术正在于,你可以从代码看到一个人的成长和影子。

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 206,214评论 6 481
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 88,307评论 2 382
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 152,543评论 0 341
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 55,221评论 1 279
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 64,224评论 5 371
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 49,007评论 1 284
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,313评论 3 399
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,956评论 0 259
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 43,441评论 1 300
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,925评论 2 323
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,018评论 1 333
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,685评论 4 322
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,234评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,240评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,464评论 1 261
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,467评论 2 352
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,762评论 2 345

推荐阅读更多精彩内容