作者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/m.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