※ 本文為 terievv 轉寄自 ptt.cc 更新時間: 2019-01-12 16:47:57
看板 Soft_Job
作者 標題 [討論] 會手癢想動前人的程式嗎？
時間 Fri Jan 11 15:08:43 2019
一個系統當然不要去改，
這樣才穩定，
因為你不知道會不會突然冒出bug，
可是有時候接受前人程式，
會看到一些違反dry原則的，
或是命名規則有問題的，
像函式用大駝峰，類別用小駝峰，
或很奇怪的名稱之類，
不然就是排版很亂的，
這種大家會手癢去改嗎？
改下去又是大工程了，結果工作越做越多
另外如果要擴充新函式，
大家會繼續照他的命名規則寫，
還是用正規的？
-----
Sent from JPTT on my Sony H4331.
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 223.137.0.197
※ 文章代碼(AID): #1SE43z8U (Soft_Job)
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1547190525.A.21E.html
※ 編輯: sec5566 (223.137.0.197), 01/11/2019 15:12:52
甚至重構都可以，
反正目前只有我在管，
可是要整個改名稱又有點多，
我主要想知道那如果自己新做的，
也要沿用前人的風格嗎？
將錯就錯還是自己寫的要用對的？
整支檔案命名不一致也是有點癢
那新的函式一般都是照前人風格繼續命，
還是後面自己的就用自己的風格？
--
推 : 得到重構批准再改1F 01/11 15:17
噓 : 因為你不知道會不會突然冒出bug 那就是沒看懂還想硬改?2F 01/11 15:19
推 : 沒unit test會加 太長會改 命名照codebase convention3F 01/11 15:25
推 : 在可以運行的最低限度下修改 有些深根蒂固也很難改 除非b4F 01/11 15:28
→ : ug已經出來了 就稍微重構乾脆順便把他改好
→ : ug已經出來了 就稍微重構乾脆順便把他改好
噓 : 不過這沒有共識 你只是想要戰吧 zzz6F 01/11 15:29
推 : 我同事用goto，我都沒去改了...7F 01/11 15:37
噓 : 大哥，你說的是團隊一致的問題吧8F 01/11 15:46
噓 : 噓id9F 01/11 15:49
推 : 邏輯或寫法有問題才改，命名問題是管理問題，除非輪到10F 01/11 16:02
→ : 自己負責不然下班比較重要
我主管說不出問題就可以改，→ : 自己負責不然下班比較重要
甚至重構都可以，
反正目前只有我在管，
可是要整個改名稱又有點多，
我主要想知道那如果自己新做的，
也要沿用前人的風格嗎？
將錯就錯還是自己寫的要用對的？
整支檔案命名不一致也是有點癢
→ : 達到融入不同專案風格的境界，甚至不同程式語言12F 01/11 16:13
→ : 問你主管，誰知道你公司是不是就用舊程式的規則13F 01/11 16:25
噓 : 命名會看不下去的代表還太淺Zzz14F 01/11 16:35
噓 : 你還來啊？15F 01/11 16:52
※ 編輯: sec5566 (223.137.0.197), 01/11/2019 18:12:20→ : 太舊的東西我會先想到重寫16F 01/11 18:33
→ : 我覺得當你的同事...有點悲傷17F 01/11 19:07
噓 : 88618F 01/11 19:24
推 : 命名規則又沒有標準答案...19F 01/11 19:55
推 : 你確定牽一髮不會動全身，你確定整個系統你改一個地方有百20F 01/11 19:58
→ : 分百，確定，絕對把握，再次確認一定不會影響其他地方或造
→ : 成其他地方出問題。
→ : 分百，確定，絕對把握，再次確認一定不會影響其他地方或造
→ : 成其他地方出問題。
噓 : 一定會被你改出問題啦 不用想了23F 01/11 19:58
→ : 你確定牽一髮不會動全身，你確定整個系統你改一個地方有百24F 01/11 19:59
→ : 分百，確定，絕對把握，再次確認一定不會影響其他地方或造
→ : 成其他地方出問題。
推 : 不然如果真的出問題只會動目前影響的，而且看了好幾遍確認
→ : 他流程邏輯找到可以下手的點改，順便重構該部份程式就好。
→ : 不然萬一系統有問題，你就有得哭了
→ : 分百，確定，絕對把握，再次確認一定不會影響其他地方或造
→ : 成其他地方出問題。
推 : 不然如果真的出問題只會動目前影響的，而且看了好幾遍確認
→ : 他流程邏輯找到可以下手的點改，順便重構該部份程式就好。
→ : 不然萬一系統有問題，你就有得哭了
推 : 沒有bug的話,不要改30F 01/11 21:08
噓 : 命名不要太初凡入聖根本不重要吧31F 01/11 22:53
→ : 我接到風行天的code都會重寫32F 01/11 23:22
→ : 會這樣寫一定有原因 不熟的話別亂動33F 01/11 23:53
→ : 例如弱掃沒過 多繞點路騙過弱掃軟體
→ : 例如弱掃沒過 多繞點路騙過弱掃軟體
推 : 題外話C# public method 大駝峰反而是標準 XD35F 01/12 00:07
推 : 現在的別人寫得就算看不下去，但只要穩定就不去動，除
→ : 非被反應問題太多或是有些架構要去調整，才會整個整理
推 : 現在的別人寫得就算看不下去，但只要穩定就不去動，除
→ : 非被反應問題太多或是有些架構要去調整，才會整個整理
→ : 寫太難看改不動或有bug再改 因為也有可能是你沒能力...38F 01/12 00:18
推 : 改排版改命名沒什麼意義吧 這東西不同人寫就是不同風格39F 01/12 00:31
那不改好了，那新的函式一般都是照前人風格繼續命，
還是後面自己的就用自己的風格？
→ : 除非一進公司就有明確規定，甚至coding style的script在掃40F 01/12 00:32
推 : 方法 C# 大駝峰, Java 小駝峰; 一天寫個語言我常弄錯.41F 01/12 02:10
我沒寫過c#原來有這種差別喔推 : 一直執著在這問題 不覺得很辛苦嗎42F 01/12 02:33
推 : 樓上 這不就是典型上班沒事幹找碴嗎XD43F 01/12 06:16
→ : 吃飽太閒喔...通常半瓶水的人很容易覺得別人的code都是44F 01/12 07:47
→ : 垃圾
※ 編輯: sec5566 (223.137.0.197), 01/12/2019 10:07:44→ : 垃圾
→ : 我只要能抄隊友的code我就一定不改46F 01/12 10:05
推 : 但是前人留下的 CSS 不管動哪行都會改到其他頁面47F 01/12 10:11
→ : if it works, don't fix it!48F 01/12 10:34
推 : goto誰說不能用？該好好update了49F 01/12 11:21
→ : 還停在70年代教科書？
→ : 還停在70年代教科書？
→ : 你上班沒有其他更重要的事了嗎?51F 01/12 12:10
→ : 以不變應萬變； 敵不動，我不動52F 01/12 13:16
推 : linux kernel 一堆 goto, 唉 怎麼辦?53F 01/12 14:22
推 : 我寫按鍵精靈都用goto ....不過現在沒寫了54F 01/12 16:44
--
※ 看板: terievv 文章推薦值: 0 目前人氣: 0 累積人氣: 75
回列表(←)
分享