Re: [閒聊] 寫code不加註解真的很顧人怨嗎

看板C_Chat (希洽)作者 (社畜)時間6月前 (2024/12/27 14:14), 編輯推噓6(8210)
留言20則, 11人參與, 6月前最新討論串6/6 (看更多)
還好八 反正註釋這種東西,也太詳細也沒人看 最多就函式功能、輸入輸出寫寫 然後複雜的演算法寫寫 我覺得程式寫的清楚比較重要 原廠給我的code,每個模組都用一堆全域變數 單獨跑可能還好,要讓兩個模組一起跑就各種奇怪的問題 測試也不全面 開發到一半,發現硬體動不起來 拿示波器測波形也沒出來 最後邊看code 邊看手冊 才發現他時鐘樹設錯 要自己除PLL頻率改參數 本來以為這已經很誇張了 但我同事會寫這樣的code 有個需求是調整倍率 然後調整之後 要在畫面上顯示一段時間消失 1、1.5、2、2.5、3這樣 通常來說可能就是 label、設定倍率(倍率)、timer(秒) 這樣的組合 先用設定倍率(倍率)設定label的值,然後設個timer多久隱藏 然後他這樣寫 label[0] label[1] label[2] label[3] label[4] timer0 timer1 timer2 timer3 timer4 每個label都是固定文字,選到哪個就用對應的定時器顯示N秒後消失 https://i.imgur.com/CLs8LGE.png
相比之下 我覺得註釋根本就可有可無 -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 220.134.21.189 (臺灣) ※ 文章網址: https://www.ptt.cc/bbs/C_Chat/M.1735280053.A.3A0.html

12/27 14:23, 6月前 , 1F
這才是實務經驗 註解根本不可信 浪費時間
12/27 14:23, 1F

12/27 14:23, 6月前 , 2F
你都說完了還有什麼問題
12/27 14:23, 2F

12/27 14:42, 6月前 , 3F
反正最後編譯起來都會變成沒有意義的文字(?)
12/27 14:42, 3F

12/27 14:53, 6月前 , 4F
實務經驗個屁啊,過猶不及。
12/27 14:53, 4F

12/27 15:04, 6月前 , 5F
還好吧,這樣寫到時候改成別的訊息很方便啊
12/27 15:04, 5F

12/27 15:04, 6月前 , 6F
當註解寫Dont remove 或Dont modify配上一個十年前的
12/27 15:04, 6F

12/27 15:04, 6月前 , 7F
日期以及一個不在的人命,敢改就看你八字夠不夠重
12/27 15:04, 7F

12/27 15:34, 6月前 , 8F
這個寫法就是實務上可行,但我前公司的code review只
12/27 15:34, 8F

12/27 15:34, 6月前 , 9F
要看的人用0.5就可以讓對方飛到天花板上
12/27 15:34, 9F

12/27 15:34, 6月前 , 10F
當然你們如果沒有這樣的code review那我也沒差
12/27 15:34, 10F

12/27 15:41, 6月前 , 11F
能讓人飛到天花板的理由很簡單,你現在知道要求是1/1
12/27 15:41, 11F

12/27 15:41, 6月前 , 12F
.5/2/2.5/3倍,那你身為一個合格的員工應該要想像到
12/27 15:41, 12F

12/27 15:41, 6月前 , 13F
以後會有0.25/1.75等需求,在沒有特別要求的情況下就
12/27 15:41, 13F

12/27 15:41, 6月前 , 14F
應該自動刻出一個generalize的功能
12/27 15:41, 14F

12/27 15:41, 6月前 , 15F
老實說這理由很見正大光明,我看到誰被電的話也只會
12/27 15:41, 15F

12/27 15:41, 6月前 , 16F
笑他活該,但你們公司如果不想這樣電人也不是不行
12/27 15:41, 16F

12/27 17:00, 6月前 , 17F
你是不是沒看過主管沒一個會軟體的公司
12/27 17:00, 17F

12/27 17:46, 6月前 , 18F
你同事的寫法才是比較實在的,你講的hard code寫法才糟
12/27 17:46, 18F

12/27 17:56, 6月前 , 19F
能動就好
12/27 17:56, 19F

12/27 19:13, 6月前 , 20F
遇到這種奇葩的PG同事,你還忍得住算你猛。
12/27 19:13, 20F
文章代碼(AID): #1dRaMrEW (C_Chat)
文章代碼(AID): #1dRaMrEW (C_Chat)