[討論] 會手癢想動前人的程式嗎？

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