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

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

完整討論串

留言數 標題 作者 日期
67 >> [討論] 會手癢想動前人的程式嗎? sec5566 2019-01-11T15:08:43
20 Re: [討論] 會手癢想動前人的程式嗎? bachelorwhc 2019-01-12T11:56:25

最新留言

[問題] 關於親x學園 5400(匿名) : 是真的 我就是住0346的....燒炭
[新聞] 開放陸資購買房地產?柯P:沒必要完全禁止 lyngu86 : 中华人民共和国台湾特别行政区首任特别行政长官柯文哲先生你也敢嘴?
[哇靠]野生詹帥 十號媽咪(匿名) : 你也是柏融粉嗎?有賴的聯絡ig嗎?
[問題] 多采多姿頭皮管理 Lu(匿名) : 我的頭髮被染成綠色的 褪色 也褪不掉 不要去 這間店不老實 騙錢的 說什麼天然草本
Fw: [閒聊] 東港的排球團一點點的進展順便宣傳 aaa(匿名) : 東港現在哪裡有固定打排球的場啊,今天都找不到
[問題] 徵求台中遠百Carhartt wip 會員 k48016 : 不好意思,想請問能否跟你借Carhartt wip的會員呢?
[賣錶] BALL Fireman NECC yomanqq : 還在嗎 請問怎麼寄信 新手第一次用
[心得] 超級反推朱崙街的世珍婦產科 Maimai(匿名) : 3年後的今天我也去看看,變好了,還不錯
[問卦] 羽田真理像誰? lksjfieubfno(匿名) : 榮倉奈奈
[閒聊] 房間有超大蟑螂怎麼辦 1122 : 買小蘇打放在蟑螂走過的路徑 蟑螂怕酸 很有效 買酒精看到蟑螂當場噴傻酒精必死無疑
[閒聊] 今日上海銀行筆試 abcexd : 12/26 二面 有接到確認電話 還沒等到QQ
[追劇] 墨西哥電視劇Amar a Muerte Jlezk(匿名) : 我反覆看了好幾次,那應該是某種會反光的東西沾到val臉頰,而不是眼淚
[新聞] 力挺到底 吳釗燮:不會換掉趙怡翔 atking : 吳長官英明,爾等屁民休得無禮狂吠
[新聞] 蘇貞昌第一份公文:核定太空計畫251億 atking : 支持蘇貞昌萬歲萬歲萬萬歲