看板 Soft_Job 關於我們 聯絡資訊
一個系統當然不要去改, 這樣才穩定, 因為你不知道會不會突然冒出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
cuteSquirrel: 得到重構批准再改01/11 15:17
THEWORLDS: 因為你不知道會不會突然冒出bug 那就是沒看懂還想硬改?01/11 15:19
steve1012: 沒unit test會加 太長會改 命名照codebase convention01/11 15:25
yaya517: 在可以運行的最低限度下修改 有些深根蒂固也很難改 除非b01/11 15:28
yaya517: ug已經出來了 就稍微重構乾脆順便把他改好01/11 15:28
steve1012: 不過這沒有共識 你只是想要戰吧 zzz01/11 15:29
APTON: 我同事用goto,我都沒去改了...01/11 15:37
final01: 大哥,你說的是團隊一致的問題吧01/11 15:46
DCTmaybe: 噓id01/11 15:49
benqm300: 邏輯或寫法有問題才改,命名問題是管理問題,除非輪到01/11 16:02
benqm300: 自己負責不然下班比較重要01/11 16:02
我主管說不出問題就可以改, 甚至重構都可以, 反正目前只有我在管, 可是要整個改名稱又有點多, 我主要想知道那如果自己新做的, 也要沿用前人的風格嗎? 將錯就錯還是自己寫的要用對的? 整支檔案命名不一致也是有點癢
pttworld: 達到融入不同專案風格的境界,甚至不同程式語言01/11 16:13
diabloevagto: 問你主管,誰知道你公司是不是就用舊程式的規則01/11 16:25
xxtuoo: 命名會看不下去的代表還太淺Zzz01/11 16:35
Noahdot: 你還來啊?01/11 16:52
※ 編輯: sec5566 (223.137.0.197), 01/11/2019 18:12:20
testPtt: 太舊的東西我會先想到重寫 01/11 18:33
MOONY135: 我覺得當你的同事...有點悲傷 01/11 19:07
alihue: 886 01/11 19:24
YahooTaiwan: 命名規則又沒有標準答案... 01/11 19:55
joery: 你確定牽一髮不會動全身,你確定整個系統你改一個地方有百 01/11 19:58
joery: 分百,確定,絕對把握,再次確認一定不會影響其他地方或造 01/11 19:58
joery: 成其他地方出問題。 01/11 19:58
lonelytea: 一定會被你改出問題啦 不用想了 01/11 19:58
joery: 你確定牽一髮不會動全身,你確定整個系統你改一個地方有百 01/11 19:59
joery: 分百,確定,絕對把握,再次確認一定不會影響其他地方或造 01/11 19:59
joery: 成其他地方出問題。 01/11 19:59
joery: 不然如果真的出問題只會動目前影響的,而且看了好幾遍確認 01/11 20:01
joery: 他流程邏輯找到可以下手的點改,順便重構該部份程式就好。 01/11 20:01
joery: 不然萬一系統有問題,你就有得哭了 01/11 20:01
stupid0319: 沒有bug的話,不要改 01/11 21:08
ChiangKeith: 命名不要太初凡入聖根本不重要吧01/11 22:53
s001582000: 我接到風行天的code都會重寫01/11 23:22
ChungLi5566: 會這樣寫一定有原因 不熟的話別亂動01/11 23:53
ChungLi5566: 例如弱掃沒過 多繞點路騙過弱掃軟體01/11 23:54
CloudyWing: 題外話C# public method 大駝峰反而是標準 XD01/12 00:07
CloudyWing: 現在的別人寫得就算看不下去,但只要穩定就不去動,除01/12 00:13
CloudyWing: 非被反應問題太多或是有些架構要去調整,才會整個整理01/12 00:13
y3k: 寫太難看改不動或有bug再改 因為也有可能是你沒能力...01/12 00:18
cphe: 改排版改命名沒什麼意義吧 這東西不同人寫就是不同風格01/12 00:31
那不改好了, 那新的函式一般都是照前人風格繼續命, 還是後面自己的就用自己的風格?
cphe: 除非一進公司就有明確規定,甚至coding style的script在掃01/12 00:32
shyangs: 方法 C# 大駝峰, Java 小駝峰; 一天寫個語言我常弄錯.01/12 02:10
我沒寫過c#原來有這種差別喔
wadechen: 一直執著在這問題 不覺得很辛苦嗎01/12 02:33
william81615: 樓上 這不就是典型上班沒事幹找碴嗎XD01/12 06:16
layolayo: 吃飽太閒喔...通常半瓶水的人很容易覺得別人的code都是01/12 07:47
layolayo: 垃圾01/12 07:47
※ 編輯: sec5566 (223.137.0.197), 01/12/2019 10:07:44
MOONY135: 我只要能抄隊友的code我就一定不改 01/12 10:05
ymcheung: 但是前人留下的 CSS 不管動哪行都會改到其他頁面 01/12 10:11
ab4daa: if it works, don't fix it! 01/12 10:34
IhateOGC: goto誰說不能用?該好好update了 01/12 11:21
IhateOGC: 還停在70年代教科書? 01/12 11:21
loadingN: 你上班沒有其他更重要的事了嗎? 01/12 12:10
tedmax100: 以不變應萬變; 敵不動,我不動 01/12 13:16
hanshsu: linux kernel 一堆 goto, 唉 怎麼辦? 01/12 14:22
prag222: 我寫按鍵精靈都用goto ....不過現在沒寫了 01/12 16:44
Ghamu: 千萬不要改 爛code 爛命名 爛排版 就是讓老闆多請你進公司 01/12 17:57
Ghamu: 工作的主因 就是有這麼多拉低產能不是生產的雜活 才需要一 01/12 17:57
Ghamu: 直請工程師進來 01/12 17:57
Ghamu: 你是領錢辦事的 達則兼善天下 窮則獨善其身 01/12 17:59
samblow2000: 勸你不要改...自己寫都不敢保證沒問題了... 01/12 18:55
doranako: 有時間再改 01/12 19:29
knives: 請問有出錯嗎,有主管指示嗎 01/13 15:52
shooter555: 覺得goto有時候能讓程式碼簡潔0.0 還不錯用啊 01/13 20:38
y3k: goto是那種有能力用好的人不問 問的人一定不能讓他用的東西 01/14 19:22
y3k: 另外linux kernel是怎樣小弟不清楚 但是跟一堆asm()比起來我 01/14 19:35
y3k: 好奇大家會認為誰優誰劣XD 01/14 19:35
fruithuang: 不出問題可以改,出問題就賴在你身上。系統忽然不穩 01/14 21:09
fruithuang: 都可賴說是你改的問題...... 01/14 21:09
a9228097: 如果只改naming 跟format就用tool改一下xd 02/02 16:42