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

作者: sec5566 (sec)   2019-01-11 15:08:43
一個系統當然不要去改,
這樣才穩定,
因為你不知道會不會突然冒出bug,
可是有時候接受前人程式,
會看到一些違反dry原則的,
或是命名規則有問題的,
像函式用大駝峰,類別用小駝峰,
或很奇怪的名稱之類,
不然就是排版很亂的,
這種大家會手癢去改嗎?
改下去又是大工程了,結果工作越做越多
另外如果要擴充新函式,
大家會繼續照他的命名規則寫,
還是用正規的?
作者: cuteSquirrel (松鼠)   2019-01-11 15:17:00
得到重構批准再改
作者: THEWORLDS (天下)   2019-01-11 15:19:00
因為你不知道會不會突然冒出bug 那就是沒看懂還想硬改?
作者: steve1012 (steve)   2019-01-11 15:25:00
沒unit test會加 太長會改 命名照codebase convention
作者: yaya517 (Abby)   2019-01-11 15:28:00
在可以運行的最低限度下修改 有些深根蒂固也很難改 除非bug已經出來了 就稍微重構乾脆順便把他改好
作者: steve1012 (steve)   2019-01-11 15:29:00
不過這沒有共識 你只是想要戰吧 zzz
作者: APTON (瑋瑋)   2019-01-11 15:37:00
我同事用goto,我都沒去改了...
作者: final01 (牛頓運動定律)   2019-01-11 15:46:00
大哥,你說的是團隊一致的問題吧
作者: DCTmaybe (竹竹人)   2019-01-11 15:49:00
噓id
作者: benqm300 (人生苦短)   2019-01-11 16:02:00
邏輯或寫法有問題才改,命名問題是管理問題,除非輪到自己負責不然下班比較重要
作者: pttworld (批踢踢世界)   2019-01-11 16:13:00
達到融入不同專案風格的境界,甚至不同程式語言
作者: diabloevagto (wi)   2019-01-11 16:25:00
問你主管,誰知道你公司是不是就用舊程式的規則
作者: xxtuoo (浪費時間不好QQ)   2019-01-11 16:35:00
命名會看不下去的代表還太淺Zzz
作者: Noahdot (駁二宮和也)   2019-01-11 16:52:00
你還來啊?
作者: testPtt (測試)   2019-01-11 18:33:00
太舊的東西我會先想到重寫
作者: MOONY135 (談無慾)   2019-01-11 19:07:00
我覺得當你的同事...有點悲傷
作者: alihue (wanda wanda)   2019-01-11 19:24:00
886
作者: YahooTaiwan (超可愛南西我老婆)   2019-01-11 19:55:00
命名規則又沒有標準答案...
作者: joery (Lin)   2019-01-11 19:58:00
你確定牽一髮不會動全身,你確定整個系統你改一個地方有百分百,確定,絕對把握,再次確認一定不會影響其他地方或造成其他地方出問題。
作者: lonelytea (霸氣逼人)   2019-01-11 19:58:00
一定會被你改出問題啦 不用想了
作者: joery (Lin)   2019-01-11 19:59:00
你確定牽一髮不會動全身,你確定整個系統你改一個地方有百分百,確定,絕對把握,再次確認一定不會影響其他地方或造成其他地方出問題。不然如果真的出問題只會動目前影響的,而且看了好幾遍確認他流程邏輯找到可以下手的點改,順便重構該部份程式就好。不然萬一系統有問題,你就有得哭了
作者: stupid0319 (徵女友)   2019-01-11 21:08:00
沒有bug的話,不要改
作者: ChiangKeith (Wait)   2019-01-11 22:53:00
命名不要太初凡入聖根本不重要吧
作者: s001582000 (仁傑)   2019-01-11 23:22:00
我接到風行天的code都會重寫
作者: ChungLi5566 (中壢56哥)   2019-01-11 23:53:00
會這樣寫一定有原因 不熟的話別亂動例如弱掃沒過 多繞點路騙過弱掃軟體
作者: CloudyWing (孤單ㄉ翼)   2019-01-12 00:07:00
題外話C# public method 大駝峰反而是標準 XD現在的別人寫得就算看不下去,但只要穩定就不去動,除非被反應問題太多或是有些架構要去調整,才會整個整理
作者: y3k (激流を制するは静水)   2019-01-12 00:18:00
寫太難看改不動或有bug再改 因為也有可能是你沒能力...
作者: cphe (魔鬼藏在垃圾筒裡)   2019-01-12 00:31:00
改排版改命名沒什麼意義吧 這東西不同人寫就是不同風格除非一進公司就有明確規定,甚至coding style的script在掃
作者: shyangs (厚呦)   2019-01-12 02:10:00
方法 C# 大駝峰, Java 小駝峰; 一天寫個語言我常弄錯.
作者: wadechen (忙)   2019-01-12 02:33:00
一直執著在這問題 不覺得很辛苦嗎
作者: william81615 (別這樣玩我)   2019-01-12 06:16:00
樓上 這不就是典型上班沒事幹找碴嗎XD
作者: layolayo (藍天晴雨)   2019-01-12 07:47:00
吃飽太閒喔...通常半瓶水的人很容易覺得別人的code都是垃圾
作者: MOONY135 (談無慾)   2019-01-12 10:05:00
我只要能抄隊友的code我就一定不改
作者: ymcheung (ymc)   2019-01-12 10:11:00
但是前人留下的 CSS 不管動哪行都會改到其他頁面
作者: ab4daa (nooooooooooooooooooo)   2019-01-12 10:34:00
if it works, don't fix it!
作者: IhateOGC (我討厭)   2019-01-12 11:21:00
goto誰說不能用?該好好update了還停在70年代教科書?
作者: loadingN (sarsaparilla)   2019-01-12 12:10:00
你上班沒有其他更重要的事了嗎?
作者: tedmax100 (Nathan-Lu)   2019-01-12 13:16:00
以不變應萬變; 敵不動,我不動
作者: hanshsu (小肉呆)   2019-01-12 14:22:00
linux kernel 一堆 goto, 唉 怎麼辦?
作者: prag222 (prag)   2019-01-12 16:44:00
我寫按鍵精靈都用goto ....不過現在沒寫了
作者: Ghamu (貓丸)   2019-01-12 17:57:00
千萬不要改 爛code 爛命名 爛排版 就是讓老闆多請你進公司工作的主因 就是有這麼多拉低產能不是生產的雜活 才需要一直請工程師進來你是領錢辦事的 達則兼善天下 窮則獨善其身
作者: samblow2000 (貓哥大大)   2019-01-12 18:55:00
勸你不要改...自己寫都不敢保證沒問題了...
作者: doranako (真愛無限)   2019-01-12 19:29:00
有時間再改
作者: knives   2019-01-13 15:52:00
請問有出錯嗎,有主管指示嗎
作者: shooter555 (shooter)   2019-01-13 20:38:00
覺得goto有時候能讓程式碼簡潔0.0 還不錯用啊
作者: y3k (激流を制するは静水)   2019-01-14 19:22:00
goto是那種有能力用好的人不問 問的人一定不能讓他用的東西另外linux kernel是怎樣小弟不清楚 但是跟一堆asm()比起來我好奇大家會認為誰優誰劣XD
作者: fruithuang (黃水果)   2019-01-14 21:09:00
不出問題可以改,出問題就賴在你身上。系統忽然不穩都可賴說是你改的問題......
作者: a9228097 (逞強是我的專長)   2019-02-02 16:42:00
如果只改naming 跟format就用tool改一下xd

Links booklink

Contact Us: admin [ a t ] ucptt.com