作者sec5566 (sec)
看板Soft_Job
标题[讨论] 会手痒想动前人的程式吗？
时间Fri Jan 11 15:08:43 2019
一个系统当然不要去改，
这样才稳定，
因为你不知道会不会突然冒出bug，
可是有时候接受前人程式，
会看到一些违反dry原则的，
或是命名规则有问题的，
像函式用大驼峰，类别用小驼峰，
或很奇怪的名称之类，
不然就是排版很乱的，
这种大家会手痒去改吗？
改下去又是大工程了，结果工作越做越多
另外如果要扩充新函式，
大家会继续照他的命名规则写，
还是用正规的？
-----
Sent from JPTT on my Sony H4331.
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 223.137.0.197
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/Soft_Job/M.1547190525.A.21E.html
※ 编辑: sec5566 (223.137.0.197), 01/11/2019 15:12:52
1F：推 cuteSquirrel: 得到重构批准再改01/11 15:17
2F：嘘 THEWORLDS: 因为你不知道会不会突然冒出bug 那就是没看懂还想硬改?01/11 15:19
3F：推 steve1012: 没unit test会加 太长会改 命名照codebase convention01/11 15:25
4F：推 yaya517: 在可以运行的最低限度下修改 有些深根蒂固也很难改 除非b01/11 15:28
5F：→ yaya517: ug已经出来了 就稍微重构乾脆顺便把他改好01/11 15:28
6F：嘘 steve1012: 不过这没有共识 你只是想要战吧 zzz01/11 15:29
7F：推 APTON: 我同事用goto，我都没去改了...01/11 15:37
8F：嘘 final01: 大哥，你说的是团队一致的问题吧01/11 15:46
9F：嘘 DCTmaybe: 嘘id01/11 15:49
10F：推 benqm300: 逻辑或写法有问题才改，命名问题是管理问题，除非轮到01/11 16:02
11F：→ benqm300: 自己负责不然下班比较重要01/11 16:02
我主管说不出问题就可以改，
甚至重构都可以，
反正目前只有我在管，
可是要整个改名称又有点多，
我主要想知道那如果自己新做的，
也要沿用前人的风格吗？
将错就错还是自己写的要用对的？
整支档案命名不一致也是有点痒
12F：→ pttworld: 达到融入不同专案风格的境界，甚至不同程式语言01/11 16:13
13F：→ diabloevagto: 问你主管，谁知道你公司是不是就用旧程式的规则01/11 16:25
14F：嘘 xxtuoo: 命名会看不下去的代表还太浅Zzz01/11 16:35
15F：嘘 Noahdot: 你还来啊？01/11 16:52
※ 编辑: sec5566 (223.137.0.197), 01/11/2019 18:12:20
16F：→ testPtt: 太旧的东西我会先想到重写 01/11 18:33
17F：→ MOONY135: 我觉得当你的同事...有点悲伤 01/11 19:07
18F：嘘 alihue: 886 01/11 19:24
19F：推 YahooTaiwan: 命名规则又没有标准答案... 01/11 19:55
20F：推 joery: 你确定牵一发不会动全身，你确定整个系统你改一个地方有百 01/11 19:58
21F：→ joery: 分百，确定，绝对把握，再次确认一定不会影响其他地方或造 01/11 19:58
22F：→ joery: 成其他地方出问题。 01/11 19:58
23F：嘘 lonelytea: 一定会被你改出问题啦 不用想了 01/11 19:58
24F：→ joery: 你确定牵一发不会动全身，你确定整个系统你改一个地方有百 01/11 19:59
25F：→ joery: 分百，确定，绝对把握，再次确认一定不会影响其他地方或造 01/11 19:59
26F：→ joery: 成其他地方出问题。 01/11 19:59
27F：推 joery: 不然如果真的出问题只会动目前影响的，而且看了好几遍确认 01/11 20:01
28F：→ joery: 他流程逻辑找到可以下手的点改，顺便重构该部份程式就好。 01/11 20:01
29F：→ joery: 不然万一系统有问题，你就有得哭了 01/11 20:01
30F：推 stupid0319: 没有bug的话,不要改 01/11 21:08
31F：嘘 ChiangKeith: 命名不要太初凡入圣根本不重要吧01/11 22:53
32F：→ s001582000: 我接到风行天的code都会重写01/11 23:22
33F：→ ChungLi5566: 会这样写一定有原因 不熟的话别乱动01/11 23:53
34F：→ ChungLi5566: 例如弱扫没过 多绕点路骗过弱扫软体01/11 23:54
35F：推 CloudyWing: 题外话C# public method 大驼峰反而是标准 XD01/12 00:07
36F：推 CloudyWing: 现在的别人写得就算看不下去，但只要稳定就不去动，除01/12 00:13
37F：→ CloudyWing: 非被反应问题太多或是有些架构要去调整，才会整个整理01/12 00:13
38F：→ y3k: 写太难看改不动或有bug再改 因为也有可能是你没能力...01/12 00:18
39F：推 cphe: 改排版改命名没什麽意义吧 这东西不同人写就是不同风格01/12 00:31
那不改好了，
那新的函式一般都是照前人风格继续命，
还是後面自己的就用自己的风格？
40F：→ cphe: 除非一进公司就有明确规定，甚至coding style的script在扫01/12 00:32
41F：推 shyangs: 方法 C# 大驼峰, Java 小驼峰; 一天写个语言我常弄错.01/12 02:10
我没写过c#原来有这种差别喔
42F：推 wadechen: 一直执着在这问题 不觉得很辛苦吗01/12 02:33
43F：推 william81615: 楼上 这不就是典型上班没事干找碴吗XD01/12 06:16
44F：→ layolayo: 吃饱太闲喔...通常半瓶水的人很容易觉得别人的code都是01/12 07:47
45F：→ layolayo: 垃圾01/12 07:47
※ 编辑: sec5566 (223.137.0.197), 01/12/2019 10:07:44
46F：→ MOONY135: 我只要能抄队友的code我就一定不改 01/12 10:05
47F：推 ymcheung: 但是前人留下的 CSS 不管动哪行都会改到其他页面 01/12 10:11
48F：→ ab4daa: if it works, don't fix it! 01/12 10:34
49F：推 IhateOGC: goto谁说不能用？该好好update了 01/12 11:21
50F：→ IhateOGC: 还停在70年代教科书？ 01/12 11:21
51F：→ loadingN: 你上班没有其他更重要的事了吗? 01/12 12:10
52F：→ tedmax100: 以不变应万变； 敌不动，我不动 01/12 13:16
53F：推 hanshsu: linux kernel 一堆 goto, 唉 怎麽办? 01/12 14:22
54F：推 prag222: 我写按键精灵都用goto ....不过现在没写了 01/12 16:44
55F：嘘 Ghamu: 千万不要改 烂code 烂命名 烂排版 就是让老板多请你进公司 01/12 17:57
56F：→ Ghamu: 工作的主因 就是有这麽多拉低产能不是生产的杂活 才需要一 01/12 17:57
57F：→ Ghamu: 直请工程师进来 01/12 17:57
58F：→ Ghamu: 你是领钱办事的 达则兼善天下 穷则独善其身 01/12 17:59
59F：推 samblow2000: 劝你不要改...自己写都不敢保证没问题了... 01/12 18:55
60F：推 doranako: 有时间再改 01/12 19:29
61F：嘘 knives: 请问有出错吗，有主管指示吗 01/13 15:52
62F：推 shooter555: 觉得goto有时候能让程式码简洁0.0 还不错用啊 01/13 20:38
63F：→ y3k: goto是那种有能力用好的人不问 问的人一定不能让他用的东西 01/14 19:22
64F：→ y3k: 另外linux kernel是怎样小弟不清楚 但是跟一堆asm()比起来我 01/14 19:35
65F：→ y3k: 好奇大家会认为谁优谁劣XD 01/14 19:35
66F：推 fruithuang: 不出问题可以改，出问题就赖在你身上。系统忽然不稳 01/14 21:09
67F：→ fruithuang: 都可赖说是你改的问题...... 01/14 21:09
68F：推 a9228097: 如果只改naming 跟format就用tool改一下xd 02/02 16:42