作者EricTCartman (阿ㄆㄧㄚˇ)
看板Soft_Job
标题Re: [讨论] 重构之前要写测试 不然不要重构
时间Fri Jul 3 21:58:53 2020
※ 引述《Ghamu (猫丸)》之铭言：
: 但事实上之前都没写测试了
: 你怎麽证明他之前是对的呢?
这就是TAD, 一般做法是假设以前人做的是对的
拿以前的output当测资 避免以後的output跟预期结果不同
技术面的错误→没有防呆/没有释放资源/overflow/没有check
这应该不在讨论范围内, 也有客观标准
行为与逻辑的部分才是有争议的, 要嘛根本没规格只有口传
要嘛就是写的人弄拙成巧 刚好做对
所以在没有规格跟明确定义的状况下写测试 只是写的人自己觉得对而已
test code也是code 也一样要维护 也一样有可能会写错
: 所以我大多都直接给他改下去
: 反正重构後东西也比较清楚
: 即使有错 也比起虾鸡巴狗烂毛程式码好除错
每个人都觉得对方code烂 现在我都用: 我就烂 的心态来写
: 之前前辈都说会动的程式码不要去碰
: 然後就一球在那边
: 我说要改 他就说
: [啊你有写测试吗?]
: 开发时程又不允许
你没听出话中话 人家前辈是好心人
大家都写程式 又不是你最聪明 所有人都知道时程不允许
你改了code 出现一堆bug 锅在你头上
对方一方面也知道那是烂code不想明讲 搞不好写烂code的人还在公司
一方面也知道重构没有多少绩效 做不好还惹得一身臭 期望值低到爆表
人家处处为你着想 你何苦先入为主
要是我是你同事 一定默默地让你重构
: 就一球在那边越来越痛苦
: 会动的烂程式码越来越多
: 不知道大家怎麽看
ptt都是菁英群
基本上大家写code都是clean code 还有落实unit test
要不是待在根本没有legacy code的新创公司
要嘛就是会把数以万计的legacy code补完unit test的楷模员工
你要是在的公司根本没有在写test
说明你公司太烂 八成没一个同事是乡民
建议你换好一点的公司 再上ptt跟大家讨论 比较有共同基准
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 1.163.96.79 (台湾)
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/Soft_Job/M.1593784735.A.38E.html
※ 编辑: EricTCartman (1.163.96.79 台湾), 07/03/2020 22:00:02
1F：推 NDark: GJ 07/03 23:12
2F：推 APTON: XD 07/04 00:12
3F：推 sharek: 中肯酸XD 07/04 00:45
4F：推 labbat: 没有人写test 因为tester把设计者气跑 tester也不想别人测 07/04 01:25
5F：→ allenxxx: 通常自认为完美的程式都会死在外行测试者手上 07/04 01:32
6F：→ allenxxx: 谁规定我一定要照你逻辑操作? 07/04 01:32
7F：推 zased: 那是edge case考虑不周全，所以才需要tester，跳脱思考框架 07/04 10:54
8F：推 Csongs: 每个人都觉人其他人code烂 我可以理解XD 07/04 11:15
9F：推 wulouise: 我也觉得自己以前写得烂..没test重构还能确保没错很神 07/04 13:19
10F：推 r030515495: 推 中肯回覆 07/04 13:32
11F：→ r030515495: 公司重视的只有产出，而不会在意100%的测试覆盖率，要 07/04 13:36
12F：→ r030515495: 重构还是先把功能做出，有时间再来写测试，让後面的 07/04 13:36
13F：→ r030515495: 维护不会改A坏B 07/04 13:36
14F：推 APTON: 可以看看TDD是怎麽用测试把product code产出来 07/04 15:46
15F：→ lazarus1121: code本来就会因为业务逻辑熟悉程度有好坏之分 07/04 21:40
16F：→ lazarus1121: 对业务够熟把烂code整理一下也是好事 07/04 21:43
17F：推 mathrew: XDDDD 07/05 12:31
18F：推 gmoz: XDDD 07/05 14:32
19F：→ waterwalk: 厉害了 我也是这样想所以离职 但是就没工作了QQ 07/05 19:04
20F：推 wadxmjh: 中肯酸 07/06 00:40
21F：→ shooter555: 中肯 07/06 10:25
22F：→ shooter555: 尤其最後两段 07/06 10:25
23F：推 csieflyman: 高手都是随便不爽不要做换公司或烂到说服老板打掉重练 07/06 14:13
24F：→ csieflyman: 就没legacy code了 根本就不需要重构及写测试 尤其是 07/06 14:13
25F：→ csieflyman: 别人写的程式 07/06 14:13
26F：推 csieflyman: 笨蛋才占着茅坑(公司)不拉屎(烂code) 高手都是坑满了 07/06 14:17
27F：→ csieflyman: 之後 换个茅坑再拉屎的好吗 07/06 14:17
28F：→ accessdenied: 高手们躺着写code也中枪 07/06 17:21
29F：→ Vitaceae: 还以为凡事必OO凡扣必UT的毒观念消失了 没想到还有着魔 07/06 19:55
30F：→ Vitaceae: 的人 07/06 19:55
31F：推 bakedgrass: 酸度有像阿ㄆㄧㄚˇ 07/07 05:45
32F：推 tay2510: 笑死 推这篇 07/07 11:03
33F：推 brianhsu: 最後一段很中肯啊，没测试的地方还是不要久待 07/07 19:25