你动了别人的代码边界

image

前言

  前段时间呢,需要和xx公司进行对接。由于手上活比较多没忙不过来,领导就先帮我把接口调试完成了,并写好了相关的demo。然后我根据demo把代码整合进业务系统,并重写了相关代码。后来领导看了我写的代码,发现和他写的的demo不太一样,然后就问我为什么要重写?在一番争论后,领导对我说了句:你到底懂不懂抽象啊,你动了别人的代码边界。

事情的经过

领导:诶,你这个代码咋和我写的demo不太一样呢?

:我看demo里面的代码有些地方,当返回错误信息的时候抛出了异常。但是实际情况是不能抛出异常,我就给他改写了。

领导:在使用的时候try catch住处理一下不就好了。

:这个确实可以,但是没必要这样做。多套一层 try catch这样不是多此一举了吗。

领导:不是时间很赶吗,那在原有的基础上加一层 try catch不是更快吗。

:说是这样说,但是重写一下那部分又不耗多少时间,耗时间的是调整业务相关的逻辑代码。

领导:就这个原因吗,还有其他的吗?

:这个只是一部分,还有有些写法看起来太臃肿了,可以用属性拷贝和类转换替换掉。这样代码看起来就比较精简了。

领导:减少了代码行数,就是你认为的精简嘛?

:是的。

:对了还有一个是,这个系统用的框架比较老。不支持枚举映射,所以我把一些参数类型变成String了。

领导:那你可以手动映射呀。

:那我还不如用String接收,在使用的地方再用枚举。

领导:在我看来,你的改代码的理由,在我看来都不是理由。

:为啥?

领导:在我看来你因为看那些写法不爽就把它给改了,可以这样理解吧。

:占一部分原因吧。

领导:那你在用Spring的时候,发现有些代码让你看起来不爽,你会改它嘛?

:不会。

领导:为啥呢?

:因为改不了,没得权限。

领导:想改还是可以改的,去github上把代码拉下来,改了重新打包。

:那这个也太麻烦了。

领导:那我是不是认为你因为麻烦或者说没有权限,所以你才不会改。

:是的

领导:这样说的话,如果我把他弄成一个jar,让你用你是不是就改不了。

:是这样的。

:虽然是重写了,但是跟直接把代码拷贝过来是一样的,并没有增加系统的复杂度。

领导:你到底懂不懂抽象啊,你动了别人的代码边界。

image

:我咋就不懂抽象了?

领导:这样和你说吧。

image

领导:原本呢,接口对接那部分是我写的,你只需要负责使用就好了。如果接口发生变更,我可以直接处理,你并不需要关心接口对接的逻辑。

:是这样的。但你的代码并没有写在业务系统里啊。

领导:你直接拷贝过去就好了,这个和提供一个Jar有啥区别,而且代码我都测试过了,可以直接使用。

: 是这样的。

领导:当接口发生改变的时候,我更改完,你直接把代码拷贝过去就好了。

:额。。。。

image

领导:但是现在你把代码重写了,就没有那一层抽象了,也就只能你自己维护了。

:这个本来就是我维护的呀。

:但是系统的抽象程度还是没变呀。

领导:是,整个系统的抽象程度确实没变都是xx服务。但是我现在和你说的是这个项目的,不是整个系统的。

领导:回到之前那个问题。

:嗯嗯

领导:这样和你说吧。当你在看代码的时候,觉得别人的代码写的可能不够规范,或者说不符合你的规范。因为你觉得不符合规范,这个只是你的主观判定,而不是一个客观的事实。在你看来不符合规范的代码,可能就是别人的规范。最好不要因为这个原因去更改别人的的代码。

:好的

总结

  不要去改变别的代码边界。当你更改了别人的代码,就意味着破坏了别人的代码边界。一旦边界被破坏,那就可能出现无法预估的风险。

结尾

  说的通俗一点就是,不要瞎鸡儿改别人的代码。不管别人写的好不好,只要没bug就行,如果有bug也是别人改。有这个时间早点下班不好吗。

  如果觉得对你有帮助,可以多多评论,多多点赞哦,也可以到我的主页看看,说不定有你喜欢的文章,也可以随手点个关注哦,谢谢。

  我是不一样的科技宅,每天进步一点点,体验不一样的生活。我们下期见!

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

推荐阅读更多精彩内容