Soft_Job 板


LINE

这是个大哉问,所以资讯太少的状况下有点难回答 但我还是试着从工程师的专业面来讲这问题好了(先不谈管理学或厚黑学XD) 要很简单的当是非题回答的话,当然是: YES,请重构 一开始需求没很复杂所以没设计好是完全正常的 要做到 Design 但又不 Over Design 的精神 就是需求改动撞到问题的时候才重构,对,就是现在 但请不要是全部打掉重练,而是想办法依据需求扩展该处就好(类似修bug) 我非常喜欢 Modern Web 2018 的一场 Terry Yin 大神的演讲 原文连结:https://hackmd.io/c/MW18/%2FefTkc1AFRK-eYdXOOylmfQ 他把这类问题解释的非常清楚又有趣 我这边针对你的问题帮你提炼一下重点,但有兴趣可以去看看原文 演讲中提到 Michael Feather 大师的书 里面提到重构的五步骤就是 1. 找到改动点 2. 找到测试点 3. 依赖隔离 4. 写测试 5. 重构+新测试 而其中最困难的步骤就是:3. 依赖隔离 所以通常争执点都在这,因为这可大可小 这点指的是,你要先把要改动的点抽离出来 搞清楚他的 input 有谁,output 有谁 最终目标是减少这段"改动点"跟其他人的相依性(解耦) 不要依赖任何神秘的全域变数或外部状态 而是靠你当下 input 进来的东西就能顺利执行,并给出 output 如果有依赖外部系统,例如资料库 那就把连线用的物件当变数传进来,而非在里面自己生成 当然最好也不要有副作用 如果这步骤难度并不太高,你可以先写个大测试保护好他,再来"抽离" 但如果难度恨天高,要做也只能先把手弄脏、刀开下去了 然後做好後续 code review 与人力测试,确保一切安稳 所以讲这麽多才讲到重点 重点就是这个抽离的步骤到底有多难,决定你重构要多久 而这时间算出来,配合商业端决策,才能决定你要不要现在做这件事 (重要系统的话迟早要做的,只是是不是现在而已) 至於当你这段程式码,已经顺利解耦之後 一切就回归到完美的世界了 在只有 input -> 纯逻辑 -> output 的时候 重构是很简单的事了对吧?因为他只剩下 1. 写测试测原本的逻辑 2. 做你想做的扩展 3. 确保旧测试没坏 4. 写新扩展的测试(也可以先4再2) 5. 新扩展的测试也全数通过 完成了,恭喜,世界因为你的努力又更美好了一点 最後聊一下测试这个问题 我想你原文说的历经很多测试 应该是指线上跑很久没坏 而不是指他有很完整的 unit/integration test 对吧? 那这其实就是债 没有被自动测试的 code 就是债,不管你写的多好 是债迟早就得还,不还还会生利息,除非这系统整个废掉不用了 利息有的生的快,有的生的慢 生的慢的也就相对不急,可以有空再说 但你现在做的这个 copy 出一份 8.7 成像的 code 等於让这个债变成 200% 的超级高利贷了,那当然越快还越好 --



※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 36.230.204.42
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/Soft_Job/M.1542287062.A.B79.html ※ 编辑: BBSealion (36.230.204.42), 11/15/2018 21:08:32
1F:推 tommyptt: 推 11/16 18:47
2F:推 landlord: 是Terry Yin唷,尹哲,Odd-e的同事 :) 11/16 21:12
3F:推 landlord: 对Odd-e的培训、顾问、社群活动,欢迎找我。 11/16 21:14
4F:→ landlord: 我是91, 目前Odd-e Taiwan的负责人。 11/16 21:14
5F:→ BBSealion: 91大神!!(一阵子忙没来逛XDD 没看到大神回应) 12/06 09:54
※ 编辑: BBSealion (60.248.176.220), 12/06/2018 09:55:25
6F:推 smily134: 推 06/28 23:49







like.gif 您可能会有兴趣的文章
icon.png[问题/行为] 猫晚上进房间会不会有憋尿问题
icon.pngRe: [闲聊] 选了错误的女孩成为魔法少女 XDDDDDDDDDD
icon.png[正妹] 瑞典 一张
icon.png[心得] EMS高领长版毛衣.墨小楼MC1002
icon.png[分享] 丹龙隔热纸GE55+33+22
icon.png[问题] 清洗洗衣机
icon.png[寻物] 窗台下的空间
icon.png[闲聊] 双极の女神1 木魔爵
icon.png[售车] 新竹 1997 march 1297cc 白色 四门
icon.png[讨论] 能从照片感受到摄影者心情吗
icon.png[狂贺] 贺贺贺贺 贺!岛村卯月!总选举NO.1
icon.png[难过] 羡慕白皮肤的女生
icon.png阅读文章
icon.png[黑特]
icon.png[问题] SBK S1安装於安全帽位置
icon.png[分享] 旧woo100绝版开箱!!
icon.pngRe: [无言] 关於小包卫生纸
icon.png[开箱] E5-2683V3 RX480Strix 快睿C1 简单测试
icon.png[心得] 苍の海贼龙 地狱 执行者16PT
icon.png[售车] 1999年Virage iO 1.8EXi
icon.png[心得] 挑战33 LV10 狮子座pt solo
icon.png[闲聊] 手把手教你不被桶之新手主购教学
icon.png[分享] Civic Type R 量产版官方照无预警流出
icon.png[售车] Golf 4 2.0 银色 自排
icon.png[出售] Graco提篮汽座(有底座)2000元诚可议
icon.png[问题] 请问补牙材质掉了还能再补吗?(台中半年内
icon.png[问题] 44th 单曲 生写竟然都给重复的啊啊!
icon.png[心得] 华南红卡/icash 核卡
icon.png[问题] 拔牙矫正这样正常吗
icon.png[赠送] 老莫高业 初业 102年版
icon.png[情报] 三大行动支付 本季掀战火
icon.png[宝宝] 博客来Amos水蜡笔5/1特价五折
icon.pngRe: [心得] 新鲜人一些面试分享
icon.png[心得] 苍の海贼龙 地狱 麒麟25PT
icon.pngRe: [闲聊] (君の名は。雷慎入) 君名二创漫画翻译
icon.pngRe: [闲聊] OGN中场影片:失踪人口局 (英文字幕)
icon.png[问题] 台湾大哥大4G讯号差
icon.png[出售] [全国]全新千寻侘草LED灯, 水草

请输入看板名称,例如:Tech_Job站内搜寻

TOP